Add UI hook for distilling web pages for test shell (relanding)

Add UI hook for distilling web pages for test shell (relanding)

This adds the Java hooks needed and a menu option to distill the current
web page in the test shell.

Original CL: https://codereview.chromium.org/185113002/

BUG=319881
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=255369

[nyquist, 2014-03-05 17:21:31]

cjhopman: PTAL

[nyquist, 2014-03-05 18:12:15]

thakis: PTAL chrome/chrome.gyp for OWNERS review

The dom_distiller_core_java dependency is moved from chromium_testshell to
chrome_java. I've discussed offline with cjhopman, and we thought it was a good
place for it.

[cjhopman, 2014-03-05 18:30:55]

lgtm.

[Nico, 2014-03-05 23:50:36]

gyp lgtm

[nyquist, 2014-03-06 00:46:09]

The CQ bit was checked by nyquist@chromium.org

[commit-bot: I haz the power, 2014-03-06 00:49:43]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/nyquist@chromium.org/183893029/20001

[commit-bot: I haz the power, 2014-03-06 00:49:47]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-03-06 00:49:48]

Failed to apply patch for
chrome/android/testshell/java/src/org/chromium/chrome/shell/ChromiumTestShellActivity.java:
While running patch -p1 --forward --force --no-backup-if-mismatch;
  can't find file to patch at input line 6
  Perhaps you used the wrong -p or --strip option?
  The text leading up to this was:
  --------------------------
  |Index:
chrome/android/testshell/java/src/org/chromium/chrome/shell/ChromiumTestShellActivity.java
  |diff --git
a/chrome/android/testshell/java/src/org/chromium/chrome/shell/ChromiumTestShellActivity.java
b/chrome/android/testshell/java/src/org/chromium/chrome/shell/ChromiumTestShellActivity.java
  |index
f82d707bcf6e99a0ed89f7fa81409aea337d7c2e..349afc1cccd342cc5db723dd3f85339c7476fb39
100644
  |---
a/chrome/android/testshell/java/src/org/chromium/chrome/shell/ChromiumTestShellActivity.java
  |+++
b/chrome/android/testshell/java/src/org/chromium/chrome/shell/ChromiumTestShellActivity.java
  --------------------------
  No file to patch.  Skipping patch.
  4 out of 4 hunks ignored

Patch:      
chrome/android/testshell/java/src/org/chromium/chrome/shell/ChromiumTestShellActivity.java
Index:
chrome/android/testshell/java/src/org/chromium/chrome/shell/ChromiumTestShellActivity.java
diff --git
a/chrome/android/testshell/java/src/org/chromium/chrome/shell/ChromiumTestShellActivity.java
b/chrome/android/testshell/java/src/org/chromium/chrome/shell/ChromiumTestShellActivity.java
index
f82d707bcf6e99a0ed89f7fa81409aea337d7c2e..349afc1cccd342cc5db723dd3f85339c7476fb39
100644
---
a/chrome/android/testshell/java/src/org/chromium/chrome/shell/ChromiumTestShellActivity.java
+++
b/chrome/android/testshell/java/src/org/chromium/chrome/shell/ChromiumTestShellActivity.java
@@ -28,6 +28,7 @@ import
org.chromium.chrome.browser.appmenu.AppMenuPropertiesDelegate;
 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;
@@ -42,6 +43,7 @@ import org.chromium.ui.base.WindowAndroid;
  */
 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;
@@ -233,6 +235,12 @@ public class ChromiumTestShellActivity extends Activity
implements AppMenuProper
                     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;
@@ -283,6 +291,9 @@ public class ChromiumTestShellActivity extends Activity
implements AppMenuProper
 
        
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);
     }

[nyquist, 2014-03-06 06:08:55]

The CQ bit was checked by nyquist@chromium.org

[commit-bot: I haz the power, 2014-03-06 06:10:38]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/nyquist@chromium.org/183893029/60001

[commit-bot: I haz the power, 2014-03-06 06:27:08]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-03-06 06:27:09]

Retried try job too often on win_rel for step(s) base_unittests, browser_tests,
interactive_ui_tests, net_unittests, unit_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_rel&nu...

[nyquist, 2014-03-06 14:19:37]

The CQ bit was checked by nyquist@chromium.org

[commit-bot: I haz the power, 2014-03-06 14:23:53]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/nyquist@chromium.org/183893029/60001

[commit-bot: I haz the power, 2014-03-06 16:48:44]

Change committed as 255369