a bug
Created attachment 231438 [details] patch
Comment on attachment 231438 [details] patch View in context: https://bugs.webkit.org/attachment.cgi?id=231438&action=review OK > Source/WebKit/mac/WebView/WebView.mm:1581 > + FrameView* view = [self _mainCoreFrame]->view(); I think "FrameView *view" is the appropriate style here.
http://trac.webkit.org/changeset/168830
Comment on attachment 231438 [details] patch View in context: https://bugs.webkit.org/attachment.cgi?id=231438&action=review >> Source/WebKit/mac/WebView/WebView.mm:1581 >> + FrameView* view = [self _mainCoreFrame]->view(); > > I think "FrameView *view" is the appropriate style here. No, the style is right. We do that different formatting thing with * only for Objective-C types, not for C++ types like FrameView. We should probably change the rule though. The only place where we need the "X *" style is in public Objective-C headers. Inside the code it would be easier if we adopted a consistent rule.