Supervised Users Initiated Installs v2

chrome/browser/extensions/extension_util.cc

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/extensions/extension_util.h"
 
 #include "base/command_line.h"
 #include "base/logging.h"
 #include "base/metrics/field_trial.h"
 #include "base/values.h"
@@ skipping 239 matching lines @@
 
   // If it is already enabled, do nothing.
   if (registry->enabled_extensions().Contains(extension_id))
     return;
 
   // If the extension is not loaded, it may need to be reloaded.
   // Example is a pre-installed extension that was unloaded when a
   // supervised user flag has been received.
   if (!extension) {
     service->ReloadExtension(extension_id);
+    service->EnableExtension(extension_id);

[Marc Treib, 2016/06/24 10:58:55]

?
I'm pretty sure we don't want to do this.

[mamir, 2016/06/24 14:43:07]

For the record, discussed in patchset 29

   }
 }
 
 bool WasInstalledByCustodian(const std::string& extension_id,
                              content::BrowserContext* context) {
   bool installed_by_custodian = false;
   ExtensionPrefs* prefs = ExtensionPrefs::Get(context);
   prefs->ReadPrefAsBoolean(extension_id, kWasInstalledByCustodianPrefName,
                            &installed_by_custodian);
   return installed_by_custodian;
@@ skipping 169 matching lines @@
   base::FieldTrialList::FindFullName(
       kSupervisedUserExtensionPermissionIncreaseFieldTrialName);
   std::string value = variations::GetVariationParamValue(
       kSupervisedUserExtensionPermissionIncreaseFieldTrialName,
       profile->IsChild() ? "child_account" : "legacy_supervised_user");
   return value == "true";
 }
 
 }  // namespace util
 }  // namespace extensions