Apps&Extensions for supervised users: Allow the custodian to remotely uninstall extensions.

chrome/browser/ui/app_list/app_list_syncable_service.cc

Index: chrome/browser/ui/app_list/app_list_syncable_service.cc
diff --git a/chrome/browser/ui/app_list/app_list_syncable_service.cc b/chrome/browser/ui/app_list/app_list_syncable_service.cc
index c1a03b6d6860343dffd8de27b81f935cd33d711a..f9f256152380bb71c47f5d87074ba879e5f198f7 100644
--- a/chrome/browser/ui/app_list/app_list_syncable_service.cc
+++ b/chrome/browser/ui/app_list/app_list_syncable_service.cc
@@ -117,8 +117,16 @@ bool IsUnRemovableDefaultApp(const std::string& id) {
 }
 
 void UninstallExtension(ExtensionService* service, const std::string& id) {
-  if (service && service->GetInstalledExtension(id))
-    service->UninstallExtension(id, extensions::UNINSTALL_REASON_SYNC, NULL);
+
+  const extensions::Extension* extension =
+      service ? service->GetInstalledExtension(id) : NULL;
+  if (extension) {
+    extensions::UninstallReason reason =
+        extension->was_installed_by_custodian() ?
+            extensions::UNINSTALL_REASON_SYNC_BY_CUSTODIAN :
+            extensions::UNINSTALL_REASON_SYNC;
+    service->UninstallExtension(id, reason, NULL);

[stevenjb, 2014/07/22 00:47:58]

It seems like this should be automatic, or there should be a helper function for
this, since the logic is not specific to the app list sync code.

[Marc Treib, 2014/07/22 09:30:07]

Ack. With the change suggested by kalman, this has become unnecessary. Thanks
for the feedback!

+  }
 }
 
 bool GetAppListItemType(AppListItem* item,