Changeset 204494 in webkit

Timestamp:

Aug 15, 2016, 8:32:38 PM (9 years ago)

Author:

Chris Dumez

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:

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• dom/EventListenerMap.cpp (modified) (1 diff)
• dom/EventListenerMap.h (modified) (1 diff)
• dom/EventTarget.cpp (modified) (1 diff)
• dom/EventTarget.h (modified) (1 diff)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2016-08-15  Chris Dumez  <cdumez@apple.com>
+
+        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  <sam@webkit.org>
 

trunk/Source/WebCore/dom/EventListenerMap.cpp

@@ -68 +68 @@
 }
 
-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()
 {

trunk/Source/WebCore/dom/EventListenerMap.h

@@ -52 +52 @@
     bool contains(const AtomicString& eventType) const { return find(eventType); }
     bool containsCapturing(const AtomicString& eventType) const;
-    bool containsActive(const AtomicString& eventType) const;
 
     void clear();

trunk/Source/WebCore/dom/EventTarget.cpp

@@ -123 +123 @@
 }
 
-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)
 {

trunk/Source/WebCore/dom/EventTarget.h

@@ -159 +159 @@
     bool hasEventListeners(const AtomicString& eventType) const;
     bool hasCapturingEventListeners(const AtomicString& eventType);
-    bool hasActiveEventListeners(const AtomicString& eventType) const;
     const EventListenerVector& getEventListeners(const AtomicString& eventType);