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

Unified Diff: trunk/src/chrome/android/testshell/java/src/org/chromium/chrome/shell/ChromiumTestShellActivity.java

Issue 187453003: Revert 254929 "Add UI hook for distilling web pages for test shell." (Closed) Base URL: svn://svn.chromium.org/chrome/
Patch Set: Created 6 years, 10 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: trunk/src/chrome/android/testshell/java/src/org/chromium/chrome/shell/ChromiumTestShellActivity.java
===================================================================
--- trunk/src/chrome/android/testshell/java/src/org/chromium/chrome/shell/ChromiumTestShellActivity.java (revision 254957)
+++ trunk/src/chrome/android/testshell/java/src/org/chromium/chrome/shell/ChromiumTestShellActivity.java (working copy)
@@ -28,7 +28,6 @@
import org.chromium.chrome.browser.printing.PrintingControllerFactory;
import org.chromium.chrome.browser.printing.TabPrinter;
import org.chromium.chrome.shell.sync.SyncController;
-import org.chromium.components.dom_distiller.core.DomDistillerUrlUtils;
import org.chromium.content.browser.ActivityContentVideoViewClient;
import org.chromium.content.browser.BrowserStartupController;
import org.chromium.content.browser.ContentView;
@@ -43,7 +42,6 @@
*/
public class ChromiumTestShellActivity extends Activity implements AppMenuPropertiesDelegate {
private static final String TAG = "ChromiumTestShellActivity";
- private static final String CHROME_DISTILLER_SCHEME = "chrome-distiller";
private WindowAndroid mWindow;
private TabManager mTabManager;
@@ -235,12 +233,6 @@
mPrintingController.startPrint(new TabPrinter(getActiveTab()));
}
return true;
- case R.id.distill_page:
- TestShellTab activeTab = getActiveTab();
- String viewUrl = DomDistillerUrlUtils.getDistillerViewUrlFromUrl(
- CHROME_DISTILLER_SCHEME, getActiveTab().getUrl());
- activeTab.loadUrlWithSanitization(viewUrl);
- return true;
case R.id.back_menu_id:
if (getActiveTab().canGoBack()) getActiveTab().goBack();
return true;
@@ -291,9 +283,6 @@
menu.findItem(R.id.print).setVisible(ApiCompatibilityUtils.isPrintingSupported());
- menu.findItem(R.id.distill_page).setVisible(
- CommandLine.getInstance().hasSwitch(TestShellSwitches.ENABLE_DOM_DISTILLER));
-
menu.setGroupVisible(R.id.MAIN_MENU, true);
}

Powered by Google App Engine
This is Rietveld 408576698