Changeset 263995 in webkit

Timestamp:

Jul 6, 2020, 4:27:56 PM (5 years ago)

Author:

dbates@webkit.org

Message:

[iOS] WAKWindow should override -resignFirstResponder and clear state
​https://bugs.webkit.org/show_bug.cgi?id=214011
<rdar://problem/65152410>

Reviewed by Tim Horton.

This is part of the fix for <rdar://problem/62615273>.

Override -resignFirstResponder to clear out internal state that tracks the first responder
in the iOS Legacy WebKit world.

• platform/ios/wak/WAKWindow.mm:

(-[WAKWindow resignFirstResponder]):

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• platform/ios/wak/WAKWindow.mm (modified) (1 diff)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2020-07-06  Daniel Bates  <dabates@apple.com>
+
+        [iOS] WAKWindow should override -resignFirstResponder and clear state
+        https://bugs.webkit.org/show_bug.cgi?id=214011
+        <rdar://problem/65152410>
+
+        Reviewed by Tim Horton.
+
+        This is part of the fix for <rdar://problem/62615273>.
+
+        Override -resignFirstResponder to clear out internal state that tracks the first responder
+        in the iOS Legacy WebKit world.
+
+        * platform/ios/wak/WAKWindow.mm:
+        (-[WAKWindow resignFirstResponder]):
+
 2020-07-06  Sam Weinig  <weinig@apple.com>
 

trunk/Source/WebCore/platform/ios/wak/WAKWindow.mm

@@ -210 +210 @@
 }
 
+- (BOOL)resignFirstResponder
+{
+    BOOL shouldResign = [super resignFirstResponder];
+    if (shouldResign && _responderView && WKViewResignFirstResponder([_responderView _viewRef])) {
+        _nextResponder = nil;
+        [_responderView release];
+        _responderView = nil;
+        return YES;
+    }
+    return NO;
+}
+
 - (BOOL)makeFirstResponder:(NSResponder *)aResponder
 {