Move Commands from Extension to CommandsHandler

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.
 {
   'targets': [
     {
       # This target contains mocks and test utilities that don't belong in
       # production libraries but are used by more than one test executable.
       'target_name': 'test_support_common',
       'type': 'static_library',
@@ skipping 1453 matching lines @@
         'browser/webdata/web_intents_table_unittest.cc',
         'common/bzip2_unittest.cc',
         'common/cancelable_task_tracker_unittest.cc',
         'common/child_process_logging_mac_unittest.mm',
         'common/chrome_paths_unittest.cc',
         'common/cloud_print/cloud_print_helpers_unittest.cc',
         'common/chrome_content_client_unittest.cc',
         'common/content_settings_helper_unittest.cc',
         'common/content_settings_pattern_parser_unittest.cc',
         'common/content_settings_pattern_unittest.cc',
+        'common/extensions/api/commands/commands_manifest_unittest.cc',
         'common/extensions/api/extension_action/script_badge_manifest_unittest.cc',
         'common/extensions/command_unittest.cc',
         'common/extensions/csp_validator_unittest.cc',
         'common/extensions/event_filter_unittest.cc',
         'common/extensions/extension_constants_unittest.cc',
         'common/extensions/extension_file_util_unittest.cc',
         'common/extensions/extension_icon_set_unittest.cc',
         'common/extensions/extension_l10n_util_unittest.cc',
         'common/extensions/extension_localization_peer_unittest.cc',
         'common/extensions/extension_resource_unittest.cc',
         'common/extensions/extension_set_unittest.cc',
         'common/extensions/extension_unittest.cc',
         'common/extensions/feature_switch_unittest.cc',
         'common/extensions/features/base_feature_provider_unittest.cc',
         'common/extensions/features/complex_feature_unittest.cc',
         'common/extensions/features/simple_feature_unittest.cc',
         'common/extensions/manifest_tests/extension_manifest_test.cc',
         'common/extensions/manifest_tests/extension_manifests_auth_unittest.cc',
         'common/extensions/manifest_tests/extension_manifests_background_unittest.cc',
         'common/extensions/manifest_tests/extension_manifests_browseraction_unittest.cc',
         'common/extensions/manifest_tests/extension_manifests_chromepermission_unittest.cc',
-        'common/extensions/manifest_tests/extension_manifests_command_unittest.cc',
         'common/extensions/manifest_tests/extension_manifests_contentscript_unittest.cc',
         'common/extensions/manifest_tests/extension_manifests_contentsecuritypolicy_unittest.cc',
         'common/extensions/manifest_tests/extension_manifests_default_unittest.cc',
         'common/extensions/manifest_tests/extension_manifests_devtools_unittest.cc',
         'common/extensions/manifest_tests/extension_manifests_excludematches_unittest.cc',
         'common/extensions/manifest_tests/extension_manifests_experimental_unittest.cc',
         'common/extensions/manifest_tests/extension_manifests_homepage_unittest.cc',
         'common/extensions/manifest_tests/extension_manifests_icon_unittest.cc',
         'common/extensions/manifest_tests/extension_manifests_initvalue_unittest.cc',
         'common/extensions/manifest_tests/extension_manifests_isolatedapp_unittest.cc',
@@ skipping 253 matching lines @@
           'sources/': [
             ['exclude', '^browser/extensions/activity_database_unittest.cc'],
             ['exclude', '^browser/extensions/activity_log_unittest.cc'],
             ['exclude', '^browser/extensions/api/'],
             ['exclude', '^browser/sync/glue/chrome_extensions_activity_monitor_unittest.cc'],
             ['exclude', '^common/extensions/api/'],
           ],
           'sources!':[
             'browser/extensions/extension_context_menu_model_unittest.cc',
             'browser/extensions/extension_ui_unittest.cc',
             'browser/extensions/permissions_updater_unittest.cc',

[nilesh, 2013/01/23 18:27:54]

Please exclude the test here (in enable_extentions==0)

[Devlin, 2013/01/23 20:21:17]

Done.  Sorry about that; I'm not very familiar with gypis.

           ],
         }],
         ['use_ash==1', {
           'sources': [
             'browser/ui/app_list/apps_model_builder_unittest.cc',
           ],
           'dependencies': [
             '../ash/ash.gyp:ash_resources',
             '../ash/ash.gyp:ash_test_support',
             '../ash/ash_strings.gyp:ash_strings',
@@ skipping 404 matching lines @@
             ['exclude', '^browser/ui/webui/flags_'],
             ['exclude', '^browser/ui/webui/help/'],
             ['exclude', '^browser/ui/webui/options/'],
             ['exclude', '^browser/ui/webui/options/'],
             ['exclude', '^browser/ui/webui/signin/'],
             ['exclude', '^browser/ui/webui/suggestions_internals'],
             ['exclude', '^browser/ui/webui/sync_promo'],
             # No service process on Android.
             ['exclude', '^browser/service/'],
             ['exclude', '^common/service_'],
+            ['exclude', '^common/extensions/extension_unittest.cc'],

[nilesh, 2013/01/23 18:27:54]

Please exclude this file using the "sources!" list in the enable_extensions==0
block.

[Devlin, 2013/01/23 20:21:17]

Done.

             ['exclude', '^service/'],
           ],
           'conditions': [
             ['gtest_target_type == "shared_library"', {
               'dependencies': [
                 '../testing/android/native_test.gyp:native_test_native_code',
               ],
             }],
           ],
         }],  # OS == android
@@ skipping 144 matching lines @@
             'unit_tests.isolate',
           ],
           'sources': [
             'unit_tests.isolate',
           ],
         },
       ],
     }],
   ],  # 'conditions'
 }