Bug 178252 – Disambiguate name colision in element and video fullscreen delegate protocol selector names.

• Status: NEW
• Component: WebKit2

[Jeremy Jones]

Disambiguate name colision in element and video fullscreen delegate protocol selector names.

[Jeremy Jones]

Created attachment 323633 [details]
Patch

[Jeremy Jones]

Created attachment 323634 [details]
Patch

[Simon Fraser (smfr)]

Comment on attachment 323634 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=323634&action=review

> Source/WebKit/ChangeLog:9
> +        but take different param types. This means that a class cannot implement both of these delegates protocols becuase of the ambiguous

becuase

> Source/WebKit/UIProcess/API/Cocoa/WKUIDelegatePrivate.h:98
> +- (void)_webViewDidEnterVideoFullscreen:(WKWebView *)webView WK_API_AVAILABLE(macosx(10.14), ios(12.0));
> +- (void)_webViewDidExitVideoFullscreen:(WKWebView *)webView WK_API_AVAILABLE(macosx(10.14), ios(12.0));

Are we already using WK_API_AVAILABLE(macosx(10.14), ios(12.0));? I thought we did something else until later.

> Source/WebKit/UIProcess/API/Cocoa/_WKFullscreenDelegate.h:43
> +- (void)_webViewWillEnterWebFullscreen:(NSView *)webView WK_API_AVAILABLE(macosx(10.14), ios(12.0));
> +- (void)_webViewDidEnterWebFullscreen:(NSView *)webView WK_API_AVAILABLE(macosx(10.14), ios(12.0));
> +- (void)_webViewWillExitWebFullscreen:(NSView *)webView WK_API_AVAILABLE(macosx(10.14), ios(12.0));
> +- (void)_webViewDidExitWebFullscreen:(NSView *)webView WK_API_AVAILABLE(macosx(10.14), ios(12.0));

Why doe these take an NSView, rather than a WKWebView?

[Tim Horton]

Comment on attachment 323634 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=323634&action=review

>> Source/WebKit/UIProcess/API/Cocoa/WKUIDelegatePrivate.h:98
>> +- (void)_webViewDidExitVideoFullscreen:(WKWebView *)webView WK_API_AVAILABLE(macosx(10.14), ios(12.0));
> 
> Are we already using WK_API_AVAILABLE(macosx(10.14), ios(12.0));? I thought we did something else until later.

Please use the TBA macros!

>> Source/WebKit/UIProcess/API/Cocoa/_WKFullscreenDelegate.h:43
>> +- (void)_webViewDidExitWebFullscreen:(NSView *)webView WK_API_AVAILABLE(macosx(10.14), ios(12.0));
> 
> Why doe these take an NSView, rather than a WKWebView?

WKView/WKWebView.

[Jeremy Jones]

(In reply to Tim Horton from comment #4)
> Comment on attachment 323634 [details]
> Patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=323634&action=review
> 
> >> Source/WebKit/UIProcess/API/Cocoa/WKUIDelegatePrivate.h:98
> >> +- (void)_webViewDidExitVideoFullscreen:(WKWebView *)webView WK_API_AVAILABLE(macosx(10.14), ios(12.0));
> > 
> > Are we already using WK_API_AVAILABLE(macosx(10.14), ios(12.0));? I thought we did something else until later.
> 
> Please use the TBA macros!

Done

> 
> >> Source/WebKit/UIProcess/API/Cocoa/_WKFullscreenDelegate.h:43
> >> +- (void)_webViewDidExitWebFullscreen:(NSView *)webView WK_API_AVAILABLE(macosx(10.14), ios(12.0));
> > 
> > Why doe these take an NSView, rather than a WKWebView?
> 
> WKView/WKWebView.

[Jeremy Jones]

Created attachment 323965 [details]
Patch

[Frédéric Wang (:fredw)]

Comment on attachment 323965 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=323965&action=review

@Jeremy: What's the status of this?

> Source/WebKit/ChangeLog:9
> +        but take different param types. This means that a class cannot implement both of these delegates protocols becuase of the ambiguous

You forgot to fix s/becuase/because/

> Source/WebKit/UIProcess/API/Cocoa/_WKFullscreenDelegate.h:43
> +- (void)_webViewDidExitWebFullscreen:(NSView *)webView WK_API_AVAILABLE(macosx(WK_MAC_TBA), ios(WK_IOS_TBA));

It looks like your comment on why NSView is used was not published.