Add methods in spdy/quic session to get stream id of pushed stream given the request url

Add methods in spdy/quic session to get stream id of pushed stream given the request url

BUG=232040
Committed: https://crrev.com/96d0202cee58e8165e100e996b7936ec538a11d0
Cr-Commit-Position: refs/heads/master@{#420574}

[Zhongyi Shi, 2016-09-21 00:17:41]

zhongyi@chromium.org changed reviewers:
+ rch@chromium.org

[Ryan Hamilton, 2016-09-22 21:41:33]

lgtm

https://codereview.chromium.org/2351373003/diff/1/net/quic/chromium/quic_chro...
File net/quic/chromium/quic_chromium_client_session.h (right):

https://codereview.chromium.org/2351373003/diff/1/net/quic/chromium/quic_chro...
net/quic/chromium/quic_chromium_client_session.h:238: // Else return 0.
nit:  // Returns the stream id of the push stream if it is not claimed yet, or 0
otherwise.

https://codereview.chromium.org/2351373003/diff/1/net/spdy/spdy_session.cc
File net/spdy/spdy_session.cc (left):

https://codereview.chromium.org/2351373003/diff/1/net/spdy/spdy_session.cc#ol...
net/spdy/spdy_session.cc:1708: }
Weird!

https://codereview.chromium.org/2351373003/diff/1/net/spdy/spdy_session.h
File net/spdy/spdy_session.h (right):

https://codereview.chromium.org/2351373003/diff/1/net/spdy/spdy_session.h#new...
net/spdy/spdy_session.h:816: // Else return 0.
nit: // Returns the stream id of the push stream if it is not claimed yet, or 0
otherwise.

[Zhongyi Shi, 2016-09-22 22:57:06]

https://codereview.chromium.org/2351373003/diff/1/net/quic/chromium/quic_chro...
File net/quic/chromium/quic_chromium_client_session.h (right):

https://codereview.chromium.org/2351373003/diff/1/net/quic/chromium/quic_chro...
net/quic/chromium/quic_chromium_client_session.h:238: // Else return 0.
On 2016/09/22 21:41:33, Ryan Hamilton wrote:
> nit:  // Returns the stream id of the push stream if it is not claimed yet, or
0
> otherwise.

Done.

https://codereview.chromium.org/2351373003/diff/1/net/spdy/spdy_session.h
File net/spdy/spdy_session.h (right):

https://codereview.chromium.org/2351373003/diff/1/net/spdy/spdy_session.h#new...
net/spdy/spdy_session.h:816: // Else return 0.
On 2016/09/22 21:41:33, Ryan Hamilton wrote:
> nit: // Returns the stream id of the push stream if it is not claimed yet, or
0
> otherwise.

Done.

[Zhongyi Shi, 2016-09-22 22:57:10]

The CQ bit was checked by zhongyi@chromium.org

[Zhongyi Shi, 2016-09-22 22:57:10]

The patchset sent to the CQ was uploaded after l-g-t-m from rch@chromium.org
Link to the patchset: https://codereview.chromium.org/2351373003/#ps20001
(title: "nits")

[commit-bot: I haz the power, 2016-09-22 22:57:44]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-23 00:59:41]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-09-23 00:59:42]

Try jobs failed on following builders:
  cast_shell_linux on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/cast_shell_linu...)
  linux_chromium_clobber_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[Zhongyi Shi, 2016-09-23 03:45:32]

The CQ bit was checked by zhongyi@chromium.org

[commit-bot: I haz the power, 2016-09-23 03:45:54]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-23 04:11:11]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-09-23 04:12:48]

Description was changed from

==========
Add methods in spdy/quic session to get stream id of pushed stream given the
request url

BUG=232040
==========

to

==========
Add methods in spdy/quic session to get stream id of pushed stream given the
request url

BUG=232040

Committed: https://crrev.com/96d0202cee58e8165e100e996b7936ec538a11d0
Cr-Commit-Position: refs/heads/master@{#420574}
==========

[commit-bot: I haz the power, 2016-09-23 04:12:49]

Patchset 2 (id:??) landed as
https://crrev.com/96d0202cee58e8165e100e996b7936ec538a11d0
Cr-Commit-Position: refs/heads/master@{#420574}