Remove stl_util's deletion function use from message center notifier settings.

Remove stl_util's deletion function use from message center notifier settings.

BUG=555865
Committed: https://crrev.com/77f2784e3217aba5f8cc05dfff5eebf88a5f1235
Cr-Commit-Position: refs/heads/master@{#438924}

[Avi (use Gerrit), 2016-12-14 17:31:55]

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

[commit-bot: I haz the power, 2016-12-14 17:33:13]

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-12-14 18:21:28]

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

[commit-bot: I haz the power, 2016-12-14 18:21:29]

Dry run: Try jobs failed on following builders:
  linux_chromium_chromeos_ozone_rel_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[Avi (use Gerrit), 2016-12-14 18:33:03]

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

[commit-bot: I haz the power, 2016-12-14 18:33:41]

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-12-14 19:48:08]

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

[commit-bot: I haz the power, 2016-12-14 19:48:09]

Dry run: This issue passed the CQ dry run.

[Avi (use Gerrit), 2016-12-14 21:05:29]

avi@chromium.org changed reviewers:
+ dewittj@chromium.org

[Avi (use Gerrit), 2016-12-14 21:05:30]

Please take a look. A bit of simplification too, along the way.

[dewittj, 2016-12-15 17:46:33]

lgtm, thanks!

https://codereview.chromium.org/2574583005/diff/20001/ui/message_center/views...
File ui/message_center/views/notifier_settings_view_unittest.cc (right):

https://codereview.chromium.org/2574583005/diff/20001/ui/message_center/views...
ui/message_center/views/notifier_settings_view_unittest.cc:137: for (auto*
button : buttons) {
can you use the actual type either here or for |buttons|?

[Avi (use Gerrit), 2016-12-15 19:17:51]

The CQ bit was checked by avi@chromium.org

[Avi (use Gerrit), 2016-12-15 19:17:51]

The patchset sent to the CQ was uploaded after l-g-t-m from dewittj@chromium.org
Link to the patchset: https://codereview.chromium.org/2574583005/#ps40001
(title: "nits")

[commit-bot: I haz the power, 2016-12-15 19:18:28]

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

[Avi (use Gerrit), 2016-12-15 19:23:29]

https://codereview.chromium.org/2574583005/diff/20001/ui/message_center/views...
File ui/message_center/views/notifier_settings_view_unittest.cc (right):

https://codereview.chromium.org/2574583005/diff/20001/ui/message_center/views...
ui/message_center/views/notifier_settings_view_unittest.cc:137: for (auto*
button : buttons) {
On 2016/12/15 17:46:33, dewittj wrote:
> can you use the actual type either here or for |buttons|?

Done.

[commit-bot: I haz the power, 2016-12-15 21:07:43]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1481829471349350,
"parent_rev": "427b5f5a4b8106cd2173a0c7c78326209eab9231", "commit_rev":
"8b56ca617fb863f27eb1832e64bbd936ebdf4054"}

[commit-bot: I haz the power, 2016-12-15 21:08:09]

Description was changed from

==========
Remove stl_util's deletion function use from message center notifier settings.

BUG=555865
==========

to

==========
Remove stl_util's deletion function use from message center notifier settings.

BUG=555865

Review-Url: https://codereview.chromium.org/2574583005
==========

[commit-bot: I haz the power, 2016-12-15 21:08:11]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-12-15 21:11:45]

Description was changed from

==========
Remove stl_util's deletion function use from message center notifier settings.

BUG=555865

Review-Url: https://codereview.chromium.org/2574583005
==========

to

==========
Remove stl_util's deletion function use from message center notifier settings.

BUG=555865

Committed: https://crrev.com/77f2784e3217aba5f8cc05dfff5eebf88a5f1235
Cr-Commit-Position: refs/heads/master@{#438924}
==========

[commit-bot: I haz the power, 2016-12-15 21:11:47]

Patchset 3 (id:??) landed as
https://crrev.com/77f2784e3217aba5f8cc05dfff5eebf88a5f1235
Cr-Commit-Position: refs/heads/master@{#438924}