[android_webview] Add test shell for webview.

android_webview/android_webview_tests.gypi

 # Copyright (c) 2012 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 {
   'targets': [
     {
-      'target_name': 'android_webview_test_apk',
+      'target_name': 'android_webview_test_java',
       'type': 'none',
       'dependencies': [
         '../base/base.gyp:base_java_test_support',
         '../content/content.gyp:content_java_test_support',
         '../net/net.gyp:net_java_test_support',
-       'android_webview_java',
+        'android_webview_java',
         'libwebviewchromium',
       ],
       'variables': {
+        'java_in_dir': '../android_webview/javatests',
+        'has_java_resources': 1,
+        'R_package': 'org.chromium.android_webview.test',
+        'R_package_relpath': 'org/chromium/android_webview/test',
+      },
+      'includes': [ '../build/java.gypi' ],
+    },
+    {
+      'target_name': 'android_webview_test_apk',
+      'type': 'none',
+      'dependencies': [
+        'android_webview_test_java',
+      ],
+      'variables': {
         'apk_name': 'AndroidWebViewTest',
-        'java_in_dir': '../android_webview/javatests',
-        'resource_dir': 'res',
-        'is_test_apk': 1,
+        'java_in_dir': '../android_webview/test_apk',
+        'is_test_apk': 0, # We want resources from android_webview_test_java.

[benm (inactive), 2013/03/19 12:55:01]

We could just remove this, I think it defaults to 0?

[mkosiba (inactive), 2013/03/19 17:11:26]

I wanted some place to stick the comment in. It seemed cleaner to leave it set
explicitly to 0 and explain why.

         'additional_input_paths': [
           '<(PRODUCT_DIR)/android_webview_test_apk/assets/asset_file.html',
           '<(PRODUCT_DIR)/android_webview_test_apk/assets/asset_icon.png',
           '<(PRODUCT_DIR)/android_webview_test_apk/assets/get_video_loading_progress_view_test.html',
         ],
       },
       'copies': [
         {
           'destination': '<(PRODUCT_DIR)/android_webview_test_apk/assets',
           'files': [
             '<(java_in_dir)/assets/asset_file.html',
             '<(java_in_dir)/assets/asset_icon.png',
             '<(java_in_dir)/assets/get_video_loading_progress_view_test.html',
           ],
         },
       ],
       'includes': [ '../build/java_apk.gypi' ],
     },
     {
+      'target_name': 'android_webview_shell_apk',
+      'type': 'none',
+      'dependencies': [
+        'android_webview_test_java',
+        'android_webview_pak',
+      ],
+      'variables': {
+        'apk_name': 'AndroidWebViewShell',
+        'java_in_dir': '../android_webview/shell_apk',
+        'native_libs_paths': ['<(SHARED_LIB_DIR)/libwebviewchromium.so'],

[benm (inactive), 2013/03/19 12:55:01]

Why doesn't android_webview_test_apk need this variable?

[mkosiba (inactive), 2013/03/19 17:11:26]

because the test apk only contains test code and at runtime "merges" with the
android_webview_apk which contains the .so.

+        'resource_dir': 'res',
+        'asset_location': '<(ant_build_out)/android_webview_apk/assets',

[benm (inactive), 2013/03/19 12:55:01]

could we refactor the android_webview_test_apk to use asset_location instead of
additional_input_paths?

[mkosiba (inactive), 2013/03/19 17:11:26]

not sure, nobody uses it like that in the tree. Would it be ok to land that as a
separate patch so that if someone needs to revert it they don't end up reverting
this change as well?

+        'additional_input_paths': [
+          '<(PRODUCT_DIR)/android_webview_apk/assets/webviewchromium.pak',
+        ],
+      },
+      'includes': [ '../build/java_apk.gypi' ],
+    },
+    {
       'target_name': 'android_webview_unittests',
       'type': '<(gtest_target_type)',
       'dependencies': [
         '../base/base.gyp:test_support_base',
         '../net/net.gyp:net_test_support',
         '../testing/android/native_test.gyp:native_test_native_code',
         '../testing/gmock.gyp:gmock',
         '../testing/gtest.gyp:gtest',
         'android_webview_common',
       ],
@@ skipping 43 matching lines @@
         'android_webview_unittests_jni',
       ],
       'variables': {
         'test_suite_name': 'android_webview_unittests',
         'input_shlib_path': '<(SHARED_LIB_DIR)/<(SHARED_LIB_PREFIX)android_webview_unittests<(SHARED_LIB_SUFFIX)',
       },
       'includes': [ '../build/apk_test.gypi' ],
     },
   ],
 }