Fix syncing of NPAPI plugins.

chrome/common/extensions/sync_helper.cc

Index: chrome/common/extensions/sync_helper.cc
diff --git a/chrome/common/extensions/sync_helper.cc b/chrome/common/extensions/sync_helper.cc
index 3cffcca357f1813b17dbe5756784133c71933d9b..e4c88f235e9e9ffc53fc6136942cf5b005bb8ae9 100644
--- a/chrome/common/extensions/sync_helper.cc
+++ b/chrome/common/extensions/sync_helper.cc
@@ -41,7 +41,8 @@ SyncType GetSyncType(const Extension* extension) {
   //
   // TODO(akalin): Relax this restriction once we've put in UI to
   // approve synced extensions.
-  if (PluginInfo::HasPlugins(extension))
+  if (PluginInfo::HasPlugins(extension) ||

[jln (very slow on Chromium), 2013/06/20 02:28:52]

Shouldn't we patch HasPlugins() instead?

It would change the semantics of HasPlugins a bit, but it would be harder to
make a mistake.

Alternatively create a new HasSomethingToDoWithPlugins() ? :)

[Jeffrey Yasskin, 2013/06/20 03:08:42]

HasPlugins is used to control content settings
(https://code.google.com/p/chromium/codesearch/#chromium/src/chrome/browser/co...)
(???), so maybe it's not such a good idea to change it.

+      extension->HasAPIPermission(APIPermission::kPlugin))
     return SYNC_TYPE_NONE;
 
   switch (extension->GetType()) {