Title: [150967] trunk/Source/WebCore
Revision
150967
Author
akl...@apple.com
Date
2013-05-30 10:05:33 -0700 (Thu, 30 May 2013)

Log Message

Fix double hash lookup in DocumentLoader::removeSubresourceLoader().
<http://webkit.org/b/117030>

Reviewed by Anders Carlsson.

* loader/DocumentLoader.cpp:
(WebCore::DocumentLoader::removeSubresourceLoader):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (150966 => 150967)


--- trunk/Source/WebCore/ChangeLog	2013-05-30 17:03:53 UTC (rev 150966)
+++ trunk/Source/WebCore/ChangeLog	2013-05-30 17:05:33 UTC (rev 150967)
@@ -1,5 +1,15 @@
 2013-05-30  Andreas Kling  <akl...@apple.com>
 
+        Fix double hash lookup in DocumentLoader::removeSubresourceLoader().
+        <http://webkit.org/b/117030>
+
+        Reviewed by Anders Carlsson.
+
+        * loader/DocumentLoader.cpp:
+        (WebCore::DocumentLoader::removeSubresourceLoader):
+
+2013-05-30  Andreas Kling  <akl...@apple.com>
+
         Fix double hash lookup in RenderView::repaintSelection() loop.
         <http://webkit.org/b/117029>
 

Modified: trunk/Source/WebCore/loader/DocumentLoader.cpp (150966 => 150967)


--- trunk/Source/WebCore/loader/DocumentLoader.cpp	2013-05-30 17:03:53 UTC (rev 150966)
+++ trunk/Source/WebCore/loader/DocumentLoader.cpp	2013-05-30 17:05:33 UTC (rev 150967)
@@ -1287,9 +1287,10 @@
 
 void DocumentLoader::removeSubresourceLoader(ResourceLoader* loader)
 {
-    if (!m_subresourceLoaders.contains(loader))
+    ResourceLoaderSet::iterator it = m_subresourceLoaders.find(loader);
+    if (it == m_subresourceLoaders.end())
         return;
-    m_subresourceLoaders.remove(loader);
+    m_subresourceLoaders.remove(it);
     checkLoadComplete();
     if (Frame* frame = m_frame)
         frame->loader()->checkLoadComplete();
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to