MD Settings: Use FireWebUIListener() helper wherever possible.

MD Settings: Use FireWebUIListener() helper wherever possible.

BUG=None

Review-Url: https://codereview.chromium.org/2886083002
Cr-Commit-Position: refs/heads/master@{#473357}
Committed: https://chromium.googlesource.com/chromium/src/+/98ac5521519151b6cbcd4f501bf640f92e3a74e4

[dpapad, 2017-05-17 00:58:25]

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

[commit-bot: I haz the power, 2017-05-17 01:00:10]

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, 2017-05-17 01:07:35]

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

[commit-bot: I haz the power, 2017-05-17 01:07:36]

Dry run: Try jobs failed on following builders:
  android_arm64_dbg_recipe on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_arm6...)
  android_clang_dbg_recipe on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_clan...)
  android_compile_dbg on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_comp...)
  cast_shell_android on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/cast_shell_a...)
  chromeos_amd64-generic_chromium_compile_only_ng on
master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_amd64-...)
  linux_chromium_asan_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_compile_dbg_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_tsan_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[dpapad, 2017-05-17 01:32:32]

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

[commit-bot: I haz the power, 2017-05-17 01:35:37]

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, 2017-05-17 04:46:09]

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

[commit-bot: I haz the power, 2017-05-17 04:46:10]

Dry run: This issue passed the CQ dry run.

[dpapad, 2017-05-17 17:28:03]

dpapad@chromium.org changed reviewers:
+ dbeam@chromium.org

[Dan Beam, 2017-05-19 01:07:00]

rs lgtm

[dpapad, 2017-05-19 19:39:43]

https://codereview.chromium.org/2886083002/diff/20001/chrome/browser/ui/webui...
File chrome/browser/ui/webui/settings/settings_cookies_view_handler.cc (right):

https://codereview.chromium.org/2886083002/diff/20001/chrome/browser/ui/webui...
chrome/browser/ui/webui/settings/settings_cookies_view_handler.cc:111:
FireWebUIListener("on.TreeItemAdded", args);
Whoops. Something wrong with my regex capturing. I'll re-confirm manually that
all changes are sane.

[dpapad, 2017-05-19 20:35:35]

https://codereview.chromium.org/2886083002/diff/20001/chrome/browser/ui/webui...
File chrome/browser/ui/webui/settings/settings_cookies_view_handler.cc (right):

https://codereview.chromium.org/2886083002/diff/20001/chrome/browser/ui/webui...
chrome/browser/ui/webui/settings/settings_cookies_view_handler.cc:111:
FireWebUIListener("on.TreeItemAdded", args);
On 2017/05/19 at 19:39:43, dpapad wrote:
> Whoops. Something wrong with my regex capturing. I'll re-confirm manually that
all changes are sane.

Fixed. This was the only occurrence I found.

[dpapad, 2017-05-19 20:36:44]

The CQ bit was checked by dpapad@chromium.org

[dpapad, 2017-05-19 20:36:45]

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

[commit-bot: I haz the power, 2017-05-19 20:37:38]

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, 2017-05-19 22:16:56]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1495226204823110,
"parent_rev": "95c841ae862de3b08195fdb2e7ad2b212a19cc64", "commit_rev":
"98ac5521519151b6cbcd4f501bf640f92e3a74e4"}

[commit-bot: I haz the power, 2017-05-19 22:17:11]

Description was changed from

==========
MD Settings: Use FireWebUIListener() helper wherever possible.

BUG=None
==========

to

==========
MD Settings: Use FireWebUIListener() helper wherever possible.

BUG=None

Review-Url: https://codereview.chromium.org/2886083002
Cr-Commit-Position: refs/heads/master@{#473357}
Committed:
https://chromium.googlesource.com/chromium/src/+/98ac5521519151b6cbcd4f501bf6...
==========

[commit-bot: I haz the power, 2017-05-19 22:17:12]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/98ac5521519151b6cbcd4f501bf6...