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

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

Issue 2319483002: Turn off isLinkPreload() when used (Closed)
Patch Set: Turn off LinkPreload when resource is used + tests Created 4 years, 3 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
Index: third_party/WebKit/Source/core/fetch/ResourceFetcher.cpp
diff --git a/third_party/WebKit/Source/core/fetch/ResourceFetcher.cpp b/third_party/WebKit/Source/core/fetch/ResourceFetcher.cpp
index 5ff0d0d3861a31c60532f733a76016d704734229..80f16370fa13e2ff049f40e7f23c569fe4d0c583 100644
--- a/third_party/WebKit/Source/core/fetch/ResourceFetcher.cpp
+++ b/third_party/WebKit/Source/core/fetch/ResourceFetcher.cpp
@@ -474,6 +474,7 @@ Resource* ResourceFetcher::requestResource(FetchRequest& request, const Resource
initializeRevalidation(request.mutableResourceRequest(), resource);
break;
case Use:
+ resource->setLinkPreload(request.isLinkPreload());
Nate Chapin 2016/09/08 17:26:47 Doesn't this have the potential to mark something
memoryCache()->updateForAccess(resource);
break;
}
@@ -868,7 +869,7 @@ void ResourceFetcher::preloadStarted(Resource* resource)
if (m_preloads && m_preloads->contains(resource))
return;
TRACE_EVENT_ASYNC_STEP_INTO0("blink.net", "Resource", resource->identifier(), "Preload");
- resource->increasePreloadCount();
+ resource->setIsPreloaded(true);
if (!m_preloads)
m_preloads = new HeapListHashSet<Member<Resource>>;
@@ -896,7 +897,7 @@ void ResourceFetcher::clearPreloads(ClearPreloadsPolicy policy)
for (auto resource : *m_preloads) {
if (policy == ClearAllPreloads || !resource->isLinkPreload()) {
- resource->decreasePreloadCount();
+ resource->setIsPreloaded(false);
if (resource->getPreloadResult() == Resource::PreloadNotReferenced)
memoryCache()->remove(resource.get());
m_preloads->remove(resource);
« no previous file with comments | « third_party/WebKit/Source/core/fetch/Resource.cpp ('k') | third_party/WebKit/Source/core/fetch/ResourceFetcherTest.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698