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

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

Context in b/17357338, follow-up CL will store CachedNetworkParams and
copy into newly created STKs.

Merge internal change: 75897792

R=rch@chromium.org

[Ryan Hamilton, 2014-09-26 03:20:12]

lgtm

[wtc, 2014-09-29 17:15:29]

wtc@chromium.org changed reviewers:
+ wtc@chromium.org

[wtc, 2014-09-29 17:15:31]

Patch set 1 LGTM.

https://codereview.chromium.org/604173002/diff/1/net/quic/crypto/source_addre...
File net/quic/crypto/source_address_token.h (right):

https://codereview.chromium.org/604173002/diff/1/net/quic/crypto/source_addre...
net/quic/crypto/source_address_token.h:101: int64 timestamp_;

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:10]

Thanks much Wan-Teh. Will rename it in the next CL.

https://codereview.chromium.org/604173002/diff/1/net/quic/crypto/source_addre...
File net/quic/crypto/source_address_token.h (right):

https://codereview.chromium.org/604173002/diff/1/net/quic/crypto/source_addre...
net/quic/crypto/source_address_token.h:101: int64 timestamp_;
On 2014/09/29 17:15:30, wtc wrote:
> 
> 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_.

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