sync: Add pre-commit update avoidance mode + flag

sync/sessions/data_type_tracker.cc

Index: sync/sessions/data_type_tracker.cc
diff --git a/sync/sessions/data_type_tracker.cc b/sync/sessions/data_type_tracker.cc
index 3eabbbe758f959f1811167649bdabd01c44f900a..1c2626eac002fe658675dc55855a6b02eb60261c 100644
--- a/sync/sessions/data_type_tracker.cc
+++ b/sync/sessions/data_type_tracker.cc
@@ -59,7 +59,17 @@ bool DataTypeTracker::IsSyncRequired() const {
   return !IsThrottled() &&
       (local_nudge_count_ > 0 ||
        local_refresh_request_count_ > 0 ||
-       !pending_payloads_.empty());
+       !pending_payloads_.empty() ||
+       local_payload_overflow_ ||
+       server_payload_overflow_);
+}
+
+bool DataTypeTracker::IsGetUpdatesRequired() const {
+  return !IsThrottled() &&
+      (local_refresh_request_count_ > 0 ||
+       !pending_payloads_.empty() ||

[tim (not reviewing), 2013/07/19 20:11:36]

HasPendingInvalidation() would actually be more readable here.  That or rename
pending_payloads to be pending_invalidation_payloads... although I still think
HasPendingInvalidation wins.

[rlarocque, 2013/07/19 20:35:46]

Done.

+       local_payload_overflow_ ||
+       server_payload_overflow_);
 }
 
 bool DataTypeTracker::HasLocalChangePending() const {