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

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: Context menu browser test to unit test and 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 d41f0c8a15236ed9f80a80a1678309a7528d31c1..1f06f412424441836728d469d6dd5d5b366c6107 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";
Ted C 2015/05/15 21:01:07 should this not be using the constant values from
megjablon 2015/05/15 22:25:28 Whoops sorry missed moving over this location too.
+ 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(
@@ -86,6 +93,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