MD Downloads: add a JS unit test for search term splitting

chrome/chrome_tests_unit.gypi

 # Copyright 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.
 {
   'variables': {
     'chrome_unit_tests_sources': [
       # histograms.xml is analyzed by AboutFlagsHistogramTest, so this
       # dependency is needed to make commit bots run unit_tests on
       # histograms.xml changes.
       '../tools/metrics/histograms/histograms.xml',
@@ skipping 1075 matching lines @@
       'browser/media/webrtc_rtp_dump_writer_unittest.cc',
       'renderer/media/chrome_webrtc_log_message_delegate_unittest.cc',
     ],
     'chrome_unit_tests_js_sources': [
       'browser/resources/google_now/background.js',
       'browser/resources/google_now/background_test_util.js',
       'browser/resources/google_now/cards.js',
       'browser/resources/google_now/common_test_util.js',
       'browser/resources/google_now/utility.js',
       'browser/resources/google_now/utility_test_util.js',
+      'browser/resources/md_downloads/action_service.js',
       'browser/resources/print_preview/data/measurement_system.js',
       'browser/resources/print_preview/print_preview_utils.js',
       'renderer/resources/extensions/notifications_custom_bindings.js',
       'renderer/resources/extensions/notifications_test_util.js',
       '../ui/webui/resources/js/cr.js',
     ],
     'chrome_unit_tests_gtestjs_sources': [
       'browser/resources/google_now/background_unittest.gtestjs',
       'browser/resources/google_now/cards_unittest.gtestjs',
       'browser/resources/google_now/utility_unittest.gtestjs',
+      'browser/resources/md_downloads/action_service_unittest.gtestjs',
       'browser/resources/print_preview/data/measurement_system_unittest.gtestjs',
       'browser/resources/print_preview/print_preview_utils_unittest.gtestjs',
       'renderer/resources/extensions/notifications_custom_bindings.gtestjs',
       'test/data/unit/framework_unittest.gtestjs',
     ],
     'chrome_unit_tests_chromeos_sources': [
       # TODO(lukasza): Move Drive tests outside of CrOS (crbug.com/498951).
       '../components/drive/change_list_loader_unittest.cc',
       '../components/drive/change_list_processor_unittest.cc',
       '../components/drive/directory_loader_unittest.cc',
@@ skipping 1106 matching lines @@
         'VCLinkerTool': {
           'conditions': [
             ['incremental_chrome_dll==1', {
               'UseLibraryDependencyInputs': "true",
             }],
           ],
         },
       },
       'sources': [
         '<@(chrome_unit_tests_sources)',
-        '<@(chrome_unit_tests_js_sources)',
-        '<@(chrome_unit_tests_gtestjs_sources)',
       ],
       'conditions': [
         ['OS!="ios"', {
           'dependencies': [
             '../components/components.gyp:autofill_content_test_support',
             '../components/components.gyp:component_metrics_proto',
             '../components/components.gyp:data_reduction_proxy_test_support',
             '../components/components.gyp:data_use_measurement_core',
             '../components/components.gyp:safe_json_test_support',
             '../components/components.gyp:webdata_services_test_support',
             '../components/components_strings.gyp:components_strings',
             '../content/app/resources/content_resources.gyp:content_resources',
             '../device/bluetooth/bluetooth.gyp:device_bluetooth_mocks',
             '../gpu/gpu.gyp:gpu_unittest_utils',
             '../media/media.gyp:media_test_support',
             '../ppapi/ppapi_internal.gyp:ppapi_unittest_shared',
             '../third_party/leveldatabase/leveldatabase.gyp:leveldatabase',
             '../third_party/libaddressinput/libaddressinput.gyp:libaddressinput',
             '../third_party/libphonenumber/libphonenumber.gyp:libphonenumber',
             '../third_party/webrtc/modules/modules.gyp:desktop_capture',
             '../tools/json_schema_compiler/test/json_schema_compiler_tests.gyp:json_schema_compiler_tests',
             '../ui/gl/gl.gyp:gl',
             '../ui/gl/gl.gyp:gl_test_support',
-            '../v8/tools/gyp/v8.gyp:v8',
-          ],
-          # TODO(scr): Use this in browser_tests too.
-          'includes': [
-            'js_unittest_rules.gypi',
           ],
           'include_dirs': [
             '<(SHARED_INTERMEDIATE_DIR)/chrome',
           ],
         }, {  # OS=="ios"
           'dependencies': [
             '../third_party/ocmock/ocmock.gyp:ocmock',
           ],
           'sources/': [
             # Exclude everything but iOS-specific files.
@@ skipping 23 matching lines @@
             '../testing/android/native_test.gyp:native_test_native_code',
             '../components/components.gyp:gcm_driver',
           ],
         }, {  # Not Android.
           'sources': [ '<@(chrome_unit_tests_non_android_sources)' ],
           'sources!': [
             'browser/download/download_request_infobar_delegate_unittest.cc',
           ],
         }],
         ['OS!="android" and OS!="ios"', {
-          'sources': [ '<@(chrome_unit_tests_non_mobile_sources)' ],
+          'dependencies': [
+            '../v8/tools/gyp/v8.gyp:v8',
+          ],
+          # TODO(scr): Use this in browser_tests too.

[dpapad, 2015/12/03 01:19:41]

Is this TODO relevant anymore?

[Dan Beam, 2015/12/11 08:17:22]

Done.

+          'includes': [
+            'js_unittest_rules.gypi',
+          ],
+          'include_dirs': [
+            '<(SHARED_INTERMEDIATE_DIR)/chrome',
+          ],
+          'sources': [
+            '<@(chrome_unit_tests_non_mobile_sources)',
+            '<@(chrome_unit_tests_js_sources)',
+            '<@(chrome_unit_tests_gtestjs_sources)',
+          ],
         }],
         ['chromeos==1', {
           'sources!': [
             'browser/signin/chrome_signin_status_metrics_provider_delegate_unittest.cc',
           ],
         }],
         ['enable_background==1', {
           'sources': [ '<@(chrome_unit_tests_background_sources)' ],
         }],
         ['enable_spellcheck==1', {
@@ skipping 675 matching lines @@
               'dependencies': [
                 '../tools/xdisplaycheck/xdisplaycheck.gyp:xdisplaycheck',
               ],
             }],
           ],
         },
       ],
     }],
   ],  # 'conditions'
 }