Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1841)

Unified Diff: Source/web/AssociatedURLLoader.cpp

Issue 256743005: Add removeAll method to sets and maps and use them (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Moved implementation of removeAll to HashTable.h and shared it. Also added it to LinkedHashSet Created 6 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « Source/platform/fonts/FontCache.cpp ('k') | Source/wtf/HashMap.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/web/AssociatedURLLoader.cpp
diff --git a/Source/web/AssociatedURLLoader.cpp b/Source/web/AssociatedURLLoader.cpp
index c7f0a29756ec7cab9cbeb587c0eee24ec09f6b25..7b4079e1e90cfc5252ad83230734c235a62025cd 100644
--- a/Source/web/AssociatedURLLoader.cpp
+++ b/Source/web/AssociatedURLLoader.cpp
@@ -111,9 +111,7 @@ const HTTPHeaderSet& HTTPResponseHeaderValidator::blockedHeaders()
m_exposedHeaders.remove("set-cookie2");
// Block Access-Control-Expose-Header itself. It could be exposed later.
m_blockedHeaders.add("access-control-expose-headers");
- HTTPHeaderSet::const_iterator end = m_exposedHeaders.end();
- for (HTTPHeaderSet::const_iterator it = m_exposedHeaders.begin(); it != end; ++it)
- m_blockedHeaders.remove(*it);
+ m_blockedHeaders.removeAll(m_exposedHeaders);
}
return m_blockedHeaders;
« no previous file with comments | « Source/platform/fonts/FontCache.cpp ('k') | Source/wtf/HashMap.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698