Remove FOR_EACH_OBSERVER from autofill and password_manager components

Remove FOR_EACH_OBSERVER from autofill and password_manager components

According to
https://groups.google.com/a/chromium.org/d/msg/chromium-dev/udrb7JoPdiY/zxK09HZlAAAJ,
the FOR_EACH_OBSERVER macro is now deprecated and should be replaced by
range-based for loops.

This CL achieves that goal for the autofill and password_manager components.
The referenced bug also tracks adding a presubmit script to avoid regressions
in the future. The presubmit check is not part of this CL.

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

[vabr (Chromium), 2016-10-12 11:58:07]

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

[commit-bot: I haz the power, 2016-10-12 11:58:12]

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-12 12:45:58]

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

[commit-bot: I haz the power, 2016-10-12 12:45:59]

Dry run: This issue passed the CQ dry run.

[vabr (Chromium), 2016-10-12 12:57:55]

vabr@chromium.org changed reviewers:
+ sebsg@chromium.org

[vabr (Chromium), 2016-10-12 12:57:56]

Hello Seb,

Could you please review this refactoring? No functional change expected.

Cheers,
Vaclav

[sebsg, 2016-10-12 16:25:57]

Good to know! And thanks for fixing it. LGTM

[vabr (Chromium), 2016-10-13 09:12:10]

On 2016/10/12 16:25:57, sebsg wrote:
> Good to know! And thanks for fixing it. LGTM

Thanks!
Sending to CQ now.

Cheers,
Vaclav

[vabr (Chromium), 2016-10-13 09:12:14]

The CQ bit was checked by vabr@chromium.org

[commit-bot: I haz the power, 2016-10-13 09:12:32]

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-13 09:18:03]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-10-13 09:19:40]

Description was changed from

==========
Remove FOR_EACH_OBSERVER from autofill and password_manager components

According to
https://groups.google.com/a/chromium.org/d/msg/chromium-dev/udrb7JoPdiY/zxK09...,
the FOR_EACH_OBSERVER macro is now deprecated and should be replaced by
range-based for loops.

This CL achieves that goal for the autofill and password_manager components.
The referenced bug also tracks adding a presubmit script to avoid regressions
in the future. The presubmit check is not part of this CL.

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

to

==========
Remove FOR_EACH_OBSERVER from autofill and password_manager components

According to
https://groups.google.com/a/chromium.org/d/msg/chromium-dev/udrb7JoPdiY/zxK09...,
the FOR_EACH_OBSERVER macro is now deprecated and should be replaced by
range-based for loops.

This CL achieves that goal for the autofill and password_manager components.
The referenced bug also tracks adding a presubmit script to avoid regressions
in the future. The presubmit check is not part of this CL.

BUG=655021

Committed: https://crrev.com/d4f239a9796979beb7984664eac50b1c49bcfd14
Cr-Commit-Position: refs/heads/master@{#424987}
==========

[commit-bot: I haz the power, 2016-10-13 09:19:41]

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