Eliminate SpdyHeaderBlock copy in SpdyStream.

Eliminate SpdyHeaderBlock copy in SpdyStream.

Eliminate one SpdyHeaderBlock copy in SpdyStream::ProduceSynStreamFrame() (soon
to be renamed ProduceHeaderFrame).  This is achived by saving the GURL as soon
as request_headers_ are set.

Note that |url_| is not necessarily the same as url_from_header_block_, see, for
example, SpdyHttpStreamTest.SpdyURLTest.

Also remove test-only SpdyStream::HasUrlFromHeaders().

BUG=621905
Committed: https://crrev.com/da23393bc3adbd89f2ed5b57c1ad59def33c3ef1
Cr-Commit-Position: refs/heads/master@{#407055}

[Bence, 2016-07-20 18:26:23]

The CQ bit was checked by bnc@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-07-20 18:26:52]

Dry run: 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-07-20 19:31:13]

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

[commit-bot: I haz the power, 2016-07-20 19:31:14]

Dry run: This issue passed the CQ dry run.

[Bence, 2016-07-21 00:39:39]

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

[Bence, 2016-07-21 00:39:40]

Ryan: PTAL.  Thank you.

[Ryan Hamilton, 2016-07-21 22:15:37]

lgtm

nice!

[Bence, 2016-07-21 22:59:50]

The CQ bit was checked by bnc@chromium.org

[commit-bot: I haz the power, 2016-07-21 23:00:32]

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-07-22 03:59:24]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-07-22 04:02:10]

Description was changed from

==========
Eliminate SpdyHeaderBlock copy in SpdyStream.

Eliminate one SpdyHeaderBlock copy in SpdyStream::ProduceSynStreamFrame() (soon
to be renamed ProduceHeaderFrame).  This is achived by saving the GURL as soon
as request_headers_ are set.

Note that |url_| is not necessarily the same as url_from_header_block_, see, for
example, SpdyHttpStreamTest.SpdyURLTest.

Also remove test-only SpdyStream::HasUrlFromHeaders().

BUG=621905
==========

to

==========
Eliminate SpdyHeaderBlock copy in SpdyStream.

Eliminate one SpdyHeaderBlock copy in SpdyStream::ProduceSynStreamFrame() (soon
to be renamed ProduceHeaderFrame).  This is achived by saving the GURL as soon
as request_headers_ are set.

Note that |url_| is not necessarily the same as url_from_header_block_, see, for
example, SpdyHttpStreamTest.SpdyURLTest.

Also remove test-only SpdyStream::HasUrlFromHeaders().

BUG=621905

Committed: https://crrev.com/da23393bc3adbd89f2ed5b57c1ad59def33c3ef1
Cr-Commit-Position: refs/heads/master@{#407055}
==========

[commit-bot: I haz the power, 2016-07-22 04:02:11]

Patchset 1 (id:??) landed as
https://crrev.com/da23393bc3adbd89f2ed5b57c1ad59def33c3ef1
Cr-Commit-Position: refs/heads/master@{#407055}