Exclude PluginService for builds with enable_plugins==0

chrome/browser/chrome_plugin_service_filter.cc

Index: chrome/browser/chrome_plugin_service_filter.cc
diff --git a/chrome/browser/chrome_plugin_service_filter.cc b/chrome/browser/chrome_plugin_service_filter.cc
index 67fb0f19c94b56f7de49da58dd4685162683664f..4dc3eabf0b67655955c9b688b1908c620a2325f4 100644
--- a/chrome/browser/chrome_plugin_service_filter.cc
+++ b/chrome/browser/chrome_plugin_service_filter.cc
@@ -150,6 +150,7 @@ void ChromePluginServiceFilter::Observe(
       break;
     }
     case chrome::NOTIFICATION_PLUGIN_ENABLE_STATUS_CHANGED: {
+#if defined(ENABLE_PLUGINS)

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

it seems that this whole file is unnecessary if ENABLE_PLUGINS isn't defined

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

Done. Had to ifdef calls to ChromePluginServiceFilter::RegisterResourceContext
in profile_impl.cc, but I think this is cleaner now.

       Profile* profile = content::Source<Profile>(source).ptr();
       PluginService::GetInstance()->PurgePluginListCache(profile, false);
       if (profile && profile->HasOffTheRecordProfile()) {
@@ -157,6 +158,7 @@ void ChromePluginServiceFilter::Observe(
             profile->GetOffTheRecordProfile(), false);
       }
       break;
+#endif
     }
     default: {
       NOTREACHED();