sync: Count and report reflected updates

sync/protocol/client_debug_info.proto

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 //
 // Common sync protocol for encrypted data.
 
 syntax = "proto2";
 
 option optimize_for = LITE_RUNTIME;
 option retain_unknown_fields = true;
 
 package sync_pb;
 
+import "get_updates_caller_info.proto";
+
 // The additional info here is from SyncerStatus. They get sent when the event
 // SYNC_CYCLE_COMPLETED  is sent.
 message SyncCycleCompletedEventInfo {
   // optional bool syncer_stuck = 1; // Was always false, now obsolete.
 
   // The client has never set these values correctly.  It set
   // num_blocking_conflicts to the total number of conflicts detected and set
   // num_non_blocking_conflicts to the number of blocking (aka. simple)
   // conflicts.
   //
   // These counters have been deprecated to avoid further confusion.  The newer
   // counters provide more detail and are less buggy.
   optional int32 num_blocking_conflicts = 2 [deprecated = true];
   optional int32 num_non_blocking_conflicts = 3 [deprecated = true];
 
   // These new conflict counters replace the ones above.
   optional int32 num_encryption_conflicts = 4;
   optional int32 num_hierarchy_conflicts = 5;
   optional int32 num_simple_conflicts = 6;
   optional int32 num_server_conflicts = 7;
+
+  // Counts to track the effective usefulness of our GetUpdate requests.
+  optional int32 num_updates_downloaded = 8;
+  optional int32 num_echo_updates_downloaded = 9;
 }
 
 message DebugEventInfo {
   // These events are sent by |SyncManager| class. Note: In the code they each
   // of these events have some additional info but we are not sending them to
   // server.
   enum EventType {
     CONNECTION_STATUS_CHANGE = 1; // Connection status change. Note this
                                   // gets generated even during a successful
                                   // connection.
@@ skipping 22 matching lines @@
   // Whether cryptographer is ready to encrypt and decrypt data.
   optional bool cryptographer_ready = 2;
 
   // Cryptographer has pending keys which indicates the correct passphrase
   // has not been provided yet.
   optional bool cryptographer_has_pending_keys = 3;
 
   // Indicates client has dropped some events to save bandwidth.
   optional bool events_dropped = 4;
 }