"Load image" context menu item to reload a LoFi image.

content/child/web_url_loader_impl.cc

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/.  Looks like all the places
that set and check it are outside content/.  @jam, is there a way for embedders
to add things to a response's extra data?

[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 "Chrome-Proxy" is a big
warning sign since it's something in components and this string is duplicated
here because content can't depend on that.

can we call the embedder to put a key-value property bag on the extra data? and
then the context menu builder code can send along this property bag to chrome
over ipc.

+  }
 
   // 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