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

Unified Diff: third_party/WebKit/Source/platform/loader/fetch/Resource.cpp

Issue 2834733003: Separate preaload matching from MemoryCache (Closed)
Patch Set: fix Created 3 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
Index: third_party/WebKit/Source/platform/loader/fetch/Resource.cpp
diff --git a/third_party/WebKit/Source/platform/loader/fetch/Resource.cpp b/third_party/WebKit/Source/platform/loader/fetch/Resource.cpp
index cb136e1aa624d12a0f664451b87f763f5038bf76..59bbe63c40f66f5844ee92af7a840af1ef64871b 100644
--- a/third_party/WebKit/Source/platform/loader/fetch/Resource.cpp
+++ b/third_party/WebKit/Source/platform/loader/fetch/Resource.cpp
@@ -42,6 +42,7 @@
#include "platform/loader/fetch/MemoryCache.h"
#include "platform/loader/fetch/ResourceClient.h"
#include "platform/loader/fetch/ResourceClientWalker.h"
+#include "platform/loader/fetch/ResourceFetcher.h"
#include "platform/loader/fetch/ResourceLoader.h"
#include "platform/network/HTTPParsers.h"
#include "platform/weborigin/KURL.h"
@@ -418,8 +419,11 @@ void Resource::GetError(const ResourceError& error) {
error_ = error;
is_revalidating_ = false;
- if (error_.IsCancellation() || !IsPreloaded())
+ if (error_.IsCancellation() || !IsPreloaded()) {
GetMemoryCache()->Remove(this);
+ if (loader_)
+ loader_->Fetcher()->RemovePreload(this);
+ }
if (!ErrorOccurred())
SetStatus(ResourceStatus::kLoadError);

Powered by Google App Engine
This is Rietveld 408576698