Add more management policy checking after extension installed

This CL adds checking for MustRemainDisabled() from management policy after an extension is installed or updated, so that extensions supposed to be disabled will be disabled initially with proper disabled reason set.

This CL also assumes that all disabled extension comes with proper disabled reason, so there are additional changes to ensure this.

1) Another extension disabled reason DISABLE_EXTERNAL_EXTENSION is added for external extensions. These extensions will be disabled initially on windows for user prompting.
2) Two tests from extension_disabled_ui_browsertest.cc is removed since these two tests are meant for legacy disables with not disabled reason set in user pref, which should be rarely seen now. And these rare cases are handled by presuming it's disabled by user action as well.

BUG=None

Review URL: https://codereview.chromium.org/714133002

Cr-Commit-Position: refs/heads/master@{#304587}

1 files changed, 14 insertions, 9 deletions

diff --git a/extensions/browser/extension_prefs.cc b/extensions/browser/extension_prefs.cc
index 4d476f5..dc91d2e 100644
--- a/extensions/browser/extension_prefs.cc
+++ b/extensions/browser/extension_prefs.cc
@@ -746,31 +746,36 @@ bool ExtensionPrefs::HasDisableReason(
 
 void ExtensionPrefs::AddDisableReason(const std::string& extension_id,
                                       Extension::DisableReason disable_reason) {
-  ModifyDisableReason(extension_id, disable_reason, DISABLE_REASON_ADD);
+  ModifyDisableReasons(extension_id, disable_reason, DISABLE_REASON_ADD);
+}
+
+void ExtensionPrefs::AddDisableReasons(const std::string& extension_id,
+                                       int disable_reasons) {
+  ModifyDisableReasons(extension_id, disable_reasons, DISABLE_REASON_ADD);
 }
 
 void ExtensionPrefs::RemoveDisableReason(
     const std::string& extension_id,
     Extension::DisableReason disable_reason) {
-  ModifyDisableReason(extension_id, disable_reason, DISABLE_REASON_REMOVE);
+  ModifyDisableReasons(extension_id, disable_reason, DISABLE_REASON_REMOVE);
 }
 
 void ExtensionPrefs::ClearDisableReasons(const std::string& extension_id) {
-  ModifyDisableReason(
-      extension_id, Extension::DISABLE_NONE, DISABLE_REASON_CLEAR);
+  ModifyDisableReasons(extension_id, Extension::DISABLE_NONE,
+                       DISABLE_REASON_CLEAR);
 }
 
-void ExtensionPrefs::ModifyDisableReason(const std::string& extension_id,
-                                         Extension::DisableReason reason,
-                                         DisableReasonChange change) {
+void ExtensionPrefs::ModifyDisableReasons(const std::string& extension_id,
+                                          int reasons,
+                                          DisableReasonChange change) {
   int old_value = GetDisableReasons(extension_id);
   int new_value = old_value;
   switch (change) {
     case DISABLE_REASON_ADD:
-      new_value |= static_cast<int>(reason);
+      new_value |= reasons;
       break;
     case DISABLE_REASON_REMOVE:
-      new_value &= ~static_cast<int>(reason);
+      new_value &= ~reasons;
       break;
     case DISABLE_REASON_CLEAR:
       new_value = Extension::DISABLE_NONE;