[Cocoa] Remove knowledge of the WKView from the WebFullScreenManagerProxy by adding a proper client
Created attachment 218714 [details] Patch
Attachment 218714 [details] did not pass style-queue: Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/WebKit2/ChangeLog', u'Source/WebKit2/UIProcess/API/Cocoa/WKViewPrivate.h', u'Source/WebKit2/UIProcess/API/mac/PageClientImpl.h', u'Source/WebKit2/UIProcess/API/mac/PageClientImpl.mm', u'Source/WebKit2/UIProcess/API/mac/WKView+FullScreen.h', u'Source/WebKit2/UIProcess/API/mac/WKView+FullScreen.mm', u'Source/WebKit2/UIProcess/API/mac/WKView.mm', u'Source/WebKit2/UIProcess/API/mac/WKViewInternal.h', u'Source/WebKit2/UIProcess/PageClient.h', u'Source/WebKit2/UIProcess/WebFullScreenManagerProxy.cpp', u'Source/WebKit2/UIProcess/WebFullScreenManagerProxy.h', u'Source/WebKit2/UIProcess/WebPageProxy.cpp', u'Source/WebKit2/UIProcess/mac/WKFullScreenWindowController.mm', u'Source/WebKit2/UIProcess/mac/WebFullScreenManagerProxyMac.mm', u'Source/WebKit2/WebKit2.xcodeproj/project.pbxproj', '--commit-queue']" exit_code: 1 ERROR: Source/WebKit2/UIProcess/API/mac/WKViewInternal.h:53: Place brace on its own line for function definitions. [whitespace/braces] [4] Total errors found: 1 in 13 files If any of these errors are false positives, please file a bug against check-webkit-style.
Comment on attachment 218714 [details] Patch Attachment 218714 [details] did not pass efl-wk2-ews (efl-wk2): Output: http://webkit-queues.appspot.com/results/47158094
Created attachment 218716 [details] Patch
Comment on attachment 218716 [details] Patch Attachment 218716 [details] did not pass efl-wk2-ews (efl-wk2): Output: http://webkit-queues.appspot.com/results/44478116
Comment on attachment 218716 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=218716&action=review > Source/WebKit2/UIProcess/API/mac/PageClientImpl.mm:567 > +#endif > + > +#if ENABLE(FULLSCREEN_API) This is unnecessary > Source/WebKit2/UIProcess/API/mac/PageClientImpl.mm:580 > + if (!m_wkView.hasFullScreenWindowController) > + return false; > + Is this part necessary? > Source/WebKit2/UIProcess/API/mac/PageClientImpl.mm:603 > +#endif This #endif could use a newline before it and a comment after it. > Source/WebKit2/UIProcess/API/mac/WKViewInternal.h:98 > +@property (readonly) BOOL hasFullScreenWindowController; > +@property (readonly) WKFullScreenWindowController *fullScreenWindowController; > +- (void)closeFullScreenWindowController; As internal methods and properties, these should probably be prefixed with an underscore.
Comment on attachment 218716 [details] Patch Attachment 218716 [details] did not pass win-ews (win): Output: http://webkit-queues.appspot.com/results/45388138
Committed r160296: <http://trac.webkit.org/changeset/160296>