Fix 'Load image' context menu item

chrome/renderer/chrome_content_renderer_client_unittest.cc

Index: chrome/renderer/chrome_content_renderer_client_unittest.cc
diff --git a/chrome/renderer/chrome_content_renderer_client_unittest.cc b/chrome/renderer/chrome_content_renderer_client_unittest.cc
index 4a4e831fdebc0825cd3ab8e6c176d319405f69e3..6639e1f63bd4878b18279ca4441d558704058aa3 100644
--- a/chrome/renderer/chrome_content_renderer_client_unittest.cc
+++ b/chrome/renderer/chrome_content_renderer_client_unittest.cc
@@ -13,8 +13,11 @@
 #include "base/test/histogram_tester.h"
 #include "build/build_config.h"
 #include "chrome/renderer/searchbox/search_bouncer.h"
+#include "components/data_reduction_proxy/core/common/data_reduction_proxy_headers.h"
 #include "content/public/common/webplugininfo.h"
 #include "testing/gtest/include/gtest/gtest.h"
+#include "third_party/WebKit/public/platform/WebString.h"
+#include "third_party/WebKit/public/platform/WebURLResponse.h"
 #include "url/gurl.h"
 
 #if defined(ENABLE_EXTENSIONS)
@@ -411,6 +414,21 @@ TEST_F(ChromeContentRendererClientTest, ShouldSuppressErrorPage) {
       std::vector<GURL>(), GURL::EmptyGURL());
 }
 
+TEST_F(ChromeContentRendererClientTest, AddImageContextMenuProperties) {
+  ChromeContentRendererClient client;
+  blink::WebURLResponse webUrlResponse;

[Lei Zhang, 2016/11/01 22:59:42]

nit: web_url_response

[megjablon, 2016/11/01 23:37:00]

Whoops, got my head in android world.

+  webUrlResponse.addHTTPHeaderField(
+      blink::WebString::fromUTF8(
+          data_reduction_proxy::chrome_proxy_content_transform_header()),
+      blink::WebString::fromUTF8(
+          data_reduction_proxy::empty_image_directive()));
+  std::map<std::string, std::string> properties;
+  client.AddImageContextMenuProperties(webUrlResponse, &properties);
+  EXPECT_EQ(
+      properties[data_reduction_proxy::chrome_proxy_content_transform_header()],
+      data_reduction_proxy::empty_image_directive());

[Lei Zhang, 2016/11/01 22:59:42]

Is this the expected value? If so, flip the arguments because EXPECT_EQ() takes
EXPECT_EQ(expected, actual).

[megjablon, 2016/11/01 23:37:00]

Done.

+}
+
 // These are tests that are common for both Android and desktop browsers.
 TEST_F(ChromeContentRendererClientTest, RewriteEmbedCommon) {
   struct TestData {