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

Unified Diff: Source/core/fetch/ResourceFetcher.cpp

Issue 723423002: Clear ResourceFetcher::m_validatedURLs if too many entries are added (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Limit by ring buffer Created 6 years, 1 month 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
« Source/core/fetch/ResourceFetcher.h ('K') | « Source/core/fetch/ResourceFetcher.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/core/fetch/ResourceFetcher.cpp
diff --git a/Source/core/fetch/ResourceFetcher.cpp b/Source/core/fetch/ResourceFetcher.cpp
index 68f29110590e6be5b393177aaab507a390eda98f..4602757f546ffd5dca1eee6201659d04917ed57e 100644
--- a/Source/core/fetch/ResourceFetcher.cpp
+++ b/Source/core/fetch/ResourceFetcher.cpp
@@ -694,7 +694,11 @@ void ResourceFetcher::requestLoadStarted(Resource* resource, const FetchRequest&
m_resourceTimingReportTimer.startOneShot(0, FROM_HERE);
}
+ if (m_validatedURLsRingBuffer.size() >= 10000) {
+ m_validatedURLs.remove(m_validatedURLsRingBuffer.takeFirst());
+ }
m_validatedURLs.add(request.resourceRequest().url());
+ m_validatedURLsRingBuffer.append(request.resourceRequest().url());
}
ResourcePtr<Resource> ResourceFetcher::requestResource(Resource::Type type, FetchRequest& request)
« Source/core/fetch/ResourceFetcher.h ('K') | « Source/core/fetch/ResourceFetcher.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698