Retire gtestjs unit tests

remoting/remoting_test.gypi

Index: remoting/remoting_test.gypi
diff --git a/remoting/remoting_test.gypi b/remoting/remoting_test.gypi
index 903d88ad2966beafef6df159573f205c487f706f..087177d58239ab987cbc6e53aa19cb82a08bf740 100644
--- a/remoting/remoting_test.gypi
+++ b/remoting/remoting_test.gypi
@@ -40,7 +40,7 @@
         '../testing/gmock/include',
       ],
       'sources': [
-        '../chrome/test/base/run_all_remoting_unittests.cc',
+        'base/run_all_unittests.cc',

[garykac, 2014/07/15 23:23:42]

alpha order

[kelvinp, 2014/07/16 00:17:55]

Done.

         'base/auth_token_util_unittest.cc',
         'base/auto_thread_task_runner_unittest.cc',
         'base/auto_thread_unittest.cc',
@@ -55,7 +55,7 @@
         'base/typed_buffer_unittest.cc',
         'base/util_unittest.cc',
         'client/audio_player_unittest.cc',
-	'client/client_status_logger_unittest.cc',
+        'client/client_status_logger_unittest.cc',
         'client/key_event_mapper_unittest.cc',
         'client/plugin/normalizing_input_filter_cros_unittest.cc',
         'client/plugin/normalizing_input_filter_mac_unittest.cc',
@@ -125,7 +125,7 @@
         'jingle_glue/fake_signal_strategy.cc',
         'jingle_glue/fake_signal_strategy.h',
         'jingle_glue/iq_sender_unittest.cc',
-	'jingle_glue/log_to_server_unittest.cc',
+        'jingle_glue/log_to_server_unittest.cc',
         'jingle_glue/mock_objects.cc',
         'jingle_glue/mock_objects.h',
         'jingle_glue/network_settings_unittest.cc',
@@ -177,40 +177,6 @@
             ],
           },
         }],
-        [ 'OS=="mac" or (OS=="linux" and chromeos==0)', {
-          # Javascript unittests are disabled on CrOS because they cause
-          # valgrind and test errors.
-          #
-          # Javascript unittests are disabled on Windows because they add a
-          # dependency on 'common_constants' which (only on Windows) requires
-          # additional dependencies:
-          #   '../content/content.gyp:content_common',
-          #   'installer_util',
-          # These targets are defined in .gypi files that would need to be
-          # included here:
-          #   '../chrome/chrome_common.gypi',
-          #   '../chrome/chrome_installer.gypi',
-          #   '../chrome/chrome_installer_util.gypi',
-          # But we can't do that because ninja will complain about multiple
-          # target definitions.
-          # TODO(garykac): Move installer_util into a proper .gyp file so that
-          # it can be included in multiple .gyp files.
-          'includes': [
-            '../chrome/js_unittest_rules.gypi',
-          ],
-          'dependencies': [
-            '../chrome/common_constants.gyp:common_constants',
-            '../v8/tools/gyp/v8.gyp:v8',
-          ],
-          'sources': [
-            '../chrome/test/base/v8_unit_test.cc',
-            '../chrome/test/base/v8_unit_test.h',
-            'webapp/browser_globals.gtestjs',
-            'webapp/all_js_load.gtestjs',
-            'webapp/format_iq.gtestjs',
-            '<@(remoting_webapp_all_js_files)',
-          ],
-        }],
         [ 'OS=="android"', {
           'dependencies!': [
             'remoting_client_plugin',
@@ -264,7 +230,7 @@
         '../testing/gmock/include',
       ],
       'sources': [
-        '../chrome/test/base/run_all_remoting_unittests.cc',
+        'base/run_all_unittests.cc',
         'codec/codec_test.cc',
         'codec/codec_test.h',
         'codec/video_encoder_vpx_perftest.cc',
@@ -330,6 +296,10 @@
       'type': 'none',
       'variables': {
         'output_dir': '<(PRODUCT_DIR)/remoting/unittests',
+        'webapp_js_files': [
+          '<@(remoting_webapp_main_html_js_files)',
+          '<@(remoting_webapp_js_wcs_sandbox_files)',
+        ]
       },
       'copies': [
         {
@@ -357,7 +327,7 @@
         {
           'destination': '<(output_dir)',
           'files': [
-            '<@(remoting_webapp_main_html_js_files)',
+            '<@(webapp_js_files)',
           ],
         },
         {
@@ -373,7 +343,7 @@
           'inputs': [
             'webapp/build-html.py',
             '<(remoting_webapp_unittest_template_main)',
-            '<@(remoting_webapp_main_html_js_files)',
+            '<@(webapp_js_files)',
             '<@(remoting_webapp_unittest_cases)'
           ],
           'outputs': [
@@ -389,7 +359,7 @@
             # exclude list.
             '--exclude-js', '<@(remoting_webapp_unittest_exclude_files)',
             '--js', '<@(remoting_webapp_unittest_cases)',
-            '--instrument-js', '<@(remoting_webapp_main_html_js_files)',
+            '--instrument-js', '<@(webapp_js_files)',
            ],
         },
       ],