Enable/disable extensions upon changes in blacklist.

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

 // Copyright (c) 2012 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/ui/webui/extensions/extension_settings_handler.h"
 
 #include "apps/app_load_service.h"
 #include "apps/app_restore_service.h"
 #include "apps/saved_files_service.h"
 #include "apps/shell_window.h"
@@ skipping 51 matching lines @@
 #include "content/public/browser/notification_source.h"
 #include "content/public/browser/notification_types.h"
 #include "content/public/browser/render_process_host.h"
 #include "content/public/browser/render_view_host.h"
 #include "content/public/browser/site_instance.h"
 #include "content/public/browser/user_metrics.h"
 #include "content/public/browser/web_contents.h"
 #include "content/public/browser/web_contents_view.h"
 #include "content/public/browser/web_ui.h"
 #include "content/public/browser/web_ui_data_source.h"
+#include "extensions/browser/blacklist_state.h"
 #include "extensions/browser/extension_error.h"
 #include "extensions/browser/extension_registry.h"
 #include "extensions/browser/lazy_background_task_queue.h"
 #include "extensions/browser/management_policy.h"
 #include "extensions/browser/pref_names.h"
 #include "extensions/browser/view_type_utils.h"
 #include "extensions/common/constants.h"
 #include "extensions/common/extension.h"
 #include "extensions/common/extension_set.h"
 #include "extensions/common/feature_switch.h"
@@ skipping 153 matching lines @@
   extension_data->SetBoolean("wantsFileAccess", extension->wants_file_access());
   extension_data->SetBoolean("allowFileAccess",
       extension_util::AllowFileAccess(extension, extension_service_));
   extension_data->SetBoolean("allow_reload",
       Manifest::IsUnpackedLocation(extension->location()));
   extension_data->SetBoolean("is_hosted_app", extension->is_hosted_app());
   extension_data->SetBoolean("is_platform_app", extension->is_platform_app());
   extension_data->SetBoolean("homepageProvided",
       ManifestURL::GetHomepageURL(extension).is_valid());
 
+  BlacklistState blacklist_state =
+      extension_service_->extension_prefs()->GetExtensionBlacklistState(
+          extension->id());
   base::string16 location_text;
   if (Manifest::IsPolicyLocation(extension->location())) {
     location_text = l10n_util::GetStringUTF16(
         IDS_OPTIONS_INSTALL_LOCATION_ENTERPRISE);
   } else if (extension->location() == Manifest::INTERNAL &&
       !ManifestURL::UpdatesFromGallery(extension)) {
     location_text = l10n_util::GetStringUTF16(
         IDS_OPTIONS_INSTALL_LOCATION_UNKNOWN);
   } else if (extension->location() == Manifest::EXTERNAL_REGISTRY) {
     location_text = l10n_util::GetStringUTF16(
         IDS_OPTIONS_INSTALL_LOCATION_3RD_PARTY);
+  } else if (blacklist_state != NOT_BLACKLISTED) {

[Finnur, 2014/01/23 15:21:38]

It is a bit weird that we'd categorize this label lower in priority than the one
for policy-extensions, the one for internal-but-not-from-webstore-extensions and
for registry-extensions. 

But lets take a step back. With the blacklist label this is really not an
either-or kind of situation anymore, like |location| is. It is not a stretch to
imagine us blacklisting one of the types of extensions that the if clauses (on
line 250-259 above) cover. We'd never see the blacklist label in that case,
which doesn't seem right.

This label should, in my mind be separate and added right after the location
based one.

[Oleg Eterevsky, 2014/01/23 16:33:06]

I'll ask a PM working on this feature to find the right people to decide what
should happen if these two states coincide. In my opinion, this shouldn't really
happen, and we should just show the more important message.

+    switch (blacklist_state) {
+      case BLACKLISTED_SECURITY_VULNERABILITY:
+        location_text = l10n_util::GetStringUTF16(
+            IDS_OPTIONS_BLACKLISTED_SECURITY_VULNERABILITY);
+        break;
+
+      case BLACKLISTED_CWS_POLICY_VIOLATION:
+        location_text = l10n_util::GetStringUTF16(
+            IDS_OPTIONS_BLACKLISTED_CWS_POLICY_VIOLATION);
+        break;
+
+      case BLACKLISTED_POTENTIALLY_UNWANTED:
+        location_text = l10n_util::GetStringUTF16(
+            IDS_OPTIONS_BLACKLISTED_POTENTIALLY_UNWANTED);
+        break;

[Finnur, 2014/01/23 15:21:38]

Why is BLACKLISTED_MALWARE not handled?

[Oleg Eterevsky, 2014/01/23 16:33:06]

Because malware extensions are not shown in the list at all. We do not even have
a string for them.

+
+      default:
+        break;
+    }
   }
   extension_data->SetString("locationText", location_text);
 
   // Force unpacked extensions to show at the top.
   if (Manifest::IsUnpackedLocation(extension->location()))
     extension_data->SetInteger("order", 1);
   else
     extension_data->SetInteger("order", 2);
 
   if (!ExtensionActionAPI::GetBrowserActionVisibility(
@@ skipping 953 matching lines @@
     extension_service_->EnableExtension(extension_id);
   } else {
     ExtensionErrorReporter::GetInstance()->ReportError(
         base::UTF8ToUTF16(JoinString(requirement_errors, ' ')),
         true /* be noisy */);
   }
   requirements_checker_.reset();
 }
 
 }  // namespace extensions