[net] clang-format HttpNetworkTransactionTest.GenerateAuthToken test.

[net] clang-format HttpNetworkTransactionTest.GenerateAuthToken test.

This is a pretty big test with a lot of test cases. clang-format pretty
much reformats the entire test beyond recognition. Subsequent patches
that may touch this test will be unreviewable if they also include the
formatting changes. Uploading the formatting changes separately.

BUG=648366
Committed: https://crrev.com/c930761972fa8611069d0e55e3f45dd0c56c2d54
Cr-Commit-Position: refs/heads/master@{#422429}

[asanka, 2016-09-30 22:23:06]

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

[commit-bot: I haz the power, 2016-09-30 22:23:57]

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

[asanka, 2016-09-30 22:49:07]

asanka@chromium.org changed reviewers:
+ mmenke@chromium.org

[asanka, 2016-09-30 22:51:10]

mmenke:

The only changes here are:

  1. Get rid of kAuthErr in line 11279, and replace its occurrences with
ERR_INVALID_AUTH_CREDENTIALS in the test cases that follow. In a dependent patch
set we are going to try test cases with different error codes.

  2. Format the test cases using clang-format.

[commit-bot: I haz the power, 2016-09-30 23:03:26]

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

[commit-bot: I haz the power, 2016-09-30 23:03:26]

Dry run: This issue passed the CQ dry run.

[mmenke, 2016-10-03 15:06:07]

On 2016/09/30 22:51:10, asanka wrote:
> mmenke:
> 
> The only changes here are:
> 
>   1. Get rid of kAuthErr in line 11279, and replace its occurrences with
> ERR_INVALID_AUTH_CREDENTIALS in the test cases that follow. In a dependent
patch
> set we are going to try test cases with different error codes.
> 
>   2. Format the test cases using clang-format.

LGTM

[mmenke, 2016-10-03 15:06:26]

On 2016/10/03 15:06:07, mmenke wrote:
> On 2016/09/30 22:51:10, asanka wrote:
> > mmenke:
> > 
> > The only changes here are:
> > 
> >   1. Get rid of kAuthErr in line 11279, and replace its occurrences with
> > ERR_INVALID_AUTH_CREDENTIALS in the test cases that follow. In a dependent
> patch
> > set we are going to try test cases with different error codes.
> > 
> >   2. Format the test cases using clang-format.
> 
> LGTM

And thanks for doing this in a separate CL, will make life much easier.

[asanka, 2016-10-03 15:14:36]

Thanks!

[asanka, 2016-10-03 15:14:39]

The CQ bit was checked by asanka@chromium.org

[commit-bot: I haz the power, 2016-10-03 15:14:53]

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-10-03 15:46:21]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-10-03 15:50:24]

Description was changed from

==========
[net] clang-format HttpNetworkTransactionTest.GenerateAuthToken test.

This is a pretty big test with a lot of test cases. clang-format pretty
much reformats the entire test beyond recognition. Subsequent patches
that may touch this test will be unreviewable if they also include the
formatting changes. Uploading the formatting changes separately.

BUG=648366
==========

to

==========
[net] clang-format HttpNetworkTransactionTest.GenerateAuthToken test.

This is a pretty big test with a lot of test cases. clang-format pretty
much reformats the entire test beyond recognition. Subsequent patches
that may touch this test will be unreviewable if they also include the
formatting changes. Uploading the formatting changes separately.

BUG=648366

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

[commit-bot: I haz the power, 2016-10-03 15:50:25]

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