Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(6733)

Unified Diff: chrome/browser/extensions/api/management/management_browsertest.cc

Issue 12093036: Move Extension Location and Type enums to Manifest, and move InstallWarning to its own file. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/extensions/api/management/management_browsertest.cc
diff --git a/chrome/browser/extensions/api/management/management_browsertest.cc b/chrome/browser/extensions/api/management/management_browsertest.cc
index 24641537d14a40508ee14de279f7c1bd24e8c9c6..e0a870be1a78fcbe232184d3c3d2e6384dd77888 100644
--- a/chrome/browser/extensions/api/management/management_browsertest.cc
+++ b/chrome/browser/extensions/api/management/management_browsertest.cc
@@ -29,6 +29,7 @@
#include "net/url_request/url_fetcher.h"
using extensions::Extension;
+using extensions::Manifest;
class ExtensionManagementTest : public ExtensionBrowserTest {
protected:
@@ -433,7 +434,7 @@ IN_PROC_BROWSER_TEST_F(ExtensionManagementTest, MAYBE_ExternalUrlUpdate) {
EXPECT_TRUE(pending_extension_manager->AddFromExternalUpdateUrl(
kExtensionId, GURL("http://localhost/autoupdate/manifest"),
- Extension::EXTERNAL_PREF_DOWNLOAD));
+ Manifest::EXTERNAL_PREF_DOWNLOAD));
// Run autoupdate and make sure version 2 of the extension was installed.
service->updater()->CheckNow(params);
@@ -456,7 +457,7 @@ IN_PROC_BROWSER_TEST_F(ExtensionManagementTest, MAYBE_ExternalUrlUpdate) {
// because of the killbit.
EXPECT_FALSE(pending_extension_manager->AddFromExternalUpdateUrl(
kExtensionId, GURL("http://localhost/autoupdate/manifest"),
- Extension::EXTERNAL_PREF_DOWNLOAD));
+ Manifest::EXTERNAL_PREF_DOWNLOAD));
EXPECT_FALSE(pending_extension_manager->IsIdPending(kExtensionId))
<< "External reinstall of a killed extension shouldn't work.";
EXPECT_TRUE(extension_prefs->IsExternalExtensionUninstalled(kExtensionId))
@@ -528,7 +529,7 @@ IN_PROC_BROWSER_TEST_F(ExtensionManagementTest, ExternalPolicyRefresh) {
const Extension* extension = service->GetExtensionById(kExtensionId, false);
ASSERT_TRUE(extension);
ASSERT_EQ("2.0", extension->VersionString());
- EXPECT_EQ(Extension::EXTERNAL_POLICY_DOWNLOAD, extension->location());
+ EXPECT_EQ(Manifest::EXTERNAL_POLICY_DOWNLOAD, extension->location());
// Try to disable and uninstall the extension which should fail.
DisableExtension(kExtensionId);
@@ -592,7 +593,7 @@ IN_PROC_BROWSER_TEST_F(ExtensionManagementTest,
ASSERT_EQ(size_before + 1, service->extensions()->size());
const Extension* extension = service->GetExtensionById(kExtensionId, false);
ASSERT_TRUE(extension);
- EXPECT_EQ(Extension::INTERNAL, extension->location());
+ EXPECT_EQ(Manifest::INTERNAL, extension->location());
EXPECT_TRUE(service->IsExtensionEnabled(kExtensionId));
// Setup the force install policy. It should override the location.
@@ -606,7 +607,7 @@ IN_PROC_BROWSER_TEST_F(ExtensionManagementTest,
ASSERT_EQ(size_before + 1, service->extensions()->size());
extension = service->GetExtensionById(kExtensionId, false);
ASSERT_TRUE(extension);
- EXPECT_EQ(Extension::EXTERNAL_POLICY_DOWNLOAD, extension->location());
+ EXPECT_EQ(Manifest::EXTERNAL_POLICY_DOWNLOAD, extension->location());
EXPECT_TRUE(service->IsExtensionEnabled(kExtensionId));
// Remove the policy, and verify that the extension was uninstalled.
@@ -623,7 +624,7 @@ IN_PROC_BROWSER_TEST_F(ExtensionManagementTest,
ASSERT_EQ(size_before + 1, service->extensions()->size());
extension = service->GetExtensionById(kExtensionId, false);
ASSERT_TRUE(extension);
- EXPECT_EQ(Extension::INTERNAL, extension->location());
+ EXPECT_EQ(Manifest::INTERNAL, extension->location());
EXPECT_TRUE(service->IsExtensionEnabled(kExtensionId));
EXPECT_TRUE(service->disabled_extensions()->is_empty());
@@ -645,7 +646,7 @@ IN_PROC_BROWSER_TEST_F(ExtensionManagementTest,
ASSERT_EQ(size_before + 1, service->extensions()->size());
extension = service->GetExtensionById(kExtensionId, false);
ASSERT_TRUE(extension);
- EXPECT_EQ(Extension::EXTERNAL_POLICY_DOWNLOAD, extension->location());
+ EXPECT_EQ(Manifest::EXTERNAL_POLICY_DOWNLOAD, extension->location());
EXPECT_TRUE(service->IsExtensionEnabled(kExtensionId));
EXPECT_TRUE(service->disabled_extensions()->is_empty());
}

Powered by Google App Engine
This is Rietveld 408576698