Split out spdy_log_util from http_log_util.

Split out spdy_log_util from http_log_util.

Motivation: ElideHeaderValueForNetLog() is used in HttpRequestHeaders
and HttpResponseHeaders, therefore must be included in a nacl build.  On
the other hand, ElideGoAwayDebugDataForNetLog() and
ElideSpdyHeaderBlockForNetLog(), and their dependencies including
SpdyHeaderBlock, UnsafeArena, and linked_hash_map, do not need to be
included in a nacl build.  This CL splits out these functions from
http_log_util into spdy_log_util, and reverts the build changes that I
made in https://crrev.com/2622703002.

Also remove NetLogCaptureMode forward declaration: instances are passed by
value, therefore declaration has to be included.

Also use |auto headers_list = base::MakeUnique| instead of |unique_ptr
headers_list(new)| in ElideSpdyHeaderBlockForNetLog().

Also add unittest for ElideSpdyHeaderBlockForNetLog().

Review-Url: https://codereview.chromium.org/2771033005
Cr-Commit-Position: refs/heads/master@{#459454}
Committed: https://chromium.googlesource.com/chromium/src/+/4fab802da9defe20515b8da9ab72e9017bded970

[Bence, 2017-03-24 13:24:24]

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

[commit-bot: I haz the power, 2017-03-24 13:24:41]

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

[Bence, 2017-03-24 14:39:07]

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

[Bence, 2017-03-24 14:39:08]

xunjieli: PTAL.  Thank you.

[xunjieli, 2017-03-24 15:05:38]

On 2017/03/24 14:39:08, Bence wrote:
> xunjieli: PTAL.  Thank you.

Nice change! The added unit tests are helpful.
LGTM

[Bence, 2017-03-24 15:06:54]

The CQ bit was checked by bnc@chromium.org

[Bence, 2017-03-24 15:07:11]

On 2017/03/24 15:05:38, xunjieli wrote:
> On 2017/03/24 14:39:08, Bence wrote:
> > xunjieli: PTAL.  Thank you.
> 
> Nice change! The added unit tests are helpful.
> LGTM

Thank you.

[commit-bot: I haz the power, 2017-03-24 15:07:27]

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, 2017-03-24 16:35:09]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1490368014850000,
"parent_rev": "22f7196580508f2f6c55a234d2c90da7f58cf8e4", "commit_rev":
"4fab802da9defe20515b8da9ab72e9017bded970"}

[commit-bot: I haz the power, 2017-03-24 16:35:58]

Description was changed from

==========
Split out spdy_log_util from http_log_util.

Motivation: ElideHeaderValueForNetLog() is used in HttpRequestHeaders
and HttpResponseHeaders, therefore must be included in a nacl build.  On
the other hand, ElideGoAwayDebugDataForNetLog() and
ElideSpdyHeaderBlockForNetLog(), and their dependencies including
SpdyHeaderBlock, UnsafeArena, and linked_hash_map, do not need to be
included in a nacl build.  This CL splits out these functions from
http_log_util into spdy_log_util, and reverts the build changes that I
made in https://crrev.com/2622703002.

Also remove NetLogCaptureMode forward declaration: instances are passed by
value, therefore declaration has to be included.

Also use |auto headers_list = base::MakeUnique| instead of |unique_ptr
headers_list(new)| in ElideSpdyHeaderBlockForNetLog().

Also add unittest for ElideSpdyHeaderBlockForNetLog().
==========

to

==========
Split out spdy_log_util from http_log_util.

Motivation: ElideHeaderValueForNetLog() is used in HttpRequestHeaders
and HttpResponseHeaders, therefore must be included in a nacl build.  On
the other hand, ElideGoAwayDebugDataForNetLog() and
ElideSpdyHeaderBlockForNetLog(), and their dependencies including
SpdyHeaderBlock, UnsafeArena, and linked_hash_map, do not need to be
included in a nacl build.  This CL splits out these functions from
http_log_util into spdy_log_util, and reverts the build changes that I
made in https://crrev.com/2622703002.

Also remove NetLogCaptureMode forward declaration: instances are passed by
value, therefore declaration has to be included.

Also use |auto headers_list = base::MakeUnique| instead of |unique_ptr
headers_list(new)| in ElideSpdyHeaderBlockForNetLog().

Also add unittest for ElideSpdyHeaderBlockForNetLog().

Review-Url: https://codereview.chromium.org/2771033005
Cr-Commit-Position: refs/heads/master@{#459454}
Committed:
https://chromium.googlesource.com/chromium/src/+/4fab802da9defe20515b8da9ab72...
==========

[commit-bot: I haz the power, 2017-03-24 16:35:59]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/4fab802da9defe20515b8da9ab72...