Exclude PluginService for builds with enable_plugins==0

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 1809 matching lines @@
           'sources/': [
             ['exclude', '^browser/safe_browsing/'],
             ['exclude', '^renderer/safe_browsing/'],
           ],
         }],
         ['enable_automation!=1', {
           'sources/': [
             ['exclude', '^browser/automation/'],
           ],
         }],
+        ['enable_plugins==0', {
+          'sources!': [
+            'browser/extensions/extension_service_unittest.cc', # Needs PluginService

[jam, 2012/12/19 01:55:15]

this seems too wide? i.e. if someone disables plugins it doesn't mean that they
want to disable extensions. if there are tests in that file that depend on
extensions, then these should be wrapped with ENABLE_PLUGINS

[nilesh, 2012/12/19 21:07:44]

Done.

+            'browser/extensions/user_script_listener_unittest.cc',
+            'browser/renderer_host/plugin_info_message_filter_unittest.cc',
+            'browser/plugins/plugin_prefs_unittest.cc',
+          ],
+        }],
         ['enable_printing!=1', {
           'sources/': [
             ['exclude', '^browser/printing/'],
             ['exclude', '^browser/ui/webui/print_preview/'],
           ],
         }],
         ['enable_captive_portal_detection!=1', {
           'sources/': [
             ['exclude', '^browser/captive_portal/'],
           ],
@@ skipping 466 matching lines @@
             'unit_tests.isolate',
           ],
           'sources': [
             'unit_tests.isolate',
           ],
         },
       ],
     }],
   ],  # 'conditions'
 }