Support javascript gtests in an extension background page.

chrome/browser/resources/chromeos/chromevox/chromevox_tests.gypi

Index: chrome/browser/resources/chromeos/chromevox/chromevox_tests.gypi
diff --git a/chrome/browser/resources/chromeos/chromevox/chromevox_tests.gypi b/chrome/browser/resources/chromeos/chromevox/chromevox_tests.gypi
index 9c462d2046eecbcad087b17510c5df6620fd29f5..fe6d095cbb3723ea212c3baacee7ca9e2eec1642 100644
--- a/chrome/browser/resources/chromeos/chromevox/chromevox_tests.gypi
+++ b/chrome/browser/resources/chromeos/chromevox/chromevox_tests.gypi
@@ -71,22 +71,65 @@
             '<@(_outputs)',
           ],
         },
+        {
+          # A JavaScript test that runs in the ChromeVox background page's
+          #  isolate.

[Peter Lundblad, 2014/06/08 23:27:21]

nit: extra space.

[David Tseng, 2014/06/10 05:35:22]

Done.

+          'rule_name': 'js2chromevox',
+          'extension': 'chromevoxjstest',

[Peter Lundblad, 2014/06/08 23:27:21]

cvxojstest?  Shorter and this rules is local to our target, so there shouldn't
be any risk for conflict.

[dmazzoni, 2014/06/09 06:59:46]

Yeah, considering the rule is local to the target I don't even think we need
cvox/chromevox in the name - how about extjs for running in the extension vs
unitjs, etc.?

[David Tseng, 2014/06/10 05:35:22]

Done.

+          'msvs_external_rule': 1,
+          'inputs': [
+            '<(gypv8sh)',
+            '<(PRODUCT_DIR)/d8<(EXECUTABLE_SUFFIX)',
+            '<(mock_js)',
+            '<(test_api_js)',
+            '<(js2gtest)',
+            '<(chromevox_test_deps_js_file)',

[Peter Lundblad, 2014/06/08 23:27:21]

Do we need the deps file for these tests?  Shouldn't they call code already
loaded as part of the extension?

[David Tseng, 2014/06/10 05:35:22]

Removed.

+            'testing/chromevox_unittest_base.js',
+          ],
+          'outputs': [
+            '<(INTERMEDIATE_DIR)/<(RULE_INPUT_DIRNAME)/<(RULE_INPUT_ROOT)-gen.cc',
+            '<(PRODUCT_DIR)/test_data/chrome/browser/resources/chromeos/chromevox/<(RULE_INPUT_DIRNAME)/<(RULE_INPUT_ROOT).chromevoxjstest',
+          ],
+          'process_outputs_as_sources': 1,
+          'action': [
+            'python',
+            '<(gypv8sh)',
+            '<(PRODUCT_DIR)/d8<(EXECUTABLE_SUFFIX)',
+            '--deps_js', '<(chromevox_test_deps_js_file)',
+            '<(mock_js)',
+            '<(test_api_js)',
+            '<(js2gtest)',
+            'chromevox',
+            '<(RULE_INPUT_PATH)',
+            'chrome/browser/resources/chromeos/chromevox/<(RULE_INPUT_DIRNAME)/<(RULE_INPUT_ROOT).chromevoxjstest',
+            '<@(_outputs)',
+          ],
+        },
       ],
       'sources': [
         '<(DEPTH)/chrome/browser/ui/webui/web_ui_test_handler.cc',
         '<(DEPTH)/chrome/browser/ui/webui/web_ui_test_handler.h',
         '<(DEPTH)/chrome/test/base/browser_tests_main.cc',
+        '<(DEPTH)/chrome/test/base/javascript_browsertest.cc',
+        '<(DEPTH)/chrome/test/base/javascrip_browsertest.h',

[dmazzoni, 2014/06/09 06:59:45]

javascrip -> javascript

[David Tseng, 2014/06/10 05:35:22]

Done.

         '<(DEPTH)/chrome/test/base/test_chrome_web_ui_controller_factory.cc',
         '<(DEPTH)/chrome/test/base/test_chrome_web_ui_controller_factory.h',
         '<(DEPTH)/chrome/test/base/web_ui_browser_test.cc',
         '<(DEPTH)/chrome/test/base/web_ui_browser_test.h',
 
+        '<(DEPTH)/chrome/browser/extensions/browsertest_util.cc',
+        '<(DEPTH)/chrome/browser/extensions/browsertest_util.h',
+
+        'chromevox_browsertest.cc',
+        'chromevox_browsertest.h',
         'common/aria_util_test.js',
         'common/cursor_selection_test.js',
         'common/editable_text_area_shadow_test.js',
         'common/key_sequence_test.js',
         'common/math_semantic_tree_test.js',
         'common/selection_util_test.js',
+
+        '<(DEPTH)/chrome/browser/resources/chromeos/chromevox2/cvox2/background/background.chromevoxjstest',
       ],
     },  # target chromevox_tests
     {