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

Unified Diff: chrome/browser/sync/profile_sync_service_session_unittest.cc

Issue 9232011: sync: Make ProfileSyncService a ProfileKeyedService. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebase Created 8 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/sync/profile_sync_service_session_unittest.cc
diff --git a/chrome/browser/sync/profile_sync_service_session_unittest.cc b/chrome/browser/sync/profile_sync_service_session_unittest.cc
index 15ec731d98158a0bc81f36012f99586d839d4ecd..7c2c3a02c0f9616bdf54b418d67e2f6adbc886b3 100644
--- a/chrome/browser/sync/profile_sync_service_session_unittest.cc
+++ b/chrome/browser/sync/profile_sync_service_session_unittest.cc
@@ -1,4 +1,4 @@
-// Copyright (c) 2011 The Chromium Authors. All rights reserved.
+// 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.
@@ -226,8 +226,10 @@ class ProfileSyncServiceSessionTest
return false;
SigninManager* signin = new SigninManager();
signin->SetAuthenticatedUsername("test_user");
+ ProfileSyncComponentsFactoryMock* factory =
+ new ProfileSyncComponentsFactoryMock();
sync_service_.reset(new TestProfileSyncService(
- &factory_,
+ factory,
profile(),
signin,
ProfileSyncService::AUTO_START,
@@ -242,13 +244,13 @@ class ProfileSyncServiceSessionTest
change_processor_ = new SessionChangeProcessor(
sync_service_.get(), model_associator_,
true /* setup_for_test */);
- EXPECT_CALL(factory_, CreateSessionSyncComponents(_, _)).
+ EXPECT_CALL(*factory, CreateSessionSyncComponents(_, _)).
WillOnce(Return(ProfileSyncComponentsFactory::SyncComponents(
model_associator_, change_processor_)));
- EXPECT_CALL(factory_, CreateDataTypeManager(_, _)).
+ EXPECT_CALL(*factory, CreateDataTypeManager(_, _)).
WillOnce(ReturnNewDataTypeManager());
sync_service_->RegisterDataTypeController(
- new SessionDataTypeController(&factory_,
+ new SessionDataTypeController(factory,
profile(),
sync_service_.get()));
profile()->GetTokenService()->IssueAuthTokenForTest(
@@ -265,7 +267,6 @@ class ProfileSyncServiceSessionTest
SessionModelAssociator* model_associator_;
SessionChangeProcessor* change_processor_;
SessionID window_id_;
- ProfileSyncComponentsFactoryMock factory_;
scoped_ptr<TestProfileSyncService> sync_service_;
const gfx::Rect window_bounds_;
bool notified_of_update_;

Powered by Google App Engine
This is Rietveld 408576698