Modify IsSyncable to only include Internal items and the Chrome Web Store

chrome/common/extensions/extension.cc

Index: chrome/common/extensions/extension.cc
diff --git a/chrome/common/extensions/extension.cc b/chrome/common/extensions/extension.cc
index 9e2ac3b960b9dda86cc5ff5029a76ffbf157d569..d30fa3c4f4ef31f4ba40453b7d443a61d2e181a4 100644
--- a/chrome/common/extensions/extension.cc
+++ b/chrome/common/extensions/extension.cc
@@ -3659,10 +3659,10 @@ Extension::SyncType Extension::GetSyncType() const {
 bool Extension::IsSyncable() const {
   // TODO(akalin): Figure out if we need to allow some other types.
 
-  // We want to sync any extensions that are shown in the launcher because
-  // their positions should sync.
+  // We want to sync any extensions that are internal and should appear on
+  // multiple machines, and the chrome web store.

[Finnur, 2012/04/12 15:01:40]

nit: 'and should appear on multiple machines' is either redundant or you are
missing further checks in the return statement...

[csharp, 2012/04/12 15:06:29]

Done.

   return location() == Extension::INTERNAL ||
-      ShouldDisplayInLauncher();
+      id() == extension_misc::kWebStoreAppId;
 }
 
 bool Extension::ShouldDisplayInLauncher() const {