Remove FOR_EACH_OBSERVER macro usage in chrome/browser/ui

Remove FOR_EACH_OBSERVER macro usage in chrome/browser/ui

Observer lists now support range-based for loops.

BUG=655021
Committed: https://crrev.com/9c2422f2cbdaee59dd22ebb9821e06fb2fdda2dd
Cr-Commit-Position: refs/heads/master@{#425881}

[Eric Willigers, 2016-10-18 00:16:51]

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

[commit-bot: I haz the power, 2016-10-18 00:17:56]

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

[Eric Willigers, 2016-10-18 00:23:00]

ericwilligers@chromium.org changed reviewers:
+ msw@chromium.org

[Eric Willigers, 2016-10-18 00:23:01]

In https://codereview.chromium.org/2422073002/
sky@ preferred explicit types instead of auto.

Should chrome/browser/ui also use explicit types?

[msw, 2016-10-18 00:49:31]

On 2016/10/18 00:23:01, Eric Willigers wrote:
> In https://codereview.chromium.org/2422073002/
> sky@ preferred explicit types instead of auto.
> 
> Should chrome/browser/ui also use explicit types?

Yes, please follow the same pattern here, thanks!

[Eric Willigers, 2016-10-18 01:11:28]

> > In https://codereview.chromium.org/2422073002/
> > sky@ preferred explicit types instead of auto.
> > 
> > Should chrome/browser/ui also use explicit types?
> 
> Yes, please follow the same pattern here, thanks!

Done.

[Eric Willigers, 2016-10-18 01:11:35]

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

[commit-bot: I haz the power, 2016-10-18 01:12:23]

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

[msw, 2016-10-18 01:24:56]

nice, lgtm!

[Eric Willigers, 2016-10-18 02:12:16]

The CQ bit was unchecked by ericwilligers@chromium.org

[Eric Willigers, 2016-10-18 02:12:23]

The CQ bit was checked by ericwilligers@chromium.org

[commit-bot: I haz the power, 2016-10-18 02:12:54]

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-18 03:19:08]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-10-18 03:21:47]

Description was changed from

==========
Remove FOR_EACH_OBSERVER macro usage in chrome/browser/ui

Observer lists now support range-based for loops.


BUG=655021
==========

to

==========
Remove FOR_EACH_OBSERVER macro usage in chrome/browser/ui

Observer lists now support range-based for loops.

BUG=655021

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

[commit-bot: I haz the power, 2016-10-18 03:21:48]

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