Bug 125427 – [Cocoa] Remove knowledge of the WKView from the WebFullScreenManagerProxy by adding a proper client

• Status: RESOLVED
• Resolution: FIXED
• Component: New Bugs

[Sam Weinig]

[Cocoa] Remove knowledge of the WKView from the WebFullScreenManagerProxy by adding a proper client

[Sam Weinig]

Created attachment 218714 [details]
Patch

[WebKit Commit Bot]

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.

[EFL EWS Bot]

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

[Sam Weinig]

Created attachment 218716 [details]
Patch

[EFL EWS Bot]

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.

[Build Bot]

Comment on attachment 218716 [details]
Patch

Attachment 218716 [details] did not pass win-ews (win):
Output: http://webkit-queues.appspot.com/results/45388138

[Sam Weinig]

Committed r160296: <http://trac.webkit.org/changeset/160296>