Title: [204494] trunk/Source/WebCore
Revision
204494
Author
[email protected]
Date
2016-08-15 20:32:38 -0700 (Mon, 15 Aug 2016)

Log Message

Drop unused EventTarget::hasActiveEventListeners()
https://bugs.webkit.org/show_bug.cgi?id=160869

Reviewed by Alex Christensen.

* dom/EventListenerMap.cpp:
(WebCore::EventListenerMap::containsActive): Deleted.
* dom/EventListenerMap.h:
* dom/EventTarget.cpp:
(WebCore::EventTarget::hasActiveEventListeners): Deleted.
* dom/EventTarget.h:

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (204493 => 204494)


--- trunk/Source/WebCore/ChangeLog	2016-08-16 02:27:09 UTC (rev 204493)
+++ trunk/Source/WebCore/ChangeLog	2016-08-16 03:32:38 UTC (rev 204494)
@@ -1,3 +1,17 @@
+2016-08-15  Chris Dumez  <[email protected]>
+
+        Drop unused EventTarget::hasActiveEventListeners()
+        https://bugs.webkit.org/show_bug.cgi?id=160869
+
+        Reviewed by Alex Christensen.
+
+        * dom/EventListenerMap.cpp:
+        (WebCore::EventListenerMap::containsActive): Deleted.
+        * dom/EventListenerMap.h:
+        * dom/EventTarget.cpp:
+        (WebCore::EventTarget::hasActiveEventListeners): Deleted.
+        * dom/EventTarget.h:
+
 2016-08-15  Sam Weinig  <[email protected]>
 
         Speed up compile times by not including wtf/Variant.h so much

Modified: trunk/Source/WebCore/dom/EventListenerMap.cpp (204493 => 204494)


--- trunk/Source/WebCore/dom/EventListenerMap.cpp	2016-08-16 02:27:09 UTC (rev 204493)
+++ trunk/Source/WebCore/dom/EventListenerMap.cpp	2016-08-16 03:32:38 UTC (rev 204494)
@@ -67,19 +67,6 @@
     return false;
 }
 
-bool EventListenerMap::containsActive(const AtomicString& eventType) const
-{
-    auto* listeners = find(eventType);
-    if (!listeners)
-        return false;
-
-    for (auto& eventListener : *listeners) {
-        if (!eventListener->isPassive())
-            return true;
-    }
-    return false;
-}
-
 void EventListenerMap::clear()
 {
     assertNoActiveIterators();

Modified: trunk/Source/WebCore/dom/EventListenerMap.h (204493 => 204494)


--- trunk/Source/WebCore/dom/EventListenerMap.h	2016-08-16 02:27:09 UTC (rev 204493)
+++ trunk/Source/WebCore/dom/EventListenerMap.h	2016-08-16 03:32:38 UTC (rev 204494)
@@ -51,7 +51,6 @@
     bool isEmpty() const { return m_entries.isEmpty(); }
     bool contains(const AtomicString& eventType) const { return find(eventType); }
     bool containsCapturing(const AtomicString& eventType) const;
-    bool containsActive(const AtomicString& eventType) const;
 
     void clear();
 

Modified: trunk/Source/WebCore/dom/EventTarget.cpp (204493 => 204494)


--- trunk/Source/WebCore/dom/EventTarget.cpp	2016-08-16 02:27:09 UTC (rev 204493)
+++ trunk/Source/WebCore/dom/EventTarget.cpp	2016-08-16 03:32:38 UTC (rev 204494)
@@ -122,14 +122,6 @@
     return nullptr;
 }
 
-bool EventTarget::hasActiveEventListeners(const AtomicString& eventType) const
-{
-    EventTargetData* eventTargetData = const_cast<EventTarget*>(this)->eventTargetData();
-    if (!eventTargetData)
-        return false;
-    return eventTargetData->eventListenerMap.containsActive(eventType);
-}
-
 bool EventTarget::clearAttributeEventListener(const AtomicString& eventType)
 {
     EventListener* listener = getAttributeEventListener(eventType);

Modified: trunk/Source/WebCore/dom/EventTarget.h (204493 => 204494)


--- trunk/Source/WebCore/dom/EventTarget.h	2016-08-16 02:27:09 UTC (rev 204493)
+++ trunk/Source/WebCore/dom/EventTarget.h	2016-08-16 03:32:38 UTC (rev 204494)
@@ -158,7 +158,6 @@
     bool hasEventListeners() const;
     bool hasEventListeners(const AtomicString& eventType) const;
     bool hasCapturingEventListeners(const AtomicString& eventType);
-    bool hasActiveEventListeners(const AtomicString& eventType) const;
     const EventListenerVector& getEventListeners(const AtomicString& eventType);
 
     bool fireEventListeners(Event&);
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to