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

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

Issue 367153005: Sync: Refactoring of DEVICE_INFO syncable type - Part 1 (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fixed scoped_ptr issue in ProfileSyncService constructor Created 6 years, 5 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_unittest.cc
diff --git a/chrome/browser/sync/profile_sync_service_unittest.cc b/chrome/browser/sync/profile_sync_service_unittest.cc
index f62539973a7abf10b8385028c0fbf6f45b3fb58c..83ded5840e1824089df983dc55622ca2c5e9ff6c 100644
--- a/chrome/browser/sync/profile_sync_service_unittest.cc
+++ b/chrome/browser/sync/profile_sync_service_unittest.cc
@@ -193,9 +193,9 @@ class ProfileSyncServiceTest : public ::testing::Test {
signin->SetAuthenticatedUsername("test");
ProfileOAuth2TokenService* oauth2_token_service =
ProfileOAuth2TokenServiceFactory::GetForProfile(profile_);
- components_factory_ = new StrictMock<ProfileSyncComponentsFactoryMock>();
+ components_factory_ = new ProfileSyncComponentsFactoryMock();
service_.reset(new ProfileSyncService(
- components_factory_,
+ scoped_ptr<ProfileSyncComponentsFactory>(components_factory_),
profile_,
make_scoped_ptr(new SupervisedUserSigninManagerWrapper(profile_,
signin)),

Powered by Google App Engine
This is Rietveld 408576698