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

Unified Diff: sync/sessions/test_util.cc

Issue 11314008: sync: Follow-up to conflict resolution refactor (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 years, 2 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: sync/sessions/test_util.cc
diff --git a/sync/sessions/test_util.cc b/sync/sessions/test_util.cc
index 4214c3d44bd16afffce64550ddede6a4e4ff3e14..1eb67b79274cfc8c41d985c6be46e0075e75f90c 100644
--- a/sync/sessions/test_util.cc
+++ b/sync/sessions/test_util.cc
@@ -8,12 +8,6 @@ namespace syncer {
namespace sessions {
namespace test_util {
-void SimulateHasMoreToSync(sessions::SyncSession* session,
- SyncerStep begin, SyncerStep end) {
- session->mutable_status_controller()->update_conflicts_resolved(true);
- ASSERT_TRUE(session->HasMoreToSync());
-}
-
void SimulateGetEncryptionKeyFailed(sessions::SyncSession* session,
SyncerStep begin, SyncerStep end) {
session->mutable_status_controller()->set_last_get_key_result(
@@ -43,9 +37,6 @@ void SimulateConnectionFailure(sessions::SyncSession* session,
void SimulateSuccess(sessions::SyncSession* session,
SyncerStep begin, SyncerStep end) {
- if (session->HasMoreToSync()) {
- ADD_FAILURE() << "Shouldn't have more to sync";
- }
ASSERT_EQ(0U, session->status_controller().num_server_changes_remaining());
session->SetFinished();
switch(end) {

Powered by Google App Engine
This is Rietveld 408576698