Index: third_party/WebKit/Source/core/frame/FrameSerializer.cpp |
diff --git a/third_party/WebKit/Source/core/frame/FrameSerializer.cpp b/third_party/WebKit/Source/core/frame/FrameSerializer.cpp |
index 6427f7e04fab9595002ef102e06221ba685c23a4..04e105240dedb6c14f08eba077c91c9cbbd5f482 100644 |
--- a/third_party/WebKit/Source/core/frame/FrameSerializer.cpp |
+++ b/third_party/WebKit/Source/core/frame/FrameSerializer.cpp |
@@ -386,11 +386,14 @@ void FrameSerializer::serializeCSSRule(CSSRule* rule) |
bool FrameSerializer::shouldAddURL(const KURL& url) |
{ |
return url.isValid() && !m_resourceURLs.contains(url) && !url.protocolIsData() |
- && !m_delegate.shouldSkipResource(url); |
+ && !m_delegate.shouldSkipResourceWithURL(url); |
} |
void FrameSerializer::addToResources(Resource* resource, PassRefPtr<SharedBuffer> data, const KURL& url) |
{ |
+ if (m_delegate.shouldSkipResource(resource)) |
Łukasz Anforowicz
2016/05/18 17:27:25
I wonder if we can skip processing a resource befo
dewittj
2016/05/19 04:13:17
I am not sure how we could get the URL (or indeed
Łukasz Anforowicz
2016/05/19 17:00:13
You're right - fetching is unavoidable, but we can
dewittj
2016/05/19 18:18:35
Does this actually cause a copy? It looks like it
Łukasz Anforowicz
2016/05/19 18:50:17
My bad - you're right, this is just a ref-count in
|
+ return; |
+ |
if (!data) { |
DLOG(ERROR) << "No data for resource " << url.getString(); |
return; |