Remove unused webkitEditableContentChanged event
authorrniwa@webkit.org <rniwa@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 18 Mar 2019 21:57:54 +0000 (21:57 +0000)
committerrniwa@webkit.org <rniwa@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 18 Mar 2019 21:57:54 +0000 (21:57 +0000)
https://bugs.webkit.org/show_bug.cgi?id=195909

Reviewed by Wenson Hsieh.

Removed webkitEditableContentChanged event which is no longer used after r206944.

Note that only WebKit's C++ code can fire events in TextControlInnerElements
since the element is an implementation detail of input and textarea elements.

* dom/EventNames.h:
* html/shadow/TextControlInnerElements.cpp:
(WebCore::TextControlInnerTextElement::defaultEventHandler):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@243106 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebCore/ChangeLog
Source/WebCore/dom/EventNames.h
Source/WebCore/html/shadow/TextControlInnerElements.cpp

index 9e2c539..94b3c52 100644 (file)
@@ -1,3 +1,19 @@
+2019-03-18  Ryosuke Niwa  <rniwa@webkit.org>
+
+        Remove unused webkitEditableContentChanged event
+        https://bugs.webkit.org/show_bug.cgi?id=195909
+
+        Reviewed by Wenson Hsieh.
+
+        Removed webkitEditableContentChanged event which is no longer used after r206944.
+
+        Note that only WebKit's C++ code can fire events in TextControlInnerElements
+        since the element is an implementation detail of input and textarea elements.
+
+        * dom/EventNames.h:
+        * html/shadow/TextControlInnerElements.cpp:
+        (WebCore::TextControlInnerTextElement::defaultEventHandler):
+
 2019-03-18  Myles C. Maxfield  <mmaxfield@apple.com>
 
         [WHLSL] Hook up the compiler to our WebGPU implementation
index f4d7d73..b20302a 100644 (file)
@@ -302,7 +302,6 @@ namespace WebCore {
     macro(webkitAnimationIteration) \
     macro(webkitAnimationStart) \
     macro(webkitBeforeTextInserted) \
-    macro(webkitEditableContentChanged) \
     macro(webkitTransitionEnd) \
     macro(webkitbeginfullscreen) \
     macro(webkitcurrentplaybacktargetiswirelesschanged) \
index 0b14679..3c67e7d 100644 (file)
@@ -148,15 +148,14 @@ void TextControlInnerTextElement::defaultEventHandler(Event& event)
     // FIXME: In the future, we should add a way to have default event listeners.
     // Then we would add one to the text field's inner div, and we wouldn't need this subclass.
     // Or possibly we could just use a normal event listener.
-    if (event.isBeforeTextInsertedEvent() || event.type() == eventNames().webkitEditableContentChangedEvent) {
-        RefPtr<Element> shadowAncestor = shadowHost();
+    if (event.isBeforeTextInsertedEvent()) {
         // A TextControlInnerTextElement can have no host if its been detached,
         // but kept alive by an EditCommand. In this case, an undo/redo can
         // cause events to be sent to the TextControlInnerTextElement. To
         // prevent an infinite loop, we must check for this case before sending
         // the event up the chain.
-        if (shadowAncestor)
-            shadowAncestor->defaultEventHandler(event);
+        if (auto host = makeRefPtr(shadowHost()))
+            host->defaultEventHandler(event);
     }
     if (!event.defaultHandled())
         HTMLDivElement::defaultEventHandler(event);