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

Unified Diff: sync/protocol/client_debug_info.proto

Issue 11416126: Track merged nudge sources (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix type ID Created 8 years, 1 month 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/client_debug_info.proto
diff --git a/sync/protocol/client_debug_info.proto b/sync/protocol/client_debug_info.proto
index 7be62d5729a917c65b6afd1261323db51bd17839..b36e32006cddff192a9a2dda862a179966691a7e 100644
--- a/sync/protocol/client_debug_info.proto
+++ b/sync/protocol/client_debug_info.proto
@@ -13,6 +13,18 @@ package sync_pb;
import "get_updates_caller_info.proto";
+// Information about hints sent with nudges.
+message TypeHint {
+ optional int32 data_type_id = 1;
+ optional string notification_hint = 2;
Raz Mathias 2012/11/27 21:52:18 To your question above, a bool would be better --
+}
+
+// Information about the source that triggered a sync.
+message SourceInfo {
+ optional GetUpdatesCallerInfo.GetUpdatesSource source = 1;
+ repeated TypeHint type_hint = 2;
+}
+
// The additional info here is from the StatusController. They get sent when
// the event SYNC_CYCLE_COMPLETED is sent.
message SyncCycleCompletedEventInfo {
@@ -38,6 +50,8 @@ message SyncCycleCompletedEventInfo {
optional int32 num_updates_downloaded = 8;
optional int32 num_reflected_updates_downloaded = 9;
optional GetUpdatesCallerInfo caller_info = 10;
+
+ repeated SourceInfo merged_source_info = 11;
}
// Datatype specifics statistics gathered at association time.

Powered by Google App Engine
This is Rietveld 408576698