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

content/child/web_url_loader_impl.cc

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/child/web_url_loader_impl.h"
 
 #include <algorithm>
 #include <deque>
 #include <string>
 
@@ skipping 964 matching lines @@
   WebURLResponseExtraDataImpl* extra_data =
       new WebURLResponseExtraDataImpl(info.npn_negotiated_protocol);
   response->setExtraData(extra_data);
   extra_data->set_was_fetched_via_spdy(info.was_fetched_via_spdy);
   extra_data->set_was_npn_negotiated(info.was_npn_negotiated);
   extra_data->set_was_alternate_protocol_available(
       info.was_alternate_protocol_available);
   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
   // certain error cases.
   if (!info.load_timing.receive_headers_end.is_null()) {
     WebURLLoadTiming timing;
     PopulateURLLoadTiming(info.load_timing, &timing);
     const TimeTicks kNullTicks;
     timing.setServiceWorkerFetchStart(
         (info.service_worker_fetch_start - kNullTicks).InSecondsF());
@@ skipping 122 matching lines @@
                                          int intra_priority_value) {
   context_->DidChangePriority(new_priority, intra_priority_value);
 }
 
 bool WebURLLoaderImpl::attachThreadedDataReceiver(
     blink::WebThreadedDataReceiver* threaded_data_receiver) {
   return context_->AttachThreadedDataReceiver(threaded_data_receiver);
 }
 
 }  // namespace content