Changeset 73191 in webkit

Timestamp:

Dec 2, 2010 1:22:42 PM (13 years ago)

Author:

Joseph Pecoraro

Message:

2010-12-02 Joseph Pecoraro <Joseph Pecoraro>

Reviewed by Darin Adler.

Update ResourceLoaderSet Enumeration
​https://bugs.webkit.org/show_bug.cgi?id=50405

• loader/DocumentLoader.cpp: (WebCore::cancelAll): instead of copying the HashSet, use copyToVector. (WebCore::setAllDefersLoading): instead of copying the HashSet, use copyToVector.

Location:

trunk/WebCore

Files:

• ChangeLog (modified) (1 diff)
• loader/DocumentLoader.cpp (modified) (1 diff)

trunk/WebCore/ChangeLog

@@ +1 @@
+2010-12-02  Joseph Pecoraro  <joepeck@webkit.org>
+
+        Reviewed by Darin Adler.
+
+        Update ResourceLoaderSet Enumeration
+        https://bugs.webkit.org/show_bug.cgi?id=50405
+
+        * loader/DocumentLoader.cpp:
+        (WebCore::cancelAll): instead of copying the HashSet, use copyToVector.
+        (WebCore::setAllDefersLoading): instead of copying the HashSet, use copyToVector.
+
 2010-12-02  Andy Estes  <aestes@apple.com>
 

trunk/WebCore/loader/DocumentLoader.cpp

@@ -59 +59 @@
 static void cancelAll(const ResourceLoaderSet& loaders)
 {
-    const ResourceLoaderSet copy = loaders;
-    ResourceLoaderSet::const_iterator end = copy.end();
-    for (ResourceLoaderSet::const_iterator it = copy.begin(); it != end; ++it)
-        (*it)->cancel();
+    Vector<RefPtr<ResourceLoader> > loadersCopy;
+    copyToVector(loaders, loadersCopy);
+    size_t size = loadersCopy.size();
+    for (unsigned i = 0; i < size; ++i)
+        loadersCopy[i]->cancel();
 }
 
 static void setAllDefersLoading(const ResourceLoaderSet& loaders, bool defers)
 {
-    const ResourceLoaderSet copy = loaders;
-    ResourceLoaderSet::const_iterator end = copy.end();
-    for (ResourceLoaderSet::const_iterator it = copy.begin(); it != end; ++it)
-        (*it)->setDefersLoading(defers);
+    Vector<RefPtr<ResourceLoader> > loadersCopy;
+    copyToVector(loaders, loadersCopy);
+    size_t size = loadersCopy.size();
+    for (unsigned i = 0; i < size; ++i)
+        loadersCopy[i]->setDefersLoading(defers);
 }