Reland of place for loops with |arraysize| with for each loops

Reland of place for loops with |arraysize| with for each loops (patchset #1 id:1 of https://codereview.chromium.org/2424793002/ )

Reason for revert:
Relanding, because the flaky unittest was fixed: http://crbug.com/656926

Original issue's description:
> Revert of Replace for loops with |arraysize| with for each loops (patchset #2 id:20001 of https://codereview.chromium.org/2417783004/ )
>
> Reason for revert:
> Speculative revert from sheriff. Suspect that the CL breaks components_unittests on Mac.
>
> https://uberchromegw.corp.google.com/i/chromium.mac/builders/Mac10.9%20Tests%20%28dbg%29/builds/31721/steps/components_unittests%20on%20Mac-10.9
>
> Not sure. Will revert the revert if it does not help.
>
> Original issue's description:
> > Replace for loops with |arraysize| with for each loops
> >
> > This change replaces for-loops involving a loop variable and the
> > |arraysize| macro with a C++11 for-each-loop where appropriate.
> >
> > A few usages of |arraysize| in a for-loop remain, in these the loop
> > variable is used as well. This can be simply logging the iteration
> > number or indexing another array of the same size.
> >
> > BUG=655950
> >
> > Committed: https://crrev.com/54e466c9fd2ff04136f8f305615910e8f9085642
> > Cr-Commit-Position: refs/heads/master@{#425651}
>
> TBR=vabr@chromium.org,jdoerrie@chromium.org
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=655950
>
> Committed: https://crrev.com/e00a337f9c62d1d32c315ecacc63cec380a36e27
> Cr-Commit-Position: refs/heads/master@{#425660}

TBR=vabr@chromium.org,henrika@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=655950
Committed: https://crrev.com/188854d02b406012ec48a550ffcdd273c8b14d53
Cr-Commit-Position: refs/heads/master@{#429578}

[jdoerrie, 2016-11-03 13:47:49]

The CQ bit was checked by jdoerrie@chromium.org

[jdoerrie, 2016-11-03 13:47:50]

Created Reland of place for loops with |arraysize| with for each loops

[commit-bot: I haz the power, 2016-11-03 13:48:10]

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-03 13:48:11]

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

[commit-bot: I haz the power, 2016-11-03 13:48:12]

No L-G-T-M from a valid reviewer yet. 
CQ run can only be started by full committers or once the patch has
received an L-G-T-M from a full committer.
Even if an L-G-T-M may have been provided, it was from a non-committer,
_not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[vabr (Chromium), 2016-11-03 14:02:38]

LGTM, as confirmed with jdoerrie, this is a no-change reland of the initial CL.
The original approvals carry-over.

Cheers,
Vaclav

[vabr (Chromium), 2016-11-03 14:02:46]

The CQ bit was checked by vabr@chromium.org

[commit-bot: I haz the power, 2016-11-03 14:03:08]

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-03 14:42:23]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-11-03 14:44:34]

Description was changed from

==========
Reland of place for loops with |arraysize| with for each loops (patchset #1 id:1
of https://codereview.chromium.org/2424793002/ )

Reason for revert:
Relanding, because the flaky unittest was fixed: http://crbug.com/656926

Original issue's description:
> Revert of Replace for loops with |arraysize| with for each loops (patchset #2
id:20001 of https://codereview.chromium.org/2417783004/ )
> 
> Reason for revert:
> Speculative revert from sheriff. Suspect that the CL breaks
components_unittests on Mac.
> 
>
https://uberchromegw.corp.google.com/i/chromium.mac/builders/Mac10.9%20Tests%...
> 
> Not sure. Will revert the revert if it does not help.
> 
> Original issue's description:
> > Replace for loops with |arraysize| with for each loops
> >
> > This change replaces for-loops involving a loop variable and the
> > |arraysize| macro with a C++11 for-each-loop where appropriate.
> >
> > A few usages of |arraysize| in a for-loop remain, in these the loop
> > variable is used as well. This can be simply logging the iteration
> > number or indexing another array of the same size.
> >
> > BUG=655950
> >
> > Committed: https://crrev.com/54e466c9fd2ff04136f8f305615910e8f9085642
> > Cr-Commit-Position: refs/heads/master@{#425651}
> 
> TBR=vabr@chromium.org,jdoerrie@chromium.org
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=655950
> 
> Committed: https://crrev.com/e00a337f9c62d1d32c315ecacc63cec380a36e27
> Cr-Commit-Position: refs/heads/master@{#425660}

TBR=vabr@chromium.org,henrika@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=655950
==========

to

==========
Reland of place for loops with |arraysize| with for each loops (patchset #1 id:1
of https://codereview.chromium.org/2424793002/ )

Reason for revert:
Relanding, because the flaky unittest was fixed: http://crbug.com/656926

Original issue's description:
> Revert of Replace for loops with |arraysize| with for each loops (patchset #2
id:20001 of https://codereview.chromium.org/2417783004/ )
>
> Reason for revert:
> Speculative revert from sheriff. Suspect that the CL breaks
components_unittests on Mac.
>
>
https://uberchromegw.corp.google.com/i/chromium.mac/builders/Mac10.9%20Tests%...
>
> Not sure. Will revert the revert if it does not help.
>
> Original issue's description:
> > Replace for loops with |arraysize| with for each loops
> >
> > This change replaces for-loops involving a loop variable and the
> > |arraysize| macro with a C++11 for-each-loop where appropriate.
> >
> > A few usages of |arraysize| in a for-loop remain, in these the loop
> > variable is used as well. This can be simply logging the iteration
> > number or indexing another array of the same size.
> >
> > BUG=655950
> >
> > Committed: https://crrev.com/54e466c9fd2ff04136f8f305615910e8f9085642
> > Cr-Commit-Position: refs/heads/master@{#425651}
>
> TBR=vabr@chromium.org,jdoerrie@chromium.org
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=655950
>
> Committed: https://crrev.com/e00a337f9c62d1d32c315ecacc63cec380a36e27
> Cr-Commit-Position: refs/heads/master@{#425660}

TBR=vabr@chromium.org,henrika@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=655950

Committed: https://crrev.com/188854d02b406012ec48a550ffcdd273c8b14d53
Cr-Commit-Position: refs/heads/master@{#429578}
==========

[commit-bot: I haz the power, 2016-11-03 14:44:35]

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