Consolidate SpdyFramer::WriteHeaderBlock() and SerializeHeaderBlockWithoutCompression().

Consolidate SpdyFramer::WriteHeaderBlock() and SerializeHeaderBlockWithoutCompression().

SpdyFramer::WriteHeaderBlock() and
SpdyFramer::SerializeHeaderBlockWithoutCompression() are identical, except one
of them is static.  This CL slays this redundancy.

* Remove WriteHeaderBlock().
* Change SerializeHeaderBlockWithoutCompression() to public so that it can be
  called from net/quic/spdy_utils.cc.
* Instantiate a SpdyFramer in net/quic/spdy_utils.cc
  (WriteHeaderBlock used to be static) and call SerializeUncompressedHeaders on
  it.

This CL lands server change 116771332 by bnc.

BUG=521600, 488484
Committed: https://crrev.com/33557b831bb6c4feb3b5cc9bc176fc0148f37e8c
Cr-Commit-Position: refs/heads/master@{#380445}

[Bence, 2016-03-10 01:17:41]

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

[Bence, 2016-03-10 01:17:41]

Ryan: PTAL.  Thank you.

[Ryan Hamilton, 2016-03-10 19:03:20]

lgtm

[Bence, 2016-03-10 19:12:05]

The CQ bit was checked by bnc@chromium.org

[commit-bot: I haz the power, 2016-03-10 19:12:41]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1776373003/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1776373003/1

[commit-bot: I haz the power, 2016-03-10 19:27:01]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-03-10 19:27:56]

Description was changed from

==========
Consolidate SpdyFramer::WriteHeaderBlock() and
SerializeHeaderBlockWithoutCompression().

SpdyFramer::WriteHeaderBlock() and
SpdyFramer::SerializeHeaderBlockWithoutCompression() are identical, except one
of them is static.  This CL slays this redundancy.

* Remove WriteHeaderBlock().
* Change SerializeHeaderBlockWithoutCompression() to public so that it can be
  called from net/quic/spdy_utils.cc.
* Instantiate a SpdyFramer in net/quic/spdy_utils.cc
  (WriteHeaderBlock used to be static) and call SerializeUncompressedHeaders on
  it.

This CL lands server change 116771332 by bnc.

BUG=521600, 488484
==========

to

==========
Consolidate SpdyFramer::WriteHeaderBlock() and
SerializeHeaderBlockWithoutCompression().

SpdyFramer::WriteHeaderBlock() and
SpdyFramer::SerializeHeaderBlockWithoutCompression() are identical, except one
of them is static.  This CL slays this redundancy.

* Remove WriteHeaderBlock().
* Change SerializeHeaderBlockWithoutCompression() to public so that it can be
  called from net/quic/spdy_utils.cc.
* Instantiate a SpdyFramer in net/quic/spdy_utils.cc
  (WriteHeaderBlock used to be static) and call SerializeUncompressedHeaders on
  it.

This CL lands server change 116771332 by bnc.

BUG=521600, 488484

Committed: https://crrev.com/33557b831bb6c4feb3b5cc9bc176fc0148f37e8c
Cr-Commit-Position: refs/heads/master@{#380445}
==========

[commit-bot: I haz the power, 2016-03-10 19:27:57]

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