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

Unified Diff: chrome/browser/ui/cocoa/cocoa_profile_test.mm

Issue 276063003: Remove a call to initialize the TemplateURLService factory in this test. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Some more adjustments. Created 6 years, 7 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/ui/cocoa/cocoa_profile_test.mm
diff --git a/chrome/browser/ui/cocoa/cocoa_profile_test.mm b/chrome/browser/ui/cocoa/cocoa_profile_test.mm
index bc634ca1f7b09030d08ce2c2446f77b84e6b4445..99c38b78a09e2c38bb4d3cafffbf4bd18c6f32f0 100644
--- a/chrome/browser/ui/cocoa/cocoa_profile_test.mm
+++ b/chrome/browser/ui/cocoa/cocoa_profile_test.mm
@@ -49,10 +49,6 @@ void CocoaProfileTest::SetUp() {
profile_ = profile_manager_.CreateTestingProfile("default");
ASSERT_TRUE(profile_);
- profile_->CreateBookmarkModel(true);
- test::WaitForBookmarkModelToLoad(
- BookmarkModelFactory::GetForProfile(profile_));
-
// TODO(shess): These are needed in case someone creates a browser
// window off of browser_. pkasting indicates that other
// platforms use a stub |BrowserWindow| and thus don't need to do
@@ -63,6 +59,10 @@ void CocoaProfileTest::SetUp() {
AutocompleteClassifierFactory::GetInstance()->SetTestingFactoryAndUse(
profile_, &AutocompleteClassifierFactory::BuildInstanceFor);
+ profile_->CreateBookmarkModel(true);
+ test::WaitForBookmarkModelToLoad(
+ BookmarkModelFactory::GetForProfile(profile_));
+
browser_.reset(CreateBrowser());
ASSERT_TRUE(browser_.get());
}

Powered by Google App Engine
This is Rietveld 408576698