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

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

Issue 10958007: sync: Refactor CreateRoot() test helper function (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Found more stuff to clean up Created 8 years, 3 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/test_profile_sync_service.cc
diff --git a/chrome/browser/sync/test_profile_sync_service.cc b/chrome/browser/sync/test_profile_sync_service.cc
index c27241769e0e0b66b4e260b4e5441a989b8aa182..dd88ee9e94cb714e283a14cb2482fa49cfac43fc 100644
--- a/chrome/browser/sync/test_profile_sync_service.cc
+++ b/chrome/browser/sync/test_profile_sync_service.cc
@@ -5,13 +5,13 @@
#include "chrome/browser/sync/test_profile_sync_service.h"
#include "chrome/browser/signin/signin_manager.h"
-#include "chrome/browser/sync/abstract_profile_sync_service_test.h"
#include "chrome/browser/sync/glue/data_type_controller.h"
#include "chrome/browser/sync/glue/sync_backend_host.h"
#include "chrome/browser/sync/profile_sync_components_factory.h"
#include "chrome/browser/sync/test/test_http_bridge_factory.h"
#include "chrome/common/chrome_notification_types.h"
#include "sync/internal_api/public/sessions/sync_session_snapshot.h"
+#include "sync/internal_api/public/test/test_user_share.h"
#include "sync/internal_api/public/user_share.h"
#include "sync/js/js_reply_handler.h"
#include "sync/protocol/encryption.pb.h"
@@ -25,6 +25,7 @@ using syncer::sessions::SyncSessionSnapshot;
using syncer::sessions::SyncSourceInfo;
using syncer::UserShare;
using syncer::syncable::Directory;
+using syncer::NIGORI;
namespace browser_sync {
@@ -118,9 +119,8 @@ void SyncBackendHostForProfileSyncTest
UserShare* user_share = GetUserShare();
Directory* directory = user_share->directory.get();
- if (!directory->initial_sync_ended_for_type(syncer::NIGORI)) {
- ProfileSyncServiceTestHelper::CreateRoot(
- syncer::NIGORI, user_share, &id_factory_);
+ if (!directory->initial_sync_ended_for_type(NIGORI)) {
+ syncer::TestUserShare::CreateRoot(NIGORI, user_share);
// A side effect of adding the NIGORI mode (normally done by the
// syncer) is a decryption attempt, which will fail the first time.
« no previous file with comments | « chrome/browser/sync/profile_sync_service_typed_url_unittest.cc ('k') | sync/internal_api/public/test/test_user_share.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698