Add a timestamp field to QUIC's CachedNetworkParams proto message.

net/quic/crypto/source_address_token.h

Index: net/quic/crypto/source_address_token.h
diff --git a/net/quic/crypto/source_address_token.h b/net/quic/crypto/source_address_token.h
index 64a3a12817284931f7e0d293b424b4d42b03934f..110135112d7a34f812bcb86e35be86496bed38e1 100644
--- a/net/quic/crypto/source_address_token.h
+++ b/net/quic/crypto/source_address_token.h
@@ -74,6 +74,9 @@ class NET_EXPORT_PRIVATE CachedNetworkParameters {
     previous_connection_state_ = previous_connection_state;
   }
 
+  int64 timestamp() const { return timestamp_; }
+  void set_timestamp(int64 timestamp) { timestamp_ = timestamp; }
+
  private:
   // serving_region_ is used to decide whether or not the bandwidth estimate and
   // min RTT are reasonable and if they should be used.
@@ -94,6 +97,8 @@ class NET_EXPORT_PRIVATE CachedNetworkParameters {
   int32 min_rtt_ms_;
   // Encodes the PreviousConnectionState enum.
   int32 previous_connection_state_;
+  // UNIX timestamp when this bandwidth estimate was created.
+  int64 timestamp_;

[wtc, 2014/09/29 17:15:30]

Nit: it seems that the naming convention includes the time unit ("_seconds" or
"_ms") in the member's name. See max_bandwidth_timestamp_seconds_ and
min_rtt_ms_.

[ramant (doing other things), 2014/09/29 17:42:09]

Hi Wan-Teh,
  Robbie pointed out it is similar to "timestamp_" field in SourceAddressToken.
Will rename both in a future CL.

 };
 
 // TODO(rtenneti): sync with server more rationally.