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

Unified Diff: chrome/browser/extensions/extension_service_unittest.h

Issue 77773002: Ensure the ExtensionSystem is ready before initializing an ExtensionAppModelBuilder (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: fix tests Created 7 years, 1 month 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_service_unittest.h
diff --git a/chrome/browser/extensions/extension_service_unittest.h b/chrome/browser/extensions/extension_service_unittest.h
index 9ca5d22ce645cd10ecd61c27420e61e6fa309a13..8ca688595149d335c7b1e68efe85556b7892c025 100644
--- a/chrome/browser/extensions/extension_service_unittest.h
+++ b/chrome/browser/extensions/extension_service_unittest.h
@@ -46,6 +46,11 @@ class ExtensionServiceTestBase : public testing::Test {
void InitializeExtensionService(const ExtensionServiceInitParams& params);
+ static void InitializeExtensionServiceForProfile(
tapted 2013/11/20 11:03:25 should return ExtensionService*
benwells 2013/11/21 00:47:47 yes :)
tapted 2013/11/21 03:29:56 Done.
+ const ExtensionServiceInitParams& params,
+ scoped_ptr<TestingProfile>* profile_ptr,
+ ExtensionService** service_ptr);
+
void InitializeInstalledExtensionService(
const base::FilePath& prefs_file,
const base::FilePath& source_install_dir);
@@ -71,6 +76,8 @@ class ExtensionServiceTestBase : public testing::Test {
protected:
ExtensionServiceInitParams CreateDefaultInitParams();
+ static ExtensionServiceInitParams CreateDefaultInitParamsInTempDir(
+ base::ScopedTempDir* temp_dir_);
tapted 2013/11/20 11:03:25 temp_dir_ shouldn't have a trailing underscore
tapted 2013/11/21 03:29:56 Done.
// Destroying at_exit_manager_ will delete all LazyInstances, so it must come
// after thread_bundle_ in the destruction order.

Powered by Google App Engine
This is Rietveld 408576698