Remove usage of FOR_EACH_OBSERVER macro in ash/aura

Remove usage of FOR_EACH_OBSERVER macro in ash/aura

Observer lists now support range-based for loops.

BUG=655021
Committed: https://crrev.com/92ce34a01bdadbeeec6ed4469574351eae9a7764
Cr-Commit-Position: refs/heads/master@{#425459}

[Eric Willigers, 2016-10-14 05:34:19]

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

[commit-bot: I haz the power, 2016-10-14 05:34:38]

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 05:59:40]

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

[commit-bot: I haz the power, 2016-10-14 05:59:40]

Dry run: This issue passed the CQ dry run.

[Eric Willigers, 2016-10-14 06:50:20]

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

[Daniel Erat, 2016-10-14 15:18:20]

lgtm with a nit

https://codereview.chromium.org/2420863002/diff/1/ash/aura/wm_shell_aura.cc
File ash/aura/wm_shell_aura.cc (right):

https://codereview.chromium.org/2420863002/diff/1/ash/aura/wm_shell_aura.cc#n...
ash/aura/wm_shell_aura.cc:317: for (auto& observer : activation_observers_)
nit: you need to add curly brackets on this one

[Eric Willigers, 2016-10-14 19:55:09]

The CQ bit was checked by ericwilligers@chromium.org

[commit-bot: I haz the power, 2016-10-14 19:55:37]

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

[Eric Willigers, 2016-10-14 19:57:49]

The CQ bit was unchecked by ericwilligers@chromium.org

[Eric Willigers, 2016-10-14 20:16:50]

https://codereview.chromium.org/2420863002/diff/1/ash/aura/wm_shell_aura.cc
File ash/aura/wm_shell_aura.cc (right):

https://codereview.chromium.org/2420863002/diff/1/ash/aura/wm_shell_aura.cc#n...
ash/aura/wm_shell_aura.cc:317: for (auto& observer : activation_observers_)
On 2016/10/14 15:18:20, Daniel Erat wrote:
> nit: you need to add curly brackets on this one

Done.

[Eric Willigers, 2016-10-14 20:16:56]

The CQ bit was checked by ericwilligers@chromium.org

[Eric Willigers, 2016-10-14 20:16:57]

The patchset sent to the CQ was uploaded after l-g-t-m from derat@chromium.org
Link to the patchset: https://codereview.chromium.org/2420863002/#ps20001
(title: "braces")

[commit-bot: I haz the power, 2016-10-14 20:17:22]

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 20:49:32]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-10-14 20:51:33]

Description was changed from

==========
Remove usage of FOR_EACH_OBSERVER macro in ash/aura

Observer lists now support range-based for loops.


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

to

==========
Remove usage of FOR_EACH_OBSERVER macro in ash/aura

Observer lists now support range-based for loops.

BUG=655021

Committed: https://crrev.com/92ce34a01bdadbeeec6ed4469574351eae9a7764
Cr-Commit-Position: refs/heads/master@{#425459}
==========

[commit-bot: I haz the power, 2016-10-14 20:51:34]

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