Provide default implementations for NotificationDelegate methods.

Provide default implementations for NotificationDelegate methods.

Many implementations of the NotificationDelegate include empty overrides
because of behavior they don't actually need to be informed of. Provide
empty default implementations so that they don't have to. (And remove the
associated TODO.)

BUG=428370
Committed: https://crrev.com/cf29ede833a3acda53607f9050612bafe52f2321
Cr-Commit-Position: refs/heads/master@{#302190}

[Peter Beverloo, 2014-10-29 17:36:44]

peter@chromium.org changed reviewers:
+ dewittj@chromium.org, mukai@chromium.org

[Peter Beverloo, 2014-10-29 17:36:44]

WDYT? Mike left a TODO about this, and given the large number of empty
implementations I agree.

[dewittj, 2014-10-29 17:51:30]

lgtm

[Peter Beverloo, 2014-10-29 20:04:22]

peter@chromium.org changed reviewers:
+ stevenjb@chromium.org, thestig@chromium.org

[Peter Beverloo, 2014-10-29 20:04:22]

+stevenjb for ash/system/
+thestig for chrome/

[stevenjb, 2014-10-29 20:06:21]

ash/system lgtm

[Lei Zhang, 2014-10-29 22:20:44]

chrome/ lgtm

[Peter Beverloo, 2014-10-30 09:52:09]

The CQ bit was checked by peter@chromium.org

[commit-bot: I haz the power, 2014-10-30 09:52:28]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/684253002/1

[commit-bot: I haz the power, 2014-10-30 09:55:33]

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

[commit-bot: I haz the power, 2014-10-30 09:55:35]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)
  linux_chromium_asan_rel on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[Peter Beverloo, 2014-10-30 11:32:29]

The CQ bit was checked by peter@chromium.org

[commit-bot: I haz the power, 2014-10-30 11:33:30]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/684253002/20001

[commit-bot: I haz the power, 2014-10-30 11:36:57]

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

[commit-bot: I haz the power, 2014-10-30 11:36:58]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[Peter Beverloo, 2014-10-30 12:42:07]

+mukai for ash/display/

[Peter Beverloo, 2014-10-30 22:00:18]

peter@chromium.org changed reviewers:
+ jamescook@chromium.org

[Peter Beverloo, 2014-10-30 22:00:19]

+James for ash/display stamp? :-)

[Daniel Erat, 2014-10-30 22:20:48]

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

[Daniel Erat, 2014-10-30 22:20:48]

lgtm for ash/display

[Peter Beverloo, 2014-10-30 22:34:19]

On 2014/10/30 22:20:48, Daniel Erat wrote:
> lgtm for ash/display

Thanks Daniel!

[Peter Beverloo, 2014-10-30 22:34:27]

The CQ bit was checked by peter@chromium.org

[commit-bot: I haz the power, 2014-10-30 22:36:09]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/684253002/20001

[commit-bot: I haz the power, 2014-10-30 23:19:31]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2014-10-30 23:20:23]

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