Reduce SpdyHeaderBlock copies with move semantics in shared code.

Reduce SpdyHeaderBlock copies with move semantics in shared code.

In order to reduce SpdyHeaderBlock copies,
* take SpdyHeaderBlock by value instead of const ref,
* std::move SpdyHeaderBlock at call sites if possible.

This lands server side change 125673600 by bnc.

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

[Bence, 2016-06-23 17:20:24]

Description was changed from

==========
Reduce SpdyHeaderBlock copies with move semantics.

In QuicSpdyStream, QuicTestPacketMaker, QuicChromiumClientStream,
BufferedSpdyFramer, and SpdyTestUtil methods,
* take SpdyHeaderBlock by value instead of const ref or unique_ptr, and/or
* return SpdyHeaderBlock by value instead of unique_ptr
in order to reduce SpdyHeaderBlock copies.

This lands server side change 125673600 by bnc.

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

to

==========
Reduce SpdyHeaderBlock copies with move semantics in shared code.

In order to reduce SpdyHeaderBlock copies,
* take SpdyHeaderBlock by value instead of const ref,
* std::move SpdyHeaderBlock at call sites if possible.

This lands server side change 125673600 by bnc.

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

[Bence, 2016-06-23 17:22:00]

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

[Bence, 2016-06-23 17:22:01]

Ryan: PTAL.  Thank you.

[Ryan Hamilton, 2016-06-23 18:18:11]

lgtm

[Bence, 2016-06-23 23:01:31]

The CQ bit was checked by bnc@chromium.org

[commit-bot: I haz the power, 2016-06-23 23:02:39]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2093553004/20001

[commit-bot: I haz the power, 2016-06-23 23:08:51]

Description was changed from

==========
Reduce SpdyHeaderBlock copies with move semantics in shared code.

In order to reduce SpdyHeaderBlock copies,
* take SpdyHeaderBlock by value instead of const ref,
* std::move SpdyHeaderBlock at call sites if possible.

This lands server side change 125673600 by bnc.

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

to

==========
Reduce SpdyHeaderBlock copies with move semantics in shared code.

In order to reduce SpdyHeaderBlock copies,
* take SpdyHeaderBlock by value instead of const ref,
* std::move SpdyHeaderBlock at call sites if possible.

This lands server side change 125673600 by bnc.

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

[commit-bot: I haz the power, 2016-06-23 23:08:55]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-06-23 23:10:34]

Description was changed from

==========
Reduce SpdyHeaderBlock copies with move semantics in shared code.

In order to reduce SpdyHeaderBlock copies,
* take SpdyHeaderBlock by value instead of const ref,
* std::move SpdyHeaderBlock at call sites if possible.

This lands server side change 125673600 by bnc.

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

to

==========
Reduce SpdyHeaderBlock copies with move semantics in shared code.

In order to reduce SpdyHeaderBlock copies,
* take SpdyHeaderBlock by value instead of const ref,
* std::move SpdyHeaderBlock at call sites if possible.

This lands server side change 125673600 by bnc.

BUG=621905

Committed: https://crrev.com/2dc1cc43f5fe0e63c0e632008f4d9a8f122f5f6d
Cr-Commit-Position: refs/heads/master@{#401749}
==========

[commit-bot: I haz the power, 2016-06-23 23:10:35]

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