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

Unified Diff: sync/protocol/sync.proto

Issue 387983004: sync: Support non-blocking initial sync in proto (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebase + fix comment Created 6 years, 5 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/protocol/sync.proto
diff --git a/sync/protocol/sync.proto b/sync/protocol/sync.proto
index 276824272fe2f7835fa37389243dc885f5b99907..c66e25e62cd61d106b42678d5e3d18259112967c 100644
--- a/sync/protocol/sync.proto
+++ b/sync/protocol/sync.proto
@@ -477,6 +477,17 @@ message GetUpdateTriggers {
// This flag is set if the invalidation server reports that it may have
// dropped some invalidations at some point. Introduced in M33.
optional bool server_dropped_hints = 6;
+
+ // This flag is set if this GetUpdate request is due at least in part due
+ // to the fact that this type has not finished initial sync yet, and the
+ // client would like to initialize itself with the server data.
+ //
+ // Only some types support performing an initial sync as part of a normal
+ // GetUpdate request. Many types must be in configure mode when fetching
+ // initial sync data.
+ //
+ // Introduced in M38.
+ optional bool initial_sync_in_progress = 7;
Nicolas Zea 2014/07/17 16:58:08 I wonder, should we specify which types are reques
rlarocque 2014/07/17 18:58:06 The triggers are per-type. I think they're co-loc
}
message GarbageCollectionDirective {

Powered by Google App Engine
This is Rietveld 408576698