Bug 186088 - Make WKWebView firstResponder after entering or exiting fullscreen.
Summary: Make WKWebView firstResponder after entering or exiting fullscreen.
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit2 (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Jeremy Jones
URL:
Keywords: InRadar
: 186087 (view as bug list)
Depends on:
Blocks:
 
Reported: 2018-05-30 04:25 PDT by Jeremy Jones
Modified: 2018-06-02 09:03 PDT (History)
4 users (show)

See Also:


Attachments
Patch (2.04 KB, patch)
2018-05-30 04:31 PDT, Jeremy Jones
no flags Details | Formatted Diff | Diff
Patch (2.05 KB, patch)
2018-06-01 16:50 PDT, Jeremy Jones
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Jeremy Jones 2018-05-30 04:25:21 PDT
Make WKWebView firstResponder after entering or exiting fullscreen.
Comment 1 Jeremy Jones 2018-05-30 04:25:46 PDT
*** Bug 186087 has been marked as a duplicate of this bug. ***
Comment 2 Jeremy Jones 2018-05-30 04:27:13 PDT
rdar://problem/40387859
Comment 3 Jeremy Jones 2018-05-30 04:31:51 PDT
Created attachment 341561 [details]
Patch
Comment 4 Jon Lee 2018-05-30 07:56:59 PDT
Comment on attachment 341561 [details]
Patch

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

> Source/WebKit/UIProcess/ios/fullscreen/WKFullScreenWindowControllerIOS.mm:538
> +            [webView becomeFirstResponder];

This block inconsistently references the web view with a local variable and a member variable. Which one should it be?
Comment 5 Jeremy Jones 2018-06-01 16:50:24 PDT
Created attachment 341806 [details]
Patch
Comment 6 Jeremy Jones 2018-06-01 17:02:51 PDT
(In reply to Jon Lee from comment #4)
> Comment on attachment 341561 [details]
> Patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=341561&action=review
> 
> > Source/WebKit/UIProcess/ios/fullscreen/WKFullScreenWindowControllerIOS.mm:538
> > +            [webView becomeFirstResponder];
> 
> This block inconsistently references the web view with a local variable and
> a member variable. Which one should it be?

Fixed.
Comment 7 WebKit Commit Bot 2018-06-02 09:03:36 PDT
Comment on attachment 341806 [details]
Patch

Clearing flags on attachment: 341806

Committed r232438: <https://trac.webkit.org/changeset/232438>
Comment 8 WebKit Commit Bot 2018-06-02 09:03:38 PDT
All reviewed patches have been landed.  Closing bug.