Exclude PluginService for builds with enable_plugins==0

chrome/chrome_tests_unit.gypi

Index: chrome/chrome_tests_unit.gypi
diff --git a/chrome/chrome_tests_unit.gypi b/chrome/chrome_tests_unit.gypi
index 91de10ae7689e18b2036cdbc97e61b84040c4273..77925db2b26455a4c42d01c868ce23d9fe2a951c 100644
--- a/chrome/chrome_tests_unit.gypi
+++ b/chrome/chrome_tests_unit.gypi
@@ -1827,6 +1827,14 @@
             ['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/'],