Revert https://codereview.chromium.org/1060993003

chrome/browser/ui/webui/extensions/extension_settings_handler.cc

Index: chrome/browser/ui/webui/extensions/extension_settings_handler.cc
diff --git a/chrome/browser/ui/webui/extensions/extension_settings_handler.cc b/chrome/browser/ui/webui/extensions/extension_settings_handler.cc
index b710c430d77c59acda4daf45059b914af0215874..cb9dfe559c0274f41c3dbb0a1b9a0a47434e5ab8 100644
--- a/chrome/browser/ui/webui/extensions/extension_settings_handler.cc
+++ b/chrome/browser/ui/webui/extensions/extension_settings_handler.cc
@@ -11,6 +11,7 @@
 #include "base/bind_helpers.h"
 #include "base/location.h"
 #include "base/message_loop/message_loop.h"
+#include "base/metrics/histogram.h"
 #include "base/prefs/pref_service.h"
 #include "base/strings/string_util.h"
 #include "base/strings/utf_string_conversions.h"
@@ -23,9 +24,11 @@
 #include "chrome/browser/extensions/extension_service.h"
 #include "chrome/browser/extensions/extension_tab_util.h"
 #include "chrome/browser/extensions/extension_ui_util.h"
+#include "chrome/browser/extensions/install_verifier.h"
 #include "chrome/browser/extensions/updater/extension_updater.h"
 #include "chrome/browser/extensions/webstore_reinstaller.h"
 #include "chrome/browser/platform_util.h"
+#include "chrome/browser/prefs/incognito_mode_prefs.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/ui/apps/app_info_dialog.h"
 #include "chrome/browser/ui/browser.h"
@@ -86,6 +89,7 @@ ExtensionSettingsHandler::ExtensionSettingsHandler()
       deleting_rvh_(NULL),
       deleting_rwh_id_(-1),
       deleting_rph_id_(-1),
+      registered_for_notifications_(false),
       warning_service_observer_(this),
       extension_prefs_observer_(this),
       extension_management_observer_(this) {
@@ -315,6 +319,12 @@ void ExtensionSettingsHandler::RegisterMessages() {
   extension_service_ =
       extensions::ExtensionSystem::Get(profile)->extension_service();
 
+  web_ui()->RegisterMessageCallback("extensionSettingsRequestExtensionsData",
+      base::Bind(&ExtensionSettingsHandler::HandleRequestExtensionsData,
+                 AsWeakPtr()));
+  web_ui()->RegisterMessageCallback("extensionSettingsToggleDeveloperMode",
+      base::Bind(&ExtensionSettingsHandler::HandleToggleDeveloperMode,
+                 AsWeakPtr()));
   web_ui()->RegisterMessageCallback("extensionSettingsLaunch",
       base::Bind(&ExtensionSettingsHandler::HandleLaunchMessage,
                  AsWeakPtr()));
@@ -330,9 +340,6 @@ void ExtensionSettingsHandler::RegisterMessages() {
   web_ui()->RegisterMessageCallback("extensionSettingsShowPath",
       base::Bind(&ExtensionSettingsHandler::HandleShowPath,
                  AsWeakPtr()));
-  web_ui()->RegisterMessageCallback("extensionSettingsRegister",
-      base::Bind(&ExtensionSettingsHandler::HandleRegisterMessage,
-                 AsWeakPtr()));
 }
 
 void ExtensionSettingsHandler::Observe(
@@ -417,6 +424,73 @@ void ExtensionSettingsHandler::ReloadUnpackedExtensions() {
   }
 }
 
+void ExtensionSettingsHandler::HandleRequestExtensionsData(
+    const base::ListValue* args) {
+  // The items which are to be written into results are also described in
+  // chrome/browser/resources/extensions/extensions.js in @typedef for
+  // ExtensionDataResponse. Please update it whenever you add or remove any keys
+  // here.
+  base::DictionaryValue results;
+
+  Profile* profile = Profile::FromWebUI(web_ui());
+
+  bool is_supervised = profile->IsSupervised();
+  bool incognito_available =
+      IncognitoModePrefs::GetAvailability(profile->GetPrefs()) !=
+          IncognitoModePrefs::DISABLED;
+  bool developer_mode =
+      !is_supervised &&
+      profile->GetPrefs()->GetBoolean(prefs::kExtensionsUIDeveloperMode);
+  results.SetBoolean("profileIsSupervised", is_supervised);
+  results.SetBoolean("incognitoAvailable", incognito_available);
+  results.SetBoolean("developerMode", developer_mode);
+  results.SetBoolean("enableAppInfoDialog", CanShowAppInfoDialog());
+
+  const bool load_unpacked_disabled =
+      ExtensionManagementFactory::GetForBrowserContext(profile)
+          ->BlacklistedByDefault();
+  results.SetBoolean("loadUnpackedDisabled", load_unpacked_disabled);
+
+  web_ui()->CallJavascriptFunction(
+      "extensions.ExtensionSettings.returnExtensionsData", results);
+
+  MaybeRegisterForNotifications();
+
+  scoped_ptr<ExtensionSet> extensions =
+      ExtensionRegistry::Get(profile)->GenerateInstalledExtensionsSet(
+          ExtensionRegistry::ENABLED |
+          ExtensionRegistry::DISABLED |
+          ExtensionRegistry::TERMINATED);
+  ExtensionPrefs* prefs = ExtensionPrefs::Get(profile);
+  bool should_do_verification_check = false;
+  for (const scoped_refptr<const Extension>& extension : *extensions) {
+    if (ui_util::ShouldDisplayInExtensionSettings(extension.get(), profile)) {
+      int disable_reasons = prefs->GetDisableReasons(extension->id());
+      if ((disable_reasons & Extension::DISABLE_NOT_VERIFIED)) {
+        should_do_verification_check = true;
+        break;
+      }
+    }
+  }
+
+  UMA_HISTOGRAM_BOOLEAN("ExtensionSettings.ShouldDoVerificationCheck",
+                        should_do_verification_check);
+  if (should_do_verification_check)
+    ExtensionSystem::Get(profile)->install_verifier()->VerifyAllExtensions();
+}
+
+void ExtensionSettingsHandler::HandleToggleDeveloperMode(
+    const base::ListValue* args) {
+  Profile* profile = Profile::FromWebUI(web_ui());
+  if (profile->IsSupervised())
+    return;
+
+  bool developer_mode_on;
+  CHECK(args->GetBoolean(0, &developer_mode_on));
+  profile->GetPrefs()->SetBoolean(prefs::kExtensionsUIDeveloperMode,
+                                  developer_mode_on);
+}
+
 void ExtensionSettingsHandler::HandleLaunchMessage(
     const base::ListValue* args) {
   CHECK_EQ(1U, args->GetSize());
@@ -477,11 +551,25 @@ void ExtensionSettingsHandler::HandleShowPath(const base::ListValue* args) {
                                   extension->path().Append(kManifestFilename));
 }
 
-void ExtensionSettingsHandler::HandleRegisterMessage(
+const Extension* ExtensionSettingsHandler::GetActiveExtension(
     const base::ListValue* args) {
-  if (!registrar_.IsEmpty())
-    return;  // Only register once.
+  std::string extension_id = base::UTF16ToUTF8(ExtractStringValue(args));
+  CHECK(!extension_id.empty());
+  return extension_service_->GetExtensionById(extension_id, false);
+}
+
+void ExtensionSettingsHandler::MaybeUpdateAfterNotification() {
+  content::WebContents* contents = web_ui()->GetWebContents();
+  if (!ignore_notifications_ && contents && contents->GetRenderViewHost())
+    HandleRequestExtensionsData(NULL);
+  deleting_rvh_ = NULL;
+}
 
+void ExtensionSettingsHandler::MaybeRegisterForNotifications() {
+  if (registered_for_notifications_)
+    return;
+
+  registered_for_notifications_  = true;
   Profile* profile = Profile::FromWebUI(web_ui());
 
   // Register for notifications that we need to reload the page.
@@ -520,22 +608,6 @@ void ExtensionSettingsHandler::HandleRegisterMessage(
   profile->GetPrefs()->ClearPref(prefs::kExtensionsUIDismissedADTPromo);
 }
 
-const Extension* ExtensionSettingsHandler::GetActiveExtension(
-    const base::ListValue* args) {
-  std::string extension_id = base::UTF16ToUTF8(ExtractStringValue(args));
-  CHECK(!extension_id.empty());
-  return extension_service_->GetExtensionById(extension_id, false);
-}
-
-void ExtensionSettingsHandler::MaybeUpdateAfterNotification() {
-  content::WebContents* contents = web_ui()->GetWebContents();
-  if (!ignore_notifications_ && contents && contents->GetRenderViewHost()) {
-    web_ui()->CallJavascriptFunction(
-        "extensions.ExtensionSettings.onExtensionsChanged");
-  }
-  deleting_rvh_ = NULL;
-}
-
 void ExtensionSettingsHandler::OnReinstallComplete(
     bool success,
     const std::string& error,