Index: components/sync/engine_impl/worker_entity_tracker.cc |
diff --git a/components/sync/engine_impl/worker_entity_tracker.cc b/components/sync/engine_impl/worker_entity_tracker.cc |
index 2e0ca7bfb65e0c6e293915302b4d1becbc147779..1ad0285125db90686cd7c11b4b75d313272bcc24 100644 |
--- a/components/sync/engine_impl/worker_entity_tracker.cc |
+++ b/components/sync/engine_impl/worker_entity_tracker.cc |
@@ -127,7 +127,7 @@ void WorkerEntityTracker::ReceiveCommitResponse(CommitResponseData* ack) { |
} |
void WorkerEntityTracker::ReceiveUpdate(const UpdateResponseData& update) { |
- if (update.response_version <= highest_gu_response_version_) |
+ if (!UpdateContainsNewVersion(update)) |
return; |
highest_gu_response_version_ = update.response_version; |
@@ -144,6 +144,13 @@ void WorkerEntityTracker::ReceiveUpdate(const UpdateResponseData& update) { |
} |
} |
+bool WorkerEntityTracker::UpdateContainsNewVersion( |
+ const UpdateResponseData& update) { |
+ if (update.response_version <= highest_gu_response_version_) |
pavely
2016/10/19 18:18:59
It is easier to read if condition follows name/sem
Gang Wu
2016/10/20 21:35:36
Done.
|
+ return false; |
+ return true; |
+} |
+ |
bool WorkerEntityTracker::ReceiveEncryptedUpdate( |
const UpdateResponseData& data) { |
if (data.response_version < highest_gu_response_version_) |