Title: [150969] trunk/Source/WebCore
Revision
150969
Author
[email protected]
Date
2013-05-30 10:09:07 -0700 (Thu, 30 May 2013)

Log Message

Fix double hash lookup in DocumentEventQueue::cancelEvent().
<http://webkit.org/b/117031>

Reviewed by Anders Carlsson.

* dom/DocumentEventQueue.cpp:
(WebCore::DocumentEventQueue::cancelEvent):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (150968 => 150969)


--- trunk/Source/WebCore/ChangeLog	2013-05-30 17:06:52 UTC (rev 150968)
+++ trunk/Source/WebCore/ChangeLog	2013-05-30 17:09:07 UTC (rev 150969)
@@ -1,5 +1,15 @@
 2013-05-30  Andreas Kling  <[email protected]>
 
+        Fix double hash lookup in DocumentEventQueue::cancelEvent().
+        <http://webkit.org/b/117031>
+
+        Reviewed by Anders Carlsson.
+
+        * dom/DocumentEventQueue.cpp:
+        (WebCore::DocumentEventQueue::cancelEvent):
+
+2013-05-30  Andreas Kling  <[email protected]>
+
         Fix double hash lookup in DocumentLoader::removeSubresourceLoader().
         <http://webkit.org/b/117030>
 

Modified: trunk/Source/WebCore/dom/DocumentEventQueue.cpp (150968 => 150969)


--- trunk/Source/WebCore/dom/DocumentEventQueue.cpp	2013-05-30 17:06:52 UTC (rev 150968)
+++ trunk/Source/WebCore/dom/DocumentEventQueue.cpp	2013-05-30 17:09:07 UTC (rev 150969)
@@ -98,8 +98,10 @@
 
 bool DocumentEventQueue::cancelEvent(Event* event)
 {
-    bool found = m_queuedEvents.contains(event);
-    m_queuedEvents.remove(event);
+    ListHashSet<RefPtr<Event>, 16>::iterator it = m_queuedEvents.find(event);
+    bool found = it != m_queuedEvents.end();
+    if (found)
+        m_queuedEvents.remove(it);
     if (m_queuedEvents.isEmpty())
         m_pendingEventTimer->stop();
     return found;
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to