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

Unified Diff: chrome/browser/sync/profile_sync_service_session_unittest.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/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 33790cfb5dc43e8ef729dbb2de0fb374e94c3492..d31df7707cc825d1aaca115a743e57689546ac23 100644
--- a/chrome/browser/sync/profile_sync_service_session_unittest.cc
+++ b/chrome/browser/sync/profile_sync_service_session_unittest.cc
@@ -41,11 +41,11 @@
#include "sync/internal_api/public/change_record.h"
#include "sync/internal_api/public/read_node.h"
#include "sync/internal_api/public/read_transaction.h"
+#include "sync/internal_api/public/test/test_user_share.h"
#include "sync/internal_api/public/write_node.h"
#include "sync/internal_api/public/write_transaction.h"
#include "sync/protocol/session_specifics.pb.h"
#include "sync/protocol/sync.pb.h"
-#include "sync/test/engine/test_id_factory.h"
#include "testing/gmock/include/gmock/gmock.h"
#include "testing/gtest/include/gtest/gtest.h"
#include "ui/base/ui_base_types.h"
@@ -160,8 +160,6 @@ class ProfileSyncServiceSessionTest
notified_of_refresh_(false) {}
ProfileSyncService* sync_service() { return sync_service_.get(); }
- TestIdFactory* ids() { return sync_service_->id_factory(); }
-
protected:
virtual TestingProfile* CreateProfile() OVERRIDE {
TestingProfile* profile = new TestingProfile();
@@ -287,8 +285,8 @@ class CreateRootHelper {
private:
void CreateRootCallback(ProfileSyncServiceSessionTest* test) {
- success_ = ProfileSyncServiceTestHelper::CreateRoot(
- syncer::SESSIONS, test->sync_service()->GetUserShare(), test->ids());
+ success_ = syncer::TestUserShare::CreateRoot(
+ syncer::SESSIONS, test->sync_service()->GetUserShare());
}
base::Closure callback_;

Powered by Google App Engine
This is Rietveld 408576698