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