Changeset 211076 in webkit

Timestamp:

Jan 23, 2017 10:01:27 PM (7 years ago)

Author:

Antti Koivisto

Message:

Maintain ordering when doing speculative loads
​https://bugs.webkit.org/show_bug.cgi?id=167324

Forgot to implement a review comment.

• NetworkProcess/cache/NetworkCacheSubresourcesEntry.cpp:

(WebKit::NetworkCache::makeSubresourceInfoVector):

Allocated initial capacity and use uncheckedAppend.

Location:

trunk/Source/WebKit2

Files:

• ChangeLog (modified) (1 diff)
• NetworkProcess/cache/NetworkCacheSubresourcesEntry.cpp (modified) (2 diffs)

trunk/Source/WebKit2/ChangeLog

@@ +1 @@
+2017-01-23  Antti Koivisto  <antti@apple.com>
+
+        Maintain ordering when doing speculative loads
+        https://bugs.webkit.org/show_bug.cgi?id=167324
+
+        Forgot to implement a review comment.
+
+        * NetworkProcess/cache/NetworkCacheSubresourcesEntry.cpp:
+        (WebKit::NetworkCache::makeSubresourceInfoVector):
+
+        Allocated initial capacity and use uncheckedAppend.
+
 2017-01-23  Commit Queue  <commit-queue@webkit.org>
 

trunk/Source/WebKit2/NetworkProcess/cache/NetworkCacheSubresourcesEntry.cpp

@@ -108 +108 @@
 {
     Vector<SubresourceInfo> result;
-    result.reserveCapacity(subresourceLoads.size());
+    result.reserveInitialCapacity(subresourceLoads.size());
     
     HashSet<Key> seenKeys;
@@ -114 +114 @@
         if (!seenKeys.add(load->key).isNewEntry)
             continue;
-        result.append({ load->key, load->request });
+        result.uncheckedAppend({ load->key, load->request });
     }