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

Unified Diff: chrome/browser/ui/android/context_menu_helper.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: Addressing comments 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: chrome/browser/ui/android/context_menu_helper.cc
diff --git a/chrome/browser/ui/android/context_menu_helper.cc b/chrome/browser/ui/android/context_menu_helper.cc
index bfbe8d36e3f267a754cd82bcf8089a70f9219bba..6838f4d18db9d162ff8877c327a8aa0077775713 100644
--- a/chrome/browser/ui/android/context_menu_helper.cc
+++ b/chrome/browser/ui/android/context_menu_helper.cc
@@ -75,6 +75,13 @@ ContextMenuHelper::CreateJavaContextMenuParams(
GURL sanitizedReferrer = (params.frame_url.is_empty() ?
params.page_url : params.frame_url).GetAsReferrer();
+ static const char lo_fi_header_key[] = "Chrome-Proxy";
+ static const char lo_fi_header_value[] = "q=low";
+ std::map<std::string, std::string>::const_iterator it =
+ params.properties.find(lo_fi_header_key);
+ bool image_was_fetched_lo_fi =
+ it != params.properties.end() && it->second == lo_fi_header_value;
+
JNIEnv* env = base::android::AttachCurrentThread();
base::android::ScopedJavaLocalRef<jobject> jmenu_info =
ContextMenuParamsAndroid::Java_ContextMenuParams_create(
@@ -85,6 +92,7 @@ ContextMenuHelper::CreateJavaContextMenuParams(
ConvertUTF8ToJavaString(env, params.unfiltered_link_url.spec()).obj(),
ConvertUTF8ToJavaString(env, params.src_url.spec()).obj(),
ConvertUTF16ToJavaString(env, params.selection_text).obj(),
+ image_was_fetched_lo_fi,
params.is_editable,
ConvertUTF8ToJavaString(env, sanitizedReferrer.spec()).obj(),
params.referrer_policy);

Powered by Google App Engine
This is Rietveld 408576698