ui/views: convert usage of FOR_EACH_OBSERVER macro

ui/views: convert usage of FOR_EACH_OBSERVER macro

ObserverList now supports range-based for loops and thus this can be
used to iterate through a list of observers as the FOR_EACH_OBSERVER
macro was deprecated.

This patch was generated by the following command line:
$ cd ui/views
$ perl -0777 -pi -e
's/FOR_EACH_OBSERVER\(\s*([\w:]+?)\s*,\s*([\w:()]+?)\s*,\s*([^;]+?)\);/for
(\1& observer : \2) observer.\3;/mg' $(git gs FOR_EACH_OBSERVER | cut -d: -f1 | uniq)

And then manually inspected for line indentation fixes.

BUG=655021
TEST=views_unittests
R=sadrul@chromium.org

Committed: https://crrev.com/725a3dbcf54b79a10d1eba8f8164f7e22d4dab06
Cr-Commit-Position: refs/heads/master@{#425359}

[tfarina, 2016-10-14 03:21:20]

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

[commit-bot: I haz the power, 2016-10-14 03:21:35]

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-10-14 04:13:02]

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

[commit-bot: I haz the power, 2016-10-14 04:13:03]

Dry run: This issue passed the CQ dry run.

[loyso (OOO), 2016-10-14 04:25:28]

LGTMed duplicate: https://codereview.chromium.org/2417463009/#

[loyso (OOO), 2016-10-14 04:26:25]

Plese, mark your intents in
https://docs.google.com/spreadsheets/d/14u7dpq7NcBTtQj3hhEOrnA5Eq3Q9N_ON4w9pL...

[loyso (OOO), 2016-10-14 04:30:53]

Let you choose, what to land. Explicit vs. implicit :)

[tfarina, 2016-10-14 12:10:50]

On 2016/10/14 04:26:25, loyso wrote:
> Plese, mark your intents in
>
https://docs.google.com/spreadsheets/d/14u7dpq7NcBTtQj3hhEOrnA5Eq3Q9N_ON4w9pL...

I can't edit from iPad, it does not let me it seems. So I left a comment on the
bug mention this.

[tfarina, 2016-10-14 12:11:55]

On 2016/10/14 04:30:53, loyso wrote:
> Let you choose, what to land. Explicit vs. implicit :)

Scott prefers explicit types (as it happened in the other review).

[sky, 2016-10-14 15:56:28]

sky@chromium.org changed reviewers:
+ sadrul@chromium.org
- sky@chromium.org

[sky, 2016-10-14 15:56:29]

sky->sadrul, I wouldn't want him to miss on the review fun of these patches ;)

[sadrul, 2016-10-14 16:06:26]

lgtm

[tfarina, 2016-10-14 16:41:23]

Description was changed from

==========
ui/views: convert usage of FOR_EACH_OBSERVER macro

ObserverList now supports range-based for loops and thus this can be
used to iterate through a list of observers as the FOR_EACH_OBSERVER
macro was deprecated.

This patch was generated by the following command line:
$ cd ui/views
$ perl -0777 -pi -e
's/FOR_EACH_OBSERVER\(\s*([\w:]+?)\s*,\s*([\w:()]+?)\s*,\s*([^;]+?)\);/for
(\1& observer : \2) observer.\3;/mg' $(git gs FOR_EACH_OBSERVER | cut -d: -f1 |
uniq)

And then manually inspected for line indentation fixes.

BUG=655021
TEST=views_unittests
R=sky@chromium.org
==========

to

==========
ui/views: convert usage of FOR_EACH_OBSERVER macro

ObserverList now supports range-based for loops and thus this can be
used to iterate through a list of observers as the FOR_EACH_OBSERVER
macro was deprecated.

This patch was generated by the following command line:
$ cd ui/views
$ perl -0777 -pi -e
's/FOR_EACH_OBSERVER\(\s*([\w:]+?)\s*,\s*([\w:()]+?)\s*,\s*([^;]+?)\);/for
(\1& observer : \2) observer.\3;/mg' $(git gs FOR_EACH_OBSERVER | cut -d: -f1 |
uniq)

And then manually inspected for line indentation fixes.

BUG=655021
TEST=views_unittests
R=sadrul@chromium.org
==========

[tfarina, 2016-10-14 16:41:30]

The CQ bit was checked by tfarina@chromium.org

[commit-bot: I haz the power, 2016-10-14 16:42:02]

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-14 16:49:21]

Description was changed from

==========
ui/views: convert usage of FOR_EACH_OBSERVER macro

ObserverList now supports range-based for loops and thus this can be
used to iterate through a list of observers as the FOR_EACH_OBSERVER
macro was deprecated.

This patch was generated by the following command line:
$ cd ui/views
$ perl -0777 -pi -e
's/FOR_EACH_OBSERVER\(\s*([\w:]+?)\s*,\s*([\w:()]+?)\s*,\s*([^;]+?)\);/for
(\1& observer : \2) observer.\3;/mg' $(git gs FOR_EACH_OBSERVER | cut -d: -f1 |
uniq)

And then manually inspected for line indentation fixes.

BUG=655021
TEST=views_unittests
R=sadrul@chromium.org
==========

to

==========
ui/views: convert usage of FOR_EACH_OBSERVER macro

ObserverList now supports range-based for loops and thus this can be
used to iterate through a list of observers as the FOR_EACH_OBSERVER
macro was deprecated.

This patch was generated by the following command line:
$ cd ui/views
$ perl -0777 -pi -e
's/FOR_EACH_OBSERVER\(\s*([\w:]+?)\s*,\s*([\w:()]+?)\s*,\s*([^;]+?)\);/for
(\1& observer : \2) observer.\3;/mg' $(git gs FOR_EACH_OBSERVER | cut -d: -f1 |
uniq)

And then manually inspected for line indentation fixes.

BUG=655021
TEST=views_unittests
R=sadrul@chromium.org
==========

[commit-bot: I haz the power, 2016-10-14 16:49:22]

Committed patchset #1 (id:1)

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

Description was changed from

==========
ui/views: convert usage of FOR_EACH_OBSERVER macro

ObserverList now supports range-based for loops and thus this can be
used to iterate through a list of observers as the FOR_EACH_OBSERVER
macro was deprecated.

This patch was generated by the following command line:
$ cd ui/views
$ perl -0777 -pi -e
's/FOR_EACH_OBSERVER\(\s*([\w:]+?)\s*,\s*([\w:()]+?)\s*,\s*([^;]+?)\);/for
(\1& observer : \2) observer.\3;/mg' $(git gs FOR_EACH_OBSERVER | cut -d: -f1 |
uniq)

And then manually inspected for line indentation fixes.

BUG=655021
TEST=views_unittests
R=sadrul@chromium.org
==========

to

==========
ui/views: convert usage of FOR_EACH_OBSERVER macro

ObserverList now supports range-based for loops and thus this can be
used to iterate through a list of observers as the FOR_EACH_OBSERVER
macro was deprecated.

This patch was generated by the following command line:
$ cd ui/views
$ perl -0777 -pi -e
's/FOR_EACH_OBSERVER\(\s*([\w:]+?)\s*,\s*([\w:()]+?)\s*,\s*([^;]+?)\);/for
(\1& observer : \2) observer.\3;/mg' $(git gs FOR_EACH_OBSERVER | cut -d: -f1 |
uniq)

And then manually inspected for line indentation fixes.

BUG=655021
TEST=views_unittests
R=sadrul@chromium.org

Committed: https://crrev.com/725a3dbcf54b79a10d1eba8f8164f7e22d4dab06
Cr-Commit-Position: refs/heads/master@{#425359}
==========

[commit-bot: I haz the power, 2016-10-14 16:53:17]

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