Title: [259183] trunk/Source/WebKit
Revision
259183
Author
[email protected]
Date
2020-03-29 17:48:55 -0700 (Sun, 29 Mar 2020)

Log Message

Attempt to fix the Mac Catalyst build after r258989
<https://bugs.webkit.org/show_bug.cgi?id=199960>

Remove code that has since been moved to WebPage::platformNeedsLayoutForEditorState().

* WebProcess/WebPage/ios/WebPageIOS.mm:
(WebKit::WebPage::getPlatformEditorState const):

Modified Paths

Diff

Modified: trunk/Source/WebKit/ChangeLog (259182 => 259183)


--- trunk/Source/WebKit/ChangeLog	2020-03-29 17:34:16 UTC (rev 259182)
+++ trunk/Source/WebKit/ChangeLog	2020-03-30 00:48:55 UTC (rev 259183)
@@ -1,3 +1,13 @@
+2020-03-29  Daniel Bates  <[email protected]>
+
+        Attempt to fix the Mac Catalyst build after r258989
+        <https://bugs.webkit.org/show_bug.cgi?id=199960>
+
+        Remove code that has since been moved to WebPage::platformNeedsLayoutForEditorState().
+
+        * WebProcess/WebPage/ios/WebPageIOS.mm:
+        (WebKit::WebPage::getPlatformEditorState const):
+
 2020-03-29  [email protected]  <[email protected]@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
 
         Build fix for: [iOS] Delay process suspension for a while after loading an app link

Modified: trunk/Source/WebKit/WebProcess/WebPage/ios/WebPageIOS.mm (259182 => 259183)


--- trunk/Source/WebKit/WebProcess/WebPage/ios/WebPageIOS.mm	2020-03-29 17:34:16 UTC (rev 259182)
+++ trunk/Source/WebKit/WebProcess/WebPage/ios/WebPageIOS.mm	2020-03-30 00:48:55 UTC (rev 259183)
@@ -260,14 +260,8 @@
     }
 
     // We only set the remaining EditorState entries if layout is done as a performance optimization
-    // to avoid the need to force a synchronous layout here to compute these entries. If we
-    // have a composition or are using a hardware keyboard then we send the full editor state
-    // immediately so that the UIProcess can update UI, including the position of the caret.
+    // to avoid the need to force a synchronous layout here to compute these entries.
     bool needsLayout = view->needsLayout();
-    bool requiresPostLayoutData = frame.editor().hasComposition();
-#if !PLATFORM(MACCATALYST)
-    requiresPostLayoutData |= m_keyboardIsAttached;
-#endif
     if (needsLayout || result.isMissingPostLayoutData)
         return;
 
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to