Refactor android_webview_shell and its tests

android_webview/android_webview_shell.gyp

Index: android_webview/android_webview_shell.gyp
diff --git a/android_webview/android_webview_shell.gyp b/android_webview/android_webview_shell.gyp
index a5d898f94f9961e6143ee2be917c2132aff3c538..49473ddcbd4a92faa998e7115927169837873c7a 100644
--- a/android_webview/android_webview_shell.gyp
+++ b/android_webview/android_webview_shell.gyp
@@ -6,22 +6,70 @@
     {
       'target_name': 'android_webview_shell_apk',
       'type': 'none',
-      'dependencies': [
-        '../base/base.gyp:base_java_test_support',
-      ],
       'variables': {
         'apk_name': 'AndroidWebViewShell',
         'java_in_dir': 'tools/WebViewShell',
         'resource_dir': 'tools/WebViewShell/res',
+        'android_manifest_path': 'tools/WebViewShell/AndroidManifest.xml', # for lint
+      },
+      'includes': [
+        '../build/java_apk.gypi',
+      ],
+    },
+    {
+      # android_webview_apk creates a .jar as a side effect. Any java
+      # targets that need that .jar in their classpath should depend on this
+      # target. For more details see the content_shell_apk_java target.
+      'target_name': 'android_webview_shell_apk_java',
+      'type': 'none',
+      'dependencies': [
+        'android_webview_shell_apk',
+      ],
+      'includes': [ '../build/apk_fake_jar.gypi' ],
+    },
+    {
+      'target_name': 'android_webview_shell_page_cycler_apk',
+      'type': 'none',
+      'dependencies': [
+        '../base/base.gyp:base_java_test_support',
+        '../content/content_shell_and_tests.gyp:content_java_test_support',
+        '../testing/android/on_device_instrumentation.gyp:broker_java',
+        '../testing/android/on_device_instrumentation.gyp:require_driver_apk',
+        'android_webview_shell_apk_java',
+      ],
+      'variables': {
+        'apk_name': 'AndroidWebViewShellPageCycler',
+        'java_in_dir': '../android_webview/tools/PageCycler',
         'is_test_apk': 1,
         'test_type': 'instrumentation',
-        'isolate_file': 'android_webview_shell_test_apk.isolate',
-        'android_manifest_path': 'tools/WebViewShell/AndroidManifest.xml', # for lint
+        'android_manifest_path': 'tools/PageCycler/AndroidManifest.xml',
       },
       'includes': [
         '../build/java_apk.gypi',
         '../build/android/test_runner.gypi',
       ],
     },
+    {
+      'target_name': 'android_webview_shell_test_apk',
+      'type': 'none',
+      'dependencies': [
+        '../base/base.gyp:base_java_test_support',
+        '../testing/android/on_device_instrumentation.gyp:broker_java',
+        '../testing/android/on_device_instrumentation.gyp:require_driver_apk',
+        'android_webview_shell_apk_java',
+      ],
+      'variables': {
+        'apk_name': 'AndroidWebViewShellTest',
+        'java_in_dir': '../android_webview/tools/WebViewShellTest',
+        'is_test_apk': 1,
+        'test_type': 'instrumentation',
+        'isolate_file': 'android_webview_shell_test_apk.isolate',
+        'android_manifest_path': 'tools/WebViewShellTest/AndroidManifest.xml',
+      },
+      'includes': [
+        '../build/java_apk.gypi',
+        '../build/android/test_runner.gypi',
+      ]
+    },
   ],
 }