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

Unified Diff: chrome/browser/sync/engine/syncer_thread.cc

Issue 6104003: sync: use progress markers instead of timestamps during GetUpdates (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Tim's fixes Created 9 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/engine/syncer_thread.cc
diff --git a/chrome/browser/sync/engine/syncer_thread.cc b/chrome/browser/sync/engine/syncer_thread.cc
index 3c0cd066be172084230bf1c92e6748ba5edf8a2f..7d9fa170ebbafc26542c45765cf11c3714aee80c 100644
--- a/chrome/browser/sync/engine/syncer_thread.cc
+++ b/chrome/browser/sync/engine/syncer_thread.cc
@@ -472,8 +472,8 @@ SyncerThread::WaitInterval SyncerThread::CalculatePollingWaitTime(
// Determine if the syncer has unfinished work to do.
SyncSessionSnapshot* snapshot = session_context_->previous_session_snapshot();
const bool syncer_has_work_to_do = snapshot &&
- (snapshot->num_server_changes_remaining > snapshot->max_local_timestamp ||
- snapshot->unsynced_count > 0);
+ (snapshot->num_server_changes_remaining > 0 ||
+ snapshot->unsynced_count > 0);
VLOG(1) << "syncer_has_work_to_do is " << syncer_has_work_to_do;
// First calculate the expected wait time, figuring in any backoff because of
« no previous file with comments | « chrome/browser/sync/engine/syncer_proto_util.cc ('k') | chrome/browser/sync/engine/syncer_thread_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698