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

Unified Diff: chrome/browser/extensions/extension_startup_browsertest.cc

Issue 12494028: Move Requirements out of Extension Class (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Created 7 years, 9 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/extension_startup_browsertest.cc
diff --git a/chrome/browser/extensions/extension_startup_browsertest.cc b/chrome/browser/extensions/extension_startup_browsertest.cc
index 83fb022b2d7b50512d9bf97c31fb218fef82abb8..c01ed32c7845477561c640680fd3ae3b96562176 100644
--- a/chrome/browser/extensions/extension_startup_browsertest.cc
+++ b/chrome/browser/extensions/extension_startup_browsertest.cc
@@ -18,7 +18,9 @@
#include "chrome/common/chrome_notification_types.h"
#include "chrome/common/chrome_paths.h"
#include "chrome/common/chrome_switches.h"
+#include "chrome/common/extensions/api/requirements/requirements_handler.h"
#include "chrome/common/extensions/feature_switch.h"
+#include "chrome/common/extensions/manifest_handler.h"
#include "chrome/test/base/in_process_browser_test.h"
#include "chrome/test/base/testing_profile.h"
#include "chrome/test/base/ui_test_utils.h"
@@ -28,12 +30,12 @@
#include "content/public/test/browser_test_utils.h"
#include "net/base/net_util.h"
-using extensions::FeatureSwitch;
-
// This file contains high-level startup tests for the extensions system. We've
// had many silly bugs where command line flags did not get propagated correctly
// into the services, so we didn't start correctly.
+namespace extensions {
+
class ExtensionStartupTestBase : public InProcessBrowserTest {
public:
ExtensionStartupTestBase() :
@@ -45,6 +47,8 @@ class ExtensionStartupTestBase : public InProcessBrowserTest {
protected:
// InProcessBrowserTest
virtual void SetUpCommandLine(CommandLine* command_line) OVERRIDE {
+ InProcessBrowserTest::SetUpCommandLine(command_line);
+
if (!enable_extensions_)
command_line->AppendSwitch(switches::kDisableExtensions);
@@ -54,6 +58,13 @@ class ExtensionStartupTestBase : public InProcessBrowserTest {
paths);
command_line->AppendSwitch(switches::kDisableExtensionsFileAccessCheck);
}
+
+ (new RequirementsHandler)->Register();
+ }
+
+ virtual void CleanUpOnMainThread() OVERRIDE {
+ ManifestHandler::ClearRegistryForTesting();
+ InProcessBrowserTest::CleanUpOnMainThread();
}
virtual bool SetUpUserDataDirectory() OVERRIDE {
@@ -91,14 +102,14 @@ class ExtensionStartupTestBase : public InProcessBrowserTest {
void WaitForServicesToStart(int num_expected_extensions,
bool expect_extensions_enabled) {
- ExtensionService* service = extensions::ExtensionSystem::Get(
+ ExtensionService* service = ExtensionSystem::Get(
browser()->profile())->extension_service();
// Count the number of non-component extensions.
int found_extensions = 0;
for (ExtensionSet::const_iterator it = service->extensions()->begin();
it != service->extensions()->end(); ++it)
- if ((*it)->location() != extensions::Manifest::COMPONENT)
+ if ((*it)->location() != Manifest::COMPONENT)
found_extensions++;
ASSERT_EQ(static_cast<uint32>(num_expected_extensions),
@@ -108,8 +119,7 @@ class ExtensionStartupTestBase : public InProcessBrowserTest {
content::WindowedNotificationObserver user_scripts_observer(
chrome::NOTIFICATION_USER_SCRIPTS_UPDATED,
content::NotificationService::AllSources());
- extensions::UserScriptMaster* master =
- extensions::ExtensionSystem::Get(browser()->profile())->
+ UserScriptMaster* master = ExtensionSystem::Get(browser()->profile())->
user_script_master();
if (!master->ScriptsReady())
user_scripts_observer.Wait();
@@ -185,13 +195,13 @@ IN_PROC_BROWSER_TEST_F(ExtensionsStartupTest, MAYBE_NoFileAccess) {
// Keep a separate list of extensions for which to disable file access, since
// doing so reloads them.
- std::vector<const extensions::Extension*> extension_list;
+ std::vector<const Extension*> extension_list;
- ExtensionService* service = extensions::ExtensionSystem::Get(
+ ExtensionService* service = ExtensionSystem::Get(
browser()->profile())->extension_service();
for (ExtensionSet::const_iterator it = service->extensions()->begin();
it != service->extensions()->end(); ++it) {
- if ((*it)->location() == extensions::Manifest::COMPONENT)
+ if ((*it)->location() == Manifest::COMPONENT)
continue;
if (service->AllowFileAccess(*it))
extension_list.push_back(*it);
@@ -280,3 +290,5 @@ IN_PROC_BROWSER_TEST_F(ExtensionsLoadMultipleTest, Test) {
WaitForServicesToStart(4, true);
TestInjection(true, true);
}
+
+} // namespace extensions

Powered by Google App Engine
This is Rietveld 408576698