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

Unified Diff: chrome/android/testshell/java/src/org/chromium/chrome/testshell/TestShellTab.java

Issue 73173002: Add ContextMenu support upstream for Android (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Added tests, addressed comments Created 7 years, 1 month 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/testshell/java/src/org/chromium/chrome/testshell/TestShellTab.java
diff --git a/chrome/android/testshell/java/src/org/chromium/chrome/testshell/TestShellTab.java b/chrome/android/testshell/java/src/org/chromium/chrome/testshell/TestShellTab.java
index 846588fed3ac79139b0cd276693a7939e2485064..7953b7503b079a53684e5562f8597dacb5781c6b 100644
--- a/chrome/android/testshell/java/src/org/chromium/chrome/testshell/TestShellTab.java
+++ b/chrome/android/testshell/java/src/org/chromium/chrome/testshell/TestShellTab.java
@@ -8,6 +8,8 @@ import android.content.Context;
import android.text.TextUtils;
import org.chromium.chrome.browser.TabBase;
+import org.chromium.chrome.browser.contextmenu.ChromeContextMenuPopulator;
+import org.chromium.chrome.browser.contextmenu.ContextMenuPopulator;
import org.chromium.chrome.browser.infobar.AutoLoginProcessor;
import org.chromium.content.browser.ContentView;
import org.chromium.content.browser.LoadUrlParams;
@@ -125,6 +127,15 @@ public class TestShellTab extends TabBase {
};
}
+ @Override
+ protected ContextMenuPopulator createContextMenuPopulator() {
+ return new ChromeContextMenuPopulator(new TabBaseChromeContextMenuItemDelegate() {
+ @Override
+ public void onOpenImageUrl(String url) {
+ loadUrlWithSanitization(url);
+ }
+ });
+ }
private class TestShellTabBaseChromeWebContentsDelegateAndroid
extends TabBaseChromeWebContentsDelegateAndroid {

Powered by Google App Engine
This is Rietveld 408576698