Treat responses with legacy LoFi headers as LoFi images.

chrome/renderer/chrome_content_renderer_client.cc

Index: chrome/renderer/chrome_content_renderer_client.cc
diff --git a/chrome/renderer/chrome_content_renderer_client.cc b/chrome/renderer/chrome_content_renderer_client.cc
index 77fd3cadaba22631c2b2c6b4747a090a3ef09c0f..0b0de3397edea18333fa5a8aaed3d76952122c9f 100644
--- a/chrome/renderer/chrome_content_renderer_client.cc
+++ b/chrome/renderer/chrome_content_renderer_client.cc
@@ -1384,13 +1384,16 @@ void ChromeContentRendererClient::AddImageContextMenuProperties(
     const WebURLResponse& response,
     std::map<std::string, std::string>* properties) {
   DCHECK(properties);
-  WebString header_key(WebString::fromASCII(
+
+  WebString cpct_value = response.httpHeaderField(WebString::fromASCII(

[Lei Zhang, 2017/04/08 01:01:07]

Why do we have to go from const char* to WebString to std::string?

[sclittle, 2017/04/10 20:56:50]

We basically want to call:

drp::IsEmptyImagePreview(
    response.HttpHeaderField(drp::chrome_proxy_header()),
    response.HttpHeaderField(
        drp::chrome_proxy_content_transform_header()));

But since response.HttpHeaderField() both takes and returns WebStrings, we need
to convert the header names to WebStrings and the returned header values to
std::strings. Do you know of a more direct way to do this?

       data_reduction_proxy::chrome_proxy_content_transform_header()));
-  if (!response.httpHeaderField(header_key).isNull() &&
-      data_reduction_proxy::IsEmptyImagePreview(
-          response.httpHeaderField(header_key).utf8())) {
-    (*properties)[
-        data_reduction_proxy::chrome_proxy_content_transform_header()] =
+  WebString chrome_proxy_value = response.httpHeaderField(
+      WebString::fromASCII(data_reduction_proxy::chrome_proxy_header()));
+
+  if (data_reduction_proxy::IsEmptyImagePreview(cpct_value.utf8(),
+                                                chrome_proxy_value.utf8())) {
+    (*properties)
+        [data_reduction_proxy::chrome_proxy_content_transform_header()] =
             data_reduction_proxy::empty_image_directive();
   }
 }