Title: [260846] trunk/Source/WebKit
Revision
260846
Author
[email protected]
Date
2020-04-28 14:31:34 -0700 (Tue, 28 Apr 2020)

Log Message

Remove unused WebPage::focusTextInputContext()
https://bugs.webkit.org/show_bug.cgi?id=211135

Reviewed by Anders Carlsson.

I accidentally forgot to remove the WebPage message and WebPage implementation that
backed -_focusTextInputContext when I removed it in r260225. Now the implementation
is unused. Let's remove it.

* WebProcess/WebPage/WebPage.cpp:
(WebKit::WebPage::focusTextInputContext): Deleted.
* WebProcess/WebPage/WebPage.h:
* WebProcess/WebPage/WebPage.messages.in:

Modified Paths

Diff

Modified: trunk/Source/WebKit/ChangeLog (260845 => 260846)


--- trunk/Source/WebKit/ChangeLog	2020-04-28 21:15:33 UTC (rev 260845)
+++ trunk/Source/WebKit/ChangeLog	2020-04-28 21:31:34 UTC (rev 260846)
@@ -1,3 +1,19 @@
+2020-04-28  Daniel Bates  <[email protected]>
+
+        Remove unused WebPage::focusTextInputContext()
+        https://bugs.webkit.org/show_bug.cgi?id=211135
+
+        Reviewed by Anders Carlsson.
+
+        I accidentally forgot to remove the WebPage message and WebPage implementation that
+        backed -_focusTextInputContext when I removed it in r260225. Now the implementation
+        is unused. Let's remove it.
+
+        * WebProcess/WebPage/WebPage.cpp:
+        (WebKit::WebPage::focusTextInputContext): Deleted.
+        * WebProcess/WebPage/WebPage.h:
+        * WebProcess/WebPage/WebPage.messages.in:
+
 2020-04-28  Christopher Reid  <[email protected]>
 
         [Win] Bundle Inspector Resources in Release builds

Modified: trunk/Source/WebKit/WebProcess/WebPage/WebPage.cpp (260845 => 260846)


--- trunk/Source/WebKit/WebProcess/WebPage/WebPage.cpp	2020-04-28 21:15:33 UTC (rev 260845)
+++ trunk/Source/WebKit/WebProcess/WebPage/WebPage.cpp	2020-04-28 21:31:34 UTC (rev 260846)
@@ -6932,16 +6932,6 @@
     completionHandler(contexts);
 }
 
-void WebPage::focusTextInputContext(const WebCore::ElementContext& textInputContext, CompletionHandler<void(bool)>&& completionHandler)
-{
-    auto element = elementForContext(textInputContext);
-
-    if (element)
-        element->focus();
-
-    completionHandler(element);
-}
-
 void WebPage::setCanShowPlaceholder(const WebCore::ElementContext& elementContext, bool canShowPlaceholder)
 {
     RefPtr<Element> element = elementForContext(elementContext);

Modified: trunk/Source/WebKit/WebProcess/WebPage/WebPage.h (260845 => 260846)


--- trunk/Source/WebKit/WebProcess/WebPage/WebPage.h	2020-04-28 21:15:33 UTC (rev 260845)
+++ trunk/Source/WebKit/WebProcess/WebPage/WebPage.h	2020-04-28 21:31:34 UTC (rev 260846)
@@ -645,7 +645,6 @@
     void selectAll();
 
     void textInputContextsInRect(WebCore::FloatRect, CompletionHandler<void(const Vector<WebCore::ElementContext>&)>&&);
-    void focusTextInputContext(const WebCore::ElementContext&, CompletionHandler<void(bool)>&&);
     void setCanShowPlaceholder(const WebCore::ElementContext&, bool);
 
 #if PLATFORM(IOS_FAMILY)

Modified: trunk/Source/WebKit/WebProcess/WebPage/WebPage.messages.in (260845 => 260846)


--- trunk/Source/WebKit/WebProcess/WebPage/WebPage.messages.in	2020-04-28 21:15:33 UTC (rev 260845)
+++ trunk/Source/WebKit/WebProcess/WebPage/WebPage.messages.in	2020-04-28 21:31:34 UTC (rev 260846)
@@ -578,7 +578,6 @@
 #endif
 
     TextInputContextsInRect(WebCore::FloatRect rect) -> (Vector<struct WebCore::ElementContext> contexts) Async
-    FocusTextInputContext(struct WebCore::ElementContext context) -> (bool success) Async
     SetCanShowPlaceholder(struct WebCore::ElementContext context, bool canShowPlaceholder)
 
 #if ENABLE(RESOURCE_LOAD_STATISTICS)
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to