Replace extension disabled infobar with a global error.

Move disabled-extension interface to namespace extensions.

It looks like this:
http://dl.dropbox.com/u/27111995/Mocks/disable-dialog.png
http://dl.dropbox.com/u/27111995/Mocks/disable-menu.png

BUG=108202
TEST=install extension, update to version that requires additional permissions. Accept upgrade. Uninstall manually from chrome://extensions. Don't crash.

Review URL: http://codereview.chromium.org/9718008

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@127887 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 81 insertions, 0 deletions

diff --git a/chrome/browser/extensions/extension_disabled_ui_browsertest.cc b/chrome/browser/extensions/extension_disabled_ui_browsertest.cc
new file mode 100644
index 0000000..d908fa0
--- /dev/null
+++ b/chrome/browser/extensions/extension_disabled_ui_browsertest.cc
@@ -0,0 +1,81 @@
+// 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/app/chrome_command_ids.h"
+#include "chrome/browser/extensions/extension_browsertest.h"
+#include "chrome/browser/extensions/extension_service.h"
+#include "chrome/browser/profiles/profile.h"
+#include "chrome/browser/ui/browser.h"
+#include "chrome/browser/ui/global_error_service.h"
+#include "chrome/browser/ui/global_error_service_factory.h"
+#include "chrome/common/extensions/extension.h"
+
+class ExtensionDisabledGlobalErrorTest : public ExtensionBrowserTest {
+ protected:
+  void SetUpOnMainThread() {
+    service_ = browser()->profile()->GetExtensionService();
+  }
+
+  // Returns the ExtensionDisabledGlobalError, if present.
+  // Caution: currently only supports one error at a time.
+  GlobalError* GetExtensionDisabledGlobalError() {
+    return GlobalErrorServiceFactory::GetForProfile(browser()->profile())->
+        GetGlobalErrorByMenuItemCommandID(IDC_EXTENSION_DISABLED_FIRST);
+  }
+
+  // Helper function to install an extension and upgrade it to a version
+  // requiring additional permissions. Returns the new disabled Extension.
+  const Extension* InstallAndUpdateIncreasingPermissionsExtension() {
+    size_t size_before = service_->extensions()->size();
+
+    // Install the initial version, which should happen just fine.
+    const Extension* extension = InstallExtension(
+        test_data_dir_.AppendASCII("permissions-low-v1.crx"), 1);
+    if (!extension)
+      return NULL;
+    if (service_->extensions()->size() != size_before + 1)
+      return NULL;
+
+    // Upgrade to a version that wants more permissions. We should disable the
+    // extension and prompt the user to reenable.
+    if (UpdateExtension(
+            extension->id(),
+            test_data_dir_.AppendASCII("permissions-high-v2.crx"), -1))
+      return NULL;
+    EXPECT_EQ(size_before, service_->extensions()->size());
+    if (service_->disabled_extensions()->size() != 1u)
+      return NULL;
+
+    return *service_->disabled_extensions()->begin();
+  }
+
+  ExtensionService* service_;
+};
+
+// Tests the process of updating an extension to one that requires higher
+// permissions, and accepting the permissions.
+IN_PROC_BROWSER_TEST_F(ExtensionDisabledGlobalErrorTest, AcceptPermissions) {
+  const Extension* extension = InstallAndUpdateIncreasingPermissionsExtension();
+  ASSERT_TRUE(extension);
+  ASSERT_TRUE(GetExtensionDisabledGlobalError());
+  const size_t size_before = service_->extensions()->size();
+
+  service_->GrantPermissionsAndEnableExtension(extension);
+  EXPECT_EQ(size_before + 1, service_->extensions()->size());
+  EXPECT_EQ(0u, service_->disabled_extensions()->size());
+  ASSERT_FALSE(GetExtensionDisabledGlobalError());
+}
+
+// Tests uninstalling an extension that was disabled due to higher permissions.
+IN_PROC_BROWSER_TEST_F(ExtensionDisabledGlobalErrorTest, Uninstall) {
+  const Extension* extension = InstallAndUpdateIncreasingPermissionsExtension();
+  ASSERT_TRUE(extension);
+  ASSERT_TRUE(GetExtensionDisabledGlobalError());
+  const size_t size_before = service_->extensions()->size();
+
+  UninstallExtension(extension->id());
+  EXPECT_EQ(size_before, service_->extensions()->size());
+  EXPECT_EQ(0u, service_->disabled_extensions()->size());
+  ASSERT_FALSE(GetExtensionDisabledGlobalError());
+}