Misc cleanup.

Misc cleanup.

This CL is a collection of changes that are too small for individual CLs.

* Mark two HttpStreamFactoryImpl::Job members const, since they are never
  changed.
* Remove unused CryptoConfigMap typedef.
* Make SpdyNetworkTransactionTest::NormalSpdyTransactionHelper::output() method
  return const ref instead of unconventional non-const ref.
* Remove unused SpdyNetworkTransactionTest::NormalSpdyTransactionHelper members.
* In two SpdyNetworkTransactionTests, use NormalSpdyTransactionHelper methods to
  start transaction and wait for callback instead of directly manipulating
  helper.trans().

Committed: https://crrev.com/4d782f4935e5a1b3c19d6ee3725928bcc7f2e936
Cr-Commit-Position: refs/heads/master@{#412817}

[Bence, 2016-08-17 19:06:18]

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

[commit-bot: I haz the power, 2016-08-17 19:06:40]

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-08-17 20:02:31]

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

[commit-bot: I haz the power, 2016-08-17 20:02:32]

Dry run: This issue passed the CQ dry run.

[Bence, 2016-08-18 12:16:04]

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

[Bence, 2016-08-18 12:16:05]

Ryan: PTAL.  Thank you.

[Ryan Hamilton, 2016-08-18 13:45:35]

The CQ bit was checked by rch@chromium.org

[Ryan Hamilton, 2016-08-18 13:45:36]

lgtm

[commit-bot: I haz the power, 2016-08-18 13:45:59]

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-08-18 13:50:18]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-08-18 13:53:49]

Description was changed from

==========
Misc cleanup.

This CL is a collection of changes that are too small for individual CLs.

* Mark two HttpStreamFactoryImpl::Job members const, since they are never
  changed.
* Remove unused CryptoConfigMap typedef.
* Make SpdyNetworkTransactionTest::NormalSpdyTransactionHelper::output() method
  return const ref instead of unconventional non-const ref.
* Remove unused SpdyNetworkTransactionTest::NormalSpdyTransactionHelper members.
* In two SpdyNetworkTransactionTests, use NormalSpdyTransactionHelper methods to
  start transaction and wait for callback instead of directly manipulating
  helper.trans().
==========

to

==========
Misc cleanup.

This CL is a collection of changes that are too small for individual CLs.

* Mark two HttpStreamFactoryImpl::Job members const, since they are never
  changed.
* Remove unused CryptoConfigMap typedef.
* Make SpdyNetworkTransactionTest::NormalSpdyTransactionHelper::output() method
  return const ref instead of unconventional non-const ref.
* Remove unused SpdyNetworkTransactionTest::NormalSpdyTransactionHelper members.
* In two SpdyNetworkTransactionTests, use NormalSpdyTransactionHelper methods to
  start transaction and wait for callback instead of directly manipulating
  helper.trans().

Committed: https://crrev.com/4d782f4935e5a1b3c19d6ee3725928bcc7f2e936
Cr-Commit-Position: refs/heads/master@{#412817}
==========

[commit-bot: I haz the power, 2016-08-18 13:53:50]

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