Add more management policy checking after extension installed

chrome/browser/extensions/extension_disabled_ui_browsertest.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 "base/files/file_path.h"
 #include "base/files/scoped_temp_dir.h"
 #include "base/threading/sequenced_worker_pool.h"
 #include "chrome/app/chrome_command_ids.h"
 #include "chrome/browser/extensions/extension_browsertest.h"
 #include "chrome/browser/extensions/extension_service.h"
@@ skipping 135 matching lines @@
 }
 
 // Tests that no error appears if the user disabled the extension.
 IN_PROC_BROWSER_TEST_F(ExtensionDisabledGlobalErrorTest, UserDisabled) {
   const Extension* extension = InstallIncreasingPermissionExtensionV1();
   DisableExtension(extension->id());
   extension = UpdateIncreasingPermissionExtension(extension, path_v2_, 0);
   ASSERT_FALSE(GetExtensionDisabledGlobalError());
 }
 
-// Test that no error appears if the disable reason is unknown
-// (but probably was by the user).
-IN_PROC_BROWSER_TEST_F(ExtensionDisabledGlobalErrorTest,
-                       UnknownReasonSamePermissions) {
-  const Extension* extension = InstallIncreasingPermissionExtensionV1();
-  DisableExtension(extension->id());
-  // Clear disable reason to simulate legacy disables.
-  ExtensionPrefs::Get(browser()->profile())
-      ->ClearDisableReasons(extension->id());
-  // Upgrade to version 2. Infer from version 1 having the same permissions
-  // granted by the user that it was disabled by the user.
-  extension = UpdateIncreasingPermissionExtension(extension, path_v2_, 0);
-  ASSERT_TRUE(extension);
-  ASSERT_FALSE(GetExtensionDisabledGlobalError());
-}
-
-// Test that an error appears if the disable reason is unknown
-// (but probably was for increased permissions).
-IN_PROC_BROWSER_TEST_F(ExtensionDisabledGlobalErrorTest,
-                       UnknownReasonHigherPermissions) {
-  const Extension* extension = InstallAndUpdateIncreasingPermissionsExtension();
-  // Clear disable reason to simulate legacy disables.
-  ExtensionPrefs::Get(service_->profile())
-      ->ClearDisableReasons(extension->id());
-  // We now have version 2 but only accepted permissions for version 1.
-  GlobalError* error = GetExtensionDisabledGlobalError();
-  ASSERT_TRUE(error);
-  // Also, remove the upgrade error for version 2.
-  GlobalErrorServiceFactory::GetForProfile(browser()->profile())->
-      RemoveGlobalError(error);
-  delete error;
-  // Upgrade to version 3, with even higher permissions. Infer from
-  // version 2 having higher-than-granted permissions that it was disabled
-  // for permissions increase.
-  extension = UpdateIncreasingPermissionExtension(extension, path_v3_, 0);
-  ASSERT_TRUE(extension);
-  ASSERT_TRUE(GetExtensionDisabledGlobalError());
-}
-
 // Test that an error appears if the extension gets disabled because a
 // version with higher permissions was installed by sync.
 IN_PROC_BROWSER_TEST_F(ExtensionDisabledGlobalErrorTest,
                        HigherPermissionsFromSync) {
   // Get data for extension v2 (disabled) into sync.
   const Extension* extension = InstallAndUpdateIncreasingPermissionsExtension();
   std::string extension_id = extension->id();
   ExtensionSyncService* sync_service = ExtensionSyncService::Get(
       browser()->profile());
   extensions::ExtensionSyncData sync_data =
@@ skipping 82 matching lines @@
 
   const Extension* extension = service_->GetExtensionById(extension_id, true);
   ASSERT_TRUE(extension);
   EXPECT_EQ("2", extension->VersionString());
   EXPECT_EQ(1u, registry_->disabled_extensions().size());
   EXPECT_EQ(Extension::DISABLE_REMOTE_INSTALL,
             ExtensionPrefs::Get(service_->profile())
                 ->GetDisableReasons(extension_id));
   EXPECT_TRUE(GetExtensionDisabledGlobalError());
 }