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

Unified Diff: content/child/web_url_loader_impl.cc

Issue 1105263004: "Load image" context menu item to reload a LoFi image. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 7 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: content/child/web_url_loader_impl.cc
diff --git a/content/child/web_url_loader_impl.cc b/content/child/web_url_loader_impl.cc
index a1a5beb4b281d6f4a625d33b26f497db68b30b81..77d8124b24653393e5381391eb2c2677327a84e2 100644
--- a/content/child/web_url_loader_impl.cc
+++ b/content/child/web_url_loader_impl.cc
@@ -982,6 +982,10 @@ void WebURLLoaderImpl::PopulateURLResponse(const GURL& url,
extra_data->set_connection_info(info.connection_info);
extra_data->set_was_fetched_via_proxy(info.was_fetched_via_proxy);
extra_data->set_proxy_server(info.proxy_server);
+ if (info.headers) {
+ extra_data->set_was_fetched_lo_fi(
+ info.headers->HasHeaderValue("Chrome-Proxy", "q=low"));
Charlie Reis 2015/05/11 22:32:58 It's not clear to me that this belongs in content/
megjablon 2015/05/11 22:58:40 Acknowledged.
jam 2015/05/11 22:59:47 i agree that this shouldn't go in content; seeing
+ }
// If there's no received headers end time, don't set load timing. This is
// the case for non-HTTP requests, requests that don't go over the wire, and

Powered by Google App Engine
This is Rietveld 408576698