Cast channel logging: log additional statistics.

extensions/browser/api/cast_channel/logging.proto

 // Copyright 2014 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.
 
 syntax = "proto2";
 
 option optimize_for = LITE_RUNTIME;
 
 package extensions.core_api.cast_channel.proto;
 
@@ skipping 113 matching lines @@
   optional ErrorState error_state = 10;
 
   optional ChallengeReplyErrorType challenge_reply_error_type = 11;
 }
 
 message AggregatedSocketEvent {
   optional int32 id = 1;
   optional int32 endpoint_id = 2;
   optional ChannelAuth channel_auth_type = 3;
   repeated SocketEvent socket_event = 4;
+  optional int64 bytes_read = 5;
+  optional int64 bytes_written = 6;
 }
 
 message Log {
   // Each AggregatedSocketEvent represents events recorded for a socket.
   repeated AggregatedSocketEvent aggregated_socket_event = 1;
 
   // Number of socket log entries evicted by the logger due to size constraints.
   optional int32 num_evicted_aggregated_socket_events = 2;
 
   // Number of event log entries evicted by the logger due to size constraints.
   optional int32 num_evicted_socket_events = 3;
 
-  // TODO(imcheng): Add other stats, such as bytes read/written across sockets.
+  optional int64 total_bytes_read = 4;

[mark a. foltz, 2014/08/13 22:06:31]

Is this data interesting?  It will be aggregated across multiple sockets which
might map onto multiple devices and include sockets that were evicted from
aggregated_socket_event.

I would be okay leaving this out to keep things simpler; it's more interesting
to keep track on a socket by socket basis.

[imcheng, 2014/08/14 00:24:34]

Yes, this was the intention.
Okay. I can remove this for now.

+  optional int64 total_bytes_written = 5;
 }