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

Unified Diff: third_party/WebKit/Source/core/loader/resource/ImageResource.cpp

Issue 2800473002: Treat responses with legacy LoFi headers as LoFi images. (Closed)
Patch Set: Rebased on master 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/core/loader/resource/ImageResource.cpp
diff --git a/third_party/WebKit/Source/core/loader/resource/ImageResource.cpp b/third_party/WebKit/Source/core/loader/resource/ImageResource.cpp
index 2c2a733d6f44d40dac7a2b73a9bcf0a0c4d0e089..584eaf2d7b97709885e9de6338d1cfd33cb64096 100644
--- a/third_party/WebKit/Source/core/loader/resource/ImageResource.cpp
+++ b/third_party/WebKit/Source/core/loader/resource/ImageResource.cpp
@@ -461,8 +461,11 @@ bool ImageResource::ShouldReloadBrokenPlaceholder() const {
static bool IsLoFiImage(const ImageResource& resource) {
if (resource.IsLoaded()) {
return resource.GetResponse()
- .HttpHeaderField("chrome-proxy-content-transform")
- .Contains("empty-image");
+ .HttpHeaderField("chrome-proxy-content-transform")
+ .Contains("empty-image") ||
+ resource.GetResponse()
+ .HttpHeaderField("chrome-proxy")
+ .Contains("q=low");
}
return resource.GetResourceRequest().GetPreviewsState() &
WebURLRequest::kServerLoFiOn;

Powered by Google App Engine
This is Rietveld 408576698