Move most of the component updater artifacts to update_client.

components/components_tests.gyp

 # Copyright 2014 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': {
     # This turns on e.g. the filename-based detection of which
     # platforms to include source files on (e.g. files ending in
     # _mac.h or _mac.cc are only compiled on MacOSX).
     'chromium_code': 1,
@@ skipping 63 matching lines @@
             'bookmarks/browser/bookmark_utils_unittest.cc',
             'browser_watcher/endsession_watcher_window_win_unittest.cc',
             'browser_watcher/exit_funnel_win_unittest.cc',
             'browser_watcher/exit_code_watcher_win_unittest.cc',
             'browser_watcher/watcher_client_win_unittest.cc',
             'browser_watcher/watcher_metrics_provider_win_unittest.cc',
             'crash/app/crash_keys_win_unittest.cc',
             'captive_portal/captive_portal_detector_unittest.cc',
             'cloud_devices/common/cloud_devices_urls_unittest.cc',
             'cloud_devices/common/printer_description_unittest.cc',
-            'component_updater/test/component_patcher_unittest.cc',
-            'component_updater/test/component_updater_ping_manager_unittest.cc',
-            'component_updater/test/crx_downloader_unittest.cc',
-            'component_updater/test/request_sender_unittest.cc',
-            'component_updater/test/update_checker_unittest.cc',
-            'component_updater/test/update_response_unittest.cc',
             'content_settings/core/browser/content_settings_mock_provider.cc',
             'content_settings/core/browser/content_settings_mock_provider.h',
             'content_settings/core/browser/content_settings_provider_unittest.cc',
             'content_settings/core/browser/content_settings_rule_unittest.cc',
             'content_settings/core/browser/content_settings_utils_unittest.cc',
             'content_settings/core/common/content_settings_pattern_parser_unittest.cc',
             'content_settings/core/common/content_settings_pattern_unittest.cc',
             'crx_file/id_util_unittest.cc',
             'data_reduction_proxy/core/browser/data_reduction_proxy_auth_request_handler_unittest.cc',
             'data_reduction_proxy/core/browser/data_reduction_proxy_bypass_protocol_unittest.cc',
@@ skipping 174 matching lines @@
             'translate/core/browser/translate_browser_metrics_unittest.cc',
             'translate/core/browser/translate_prefs_unittest.cc',
             'translate/core/browser/translate_script_unittest.cc',
             'translate/core/common/translate_metrics_unittest.cc',
             'translate/core/common/translate_util_unittest.cc',
             'translate/core/language_detection/language_detection_util_unittest.cc',
             'translate/ios/browser/js_translate_manager_unittest.mm',
             'translate/ios/browser/language_detection_controller_unittest.mm',
             'translate/ios/browser/translate_controller_unittest.mm',
             'ui/zoom/page_zoom_unittests.cc',
+            'update_client/test/component_patcher_unittest.cc',
+            'update_client/test/crx_downloader_unittest.cc',
+            'update_client/test/ping_manager_unittest.cc',
+            'update_client/test/request_sender_unittest.cc',
+            'update_client/test/update_checker_unittest.cc',
+            'update_client/test/update_response_unittest.cc',
             'update_client/update_query_params_unittest.cc',
             'url_matcher/regex_set_matcher_unittest.cc',
             'url_matcher/string_pattern_unittest.cc',
             'url_matcher/substring_set_matcher_unittest.cc',
             'url_matcher/url_matcher_factory_unittest.cc',
             'url_matcher/url_matcher_unittest.cc',
             'url_fixer/url_fixer_unittest.cc',
             'variations/active_field_trials_unittest.cc',
             'variations/caching_permuted_entropy_provider_unittest.cc',
             'variations/entropy_provider_unittest.cc',
@@ skipping 49 matching lines @@
             'components.gyp:bookmarks_browser',
             'components.gyp:bookmarks_test_support',
 
             # Dependencies of captive_portal
             'components.gyp:captive_portal_test_support',
             '../net/net.gyp:net_test_support',
 
             # Dependencies of cloud_devices
             'components.gyp:cloud_devices_common',
 
             # Dependencies of component_updater

[blundell, 2015/01/21 10:29:04]

there aren't any component_updater unittests now so these dependencies can go
away, right?

[Sorin Jianu, 2015/01/21 18:29:46]

Done.

             'components.gyp:component_updater',
-            'components.gyp:component_updater_test_support',
-            '../third_party/libxml/libxml.gyp:libxml',
+            'components.gyp:update_client',
 
             # Dependencies of content_settings
             'components.gyp:content_settings_core_browser',
             'components.gyp:content_settings_core_common',
             'components.gyp:content_settings_core_test_support',
 
             # Dependencies of packed CT EV white list
             'components.gyp:packed_ct_ev_whitelist',
 
             # Dependencies of crash
@@ skipping 128 matching lines @@
             'components.gyp:translate_core_language_detection',
 
             # Dependencies of wallpaper
             'components.gyp:wallpaper',
 
             # Dependencies of ui/zoom
             'components.gyp:ui_zoom',
 
             # Dependencies of update_client
             'components.gyp:update_client',
+            'components.gyp:update_client_test_support',
+            '../third_party/libxml/libxml.gyp:libxml',
+
 
             # Dependencies of url_fixer
             'components.gyp:url_fixer',
             '../url/url.gyp:url_lib',
 
             # Dependencies of variations
             'components.gyp:variations',
             'components.gyp:variations_http_provider',
 
             # Dependencies of web_resource
@@ skipping 312 matching lines @@
                 'timers/alarm_timer_unittest.cc',
                 'wifi_sync/wifi_config_delegate_chromeos_unittest.cc',
                 'wifi_sync/wifi_credential_unittest.cc',
                 'wifi_sync/wifi_security_class_chromeos_unittest.cc',
                 'wifi_sync/wifi_security_class_unittest.cc',
               ],
               'sources!': [
                 'storage_monitor/storage_monitor_linux_unittest.cc',
               ],
               'dependencies': [
-                # Dependencies of wifi_sync
+                # Dependencies of wifi_sync
                 'components.gyp:wifi_sync',
 
                 'components.gyp:pairing',
                 '../chromeos/chromeos.gyp:chromeos_test_support',
               ],
             }],
             ['OS=="linux"', {
               'sources': [
                 'metrics/serialization/serialization_utils_unittest.cc',
               ],
@@ skipping 296 matching lines @@
             '../build/isolate.gypi',
           ],
           'sources': [
             'components_unittests.isolate',
           ],
         },
       ],
     }],
   ],
 }