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

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/contextmenu/ContextMenuHelper.java

Issue 1182293002: Save image should download the original image when Data Saver is enabled. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: nits Created 5 years, 6 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/android/java/src/org/chromium/chrome/browser/contextmenu/ContextMenuHelper.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/contextmenu/ContextMenuHelper.java b/chrome/android/java/src/org/chromium/chrome/browser/contextmenu/ContextMenuHelper.java
index d818e820b990ba20d604deaa0ce4e626244480ba..a7021edd79906027c52e0fb77714b44362762b25 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/contextmenu/ContextMenuHelper.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/contextmenu/ContextMenuHelper.java
@@ -20,6 +20,9 @@ import org.chromium.content.browser.ContentViewCore;
* A helper class that handles generating context menus for {@link ContentViewCore}s.
*/
public class ContextMenuHelper implements OnCreateContextMenuListener, OnMenuItemClickListener {
+ private static final String DATA_REDUCTION_PROXY_PASSTHROUGH_HEADER =
+ "Chrome-Proxy: pass-through\r\n";
+
private long mNativeContextMenuHelper;
private ContextMenuPopulator mPopulator;
@@ -75,8 +78,11 @@ public class ContextMenuHelper implements OnCreateContextMenuListener, OnMenuIte
* Starts a download based on the current {@link ContextMenuParams}.
* @param isLink Whether or not the download target is a link.
*/
- public void startContextMenuDownload(boolean isLink) {
- if (mNativeContextMenuHelper != 0) nativeOnStartDownload(mNativeContextMenuHelper, isLink);
+ public void startContextMenuDownload(boolean isLink, boolean isDataReductionProxyEnabled) {
+ if (mNativeContextMenuHelper != 0) {
+ nativeOnStartDownload(mNativeContextMenuHelper, isLink,
+ isDataReductionProxyEnabled ? DATA_REDUCTION_PROXY_PASSTHROUGH_HEADER : null);
+ }
}
@Override
@@ -108,5 +114,6 @@ public class ContextMenuHelper implements OnCreateContextMenuListener, OnMenuIte
return (mPopulator != null && mPopulator.shouldShowContextMenu(params));
}
- private native void nativeOnStartDownload(long nativeContextMenuHelper, boolean isLink);
+ private native void nativeOnStartDownload(
+ long nativeContextMenuHelper, boolean isLink, String headers);
}

Powered by Google App Engine
This is Rietveld 408576698