Issue 1425823005: Update the comment of --quic_version.

https://codereview.chromium.org/1425823005/diff/1/net/tools/quic/quic_client_bin.cc File net/tools/quic/quic_client_bin.cc (right): https://codereview.chromium.org/1425823005/diff/1/net/tools/quic/quic_client_bin.cc#newcode22 net/tools/quic/quic_client_bin.cc:22: // quic_client http://www.google.com --quic_version=23 --host=${IP} Can you updated quic_simple_client_bin.cc ...

5 years, 1 month ago (2015-11-10 21:33:13 UTC) #2

rjshade

On 2015/11/10 21:33:13, Ryan Hamilton wrote: > https://codereview.chromium.org/1425823005/diff/1/net/tools/quic/quic_client_bin.cc > File net/tools/quic/quic_client_bin.cc (right): > > https://codereview.chromium.org/1425823005/diff/1/net/tools/quic/quic_client_bin.cc#newcode22 ...

5 years, 1 month ago (2015-11-11 20:27:30 UTC) #3

Ryan Hamilton

5 years, 1 month ago (2015-11-11 22:14:14 UTC) #4

On 2015/11/11 20:27:30, rjshade wrote:
> On 2015/11/10 21:33:13, Ryan Hamilton wrote:
> >
>
https://codereview.chromium.org/1425823005/diff/1/net/tools/quic/quic_client_...
> > File net/tools/quic/quic_client_bin.cc (right):
> > 
> >
>
https://codereview.chromium.org/1425823005/diff/1/net/tools/quic/quic_client_...
> > net/tools/quic/quic_client_bin.cc:22: //   quic_client http://www.google.com
> > --quic_version=23  --host=${IP}
> > Can you updated quic_simple_client_bin.cc too?
> > 
> > Feel free to do in the Final CL if that is easier.
> 
> Done in Final CL.

lgtm (though I don't yet see the final CL)

Expand Messages | Collapse Messages