Gracefully handle failed Mojo calls in the NotificationManager

Gracefully handle failed Mojo calls in the NotificationManager

This continues to be an unexpected failure so we keep the DCHECK, but
now with a predefined return value to handle this.

BUG=713545
Review-Url: https://codereview.chromium.org/2847433002
Cr-Commit-Position: refs/heads/master@{#467396}
Committed: https://chromium.googlesource.com/chromium/src/+/1fbce15166ac6fdbd967a1249375afdbd711de8e

[Peter Beverloo, 2017-04-26 14:40:03]

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

[commit-bot: I haz the power, 2017-04-26 14:40:21]

Dry run: CQ is trying da patch.

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

[Peter Beverloo, 2017-04-26 14:40:41]

peter@chromium.org changed reviewers:
+ awdf@chromium.org, elawrence@chromium.org

[Peter Beverloo, 2017-04-26 14:40:41]

+elawrence PTAL

[elawrence, 2017-04-26 14:58:14]

On 2017/04/26 14:40:41, Peter Beverloo wrote:
> +elawrence PTAL

LGTM.

[commit-bot: I haz the power, 2017-04-26 18:36:26]

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

[commit-bot: I haz the power, 2017-04-26 18:36:27]

Dry run: This issue passed the CQ dry run.

[Peter Beverloo, 2017-04-26 18:38:04]

The CQ bit was checked by peter@chromium.org

[commit-bot: I haz the power, 2017-04-26 18:38:53]

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-04-26 18:50:11]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1493231884229840, "parent_rev":
"2523efeb9a9d25f67c6d4799d0ca6687b674517f", "commit_rev":
"1fbce15166ac6fdbd967a1249375afdbd711de8e"}

[commit-bot: I haz the power, 2017-04-26 18:50:47]

Description was changed from

==========
Gracefully handle failed Mojo calls in the NotificationManager

This continues to be an unexpected failure so we keep the DCHECK, but
now with a predefined return value to handle this.

BUG=713545
==========

to

==========
Gracefully handle failed Mojo calls in the NotificationManager

This continues to be an unexpected failure so we keep the DCHECK, but
now with a predefined return value to handle this.

BUG=713545

Review-Url: https://codereview.chromium.org/2847433002
Cr-Commit-Position: refs/heads/master@{#467396}
Committed:
https://chromium.googlesource.com/chromium/src/+/1fbce15166ac6fdbd967a1249375...
==========

[commit-bot: I haz the power, 2017-04-26 18:50:48]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/1fbce15166ac6fdbd967a1249375...