Replace for each loops

Replace for each loops
BUG=655950
Committed: https://crrev.com/9bd6ef5d4c1f52c0cbc5da2075f19c10577e337b
Cr-Commit-Position: refs/heads/master@{#429052}

[Yoonjae, 2016-10-31 03:20:52]

Description was changed from

==========
Replace for each loops
BUG=655950
==========

to

==========
Replace for each loops
BUG=655950
==========

[Yoonjae, 2016-10-31 03:20:52]

yoonjae.cho92@gmail.com changed reviewers:
+ mmenke@chromium.org

[Yoonjae, 2016-10-31 03:21:19]

@mmenke
PTAL.

[mmenke, 2016-10-31 14:49:30]

Looks good, just a pair of very minor comments.

And for the record, I've verified that you've signed the CLA.

https://codereview.chromium.org/2459243002/diff/20001/net/http/http_util.cc
File net/http/http_util.cc (right):

https://codereview.chromium.org/2459243002/diff/20001/net/http/http_util.cc#n...
net/http/http_util.cc:352: }
nit:  Don't use braces for ifs with a single line conditional and single line
body.

https://codereview.chromium.org/2459243002/diff/20001/net/http/http_util.cc#n...
net/http/http_util.cc:423: return true;
nit:  Mind adding braces to this if, though, while you're here, since the
condition takes up two lines?

[Yoonjae, 2016-11-01 09:10:35]

On 2016/10/31 14:49:30, mmenke (Busy Nov 2-3) wrote:
> Looks good, just a pair of very minor comments.
> 
> And for the record, I've verified that you've signed the CLA.
> 
> https://codereview.chromium.org/2459243002/diff/20001/net/http/http_util.cc
> File net/http/http_util.cc (right):
> 
>
https://codereview.chromium.org/2459243002/diff/20001/net/http/http_util.cc#n...
> net/http/http_util.cc:352: }
> nit:  Don't use braces for ifs with a single line conditional and single line
> body.
> 
>
https://codereview.chromium.org/2459243002/diff/20001/net/http/http_util.cc#n...
> net/http/http_util.cc:423: return true;
> nit:  Mind adding braces to this if, though, while you're here, since the
> condition takes up two lines?

@mmenke
Done. PTAL.

[mmenke, 2016-11-01 13:53:30]

LGTM, thanks for the cleanup!

[Yoonjae, 2016-11-01 14:55:03]

The CQ bit was checked by yoonjae.cho92@gmail.com

[commit-bot: I haz the power, 2016-11-01 14:55:19]

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-11-01 16:56:21]

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

[commit-bot: I haz the power, 2016-11-01 16:56:21]

Try jobs failed on following builders:
  win_chromium_x64_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

[mmenke, 2016-11-01 17:20:57]

The CQ bit was checked by mmenke@chromium.org

[commit-bot: I haz the power, 2016-11-01 17:21:17]

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

[mmenke, 2016-11-01 17:21:27]

On 2016/11/01 16:56:21, commit-bot: I haz the power wrote:
> Try jobs failed on following builders:
>   win_chromium_x64_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
>
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

That looks unrelated, so I'll re-run the tryjob.

[commit-bot: I haz the power, 2016-11-01 18:11:33]

Description was changed from

==========
Replace for each loops
BUG=655950
==========

to

==========
Replace for each loops
BUG=655950
==========

[commit-bot: I haz the power, 2016-11-01 18:11:34]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-11-01 18:24:57]

Description was changed from

==========
Replace for each loops
BUG=655950
==========

to

==========
Replace for each loops
BUG=655950

Committed: https://crrev.com/9bd6ef5d4c1f52c0cbc5da2075f19c10577e337b
Cr-Commit-Position: refs/heads/master@{#429052}
==========

[commit-bot: I haz the power, 2016-11-01 18:24:58]

Patchset 3 (id:??) landed as
https://crrev.com/9bd6ef5d4c1f52c0cbc5da2075f19c10577e337b
Cr-Commit-Position: refs/heads/master@{#429052}