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

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/tab/TabContextMenuItemDelegate.java

Issue 1741153002: Switch back to open image in new tab where applicable. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix tests Created 4 years, 9 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/tab/TabContextMenuItemDelegate.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/tab/TabContextMenuItemDelegate.java b/chrome/android/java/src/org/chromium/chrome/browser/tab/TabContextMenuItemDelegate.java
index 6873f397e019370de2f7d3f00ac255efc20642c3..06fc49d24391cb8ea4f9f7c835c66eddc808ce54 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/tab/TabContextMenuItemDelegate.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/tab/TabContextMenuItemDelegate.java
@@ -27,6 +27,9 @@ import java.util.Locale;
* A default {@link ContextMenuItemDelegate} that supports the context menu functionality in Tab.
*/
public class TabContextMenuItemDelegate implements ContextMenuItemDelegate {
+ public static final String PAGESPEED_PASSTHROUGH_HEADERS =
+ "Chrome-Proxy: pass-through\nCache-Control: no-cache";
+
private final Clipboard mClipboard;
private final Tab mTab;
@@ -108,6 +111,16 @@ public class TabContextMenuItemDelegate implements ContextMenuItemDelegate {
}
@Override
+ public void onOpenImageInNewTab(String url, Referrer referrer) {
+ boolean useOriginal = isSpdyProxyEnabledForUrl(url);
+ LoadUrlParams loadUrlParams = new LoadUrlParams(url);
+ loadUrlParams.setVerbatimHeaders(useOriginal ? PAGESPEED_PASSTHROUGH_HEADERS : null);
+ loadUrlParams.setReferrer(referrer);
+ mTab.getActivity().getTabModelSelector().openNewTab(loadUrlParams,
+ TabLaunchType.FROM_LONGPRESS_BACKGROUND, mTab, isIncognito());
+ }
+
+ @Override
public void onOpenInChrome(String linkUrl, String pageUrl) {
Intent chromeIntent = new Intent(Intent.ACTION_VIEW, Uri.parse(linkUrl));
chromeIntent.setPackage(mTab.getApplicationContext().getPackageName());

Powered by Google App Engine
This is Rietveld 408576698