html_viewer: Add a mock WebNotificationManager.

html_viewer: Add a mock WebNotificationManager.

Huffington Post websites check that they have permission to pop up
notifications. If we don't have this mock version, we crash.

BUG=none

Committed: https://crrev.com/473278a10064f7a6c682b210017f8158a7000f52
Cr-Commit-Position: refs/heads/master@{#325349}

[Elliot Glaysher, 2015-04-15 23:10:01]

erg@chromium.org changed reviewers:
+ sky@chromium.org

[Elliot Glaysher, 2015-04-15 23:10:01]

https://codereview.chromium.org/1093433002/diff/1/mojo/services/html_viewer/w...
File mojo/services/html_viewer/webnotificationmanager_impl.cc (right):

https://codereview.chromium.org/1093433002/diff/1/mojo/services/html_viewer/w...
mojo/services/html_viewer/webnotificationmanager_impl.cc:58: return
blink::WebNotificationPermission();
This is the function that gets called fairly consistently; we crash if we don't
have the object which implements this.

[sky, 2015-04-15 23:17:46]

LGTM

https://codereview.chromium.org/1093433002/diff/1/mojo/services/html_viewer/B...
File mojo/services/html_viewer/BUILD.gn (right):

https://codereview.chromium.org/1093433002/diff/1/mojo/services/html_viewer/B...
mojo/services/html_viewer/BUILD.gn:79: "webnotificationmanager_impl.cc",
I don't get our naming of these files. I would have expected
web_notification_manager_impl. And if we're ignoring caps, shouldn't it be
webnotificationmanagerimpl. But then the others follow the pattern you have
here. *SIGH*

https://codereview.chromium.org/1093433002/diff/1/mojo/services/html_viewer/w...
File mojo/services/html_viewer/webnotificationmanager_impl.h (right):

https://codereview.chromium.org/1093433002/diff/1/mojo/services/html_viewer/w...
mojo/services/html_viewer/webnotificationmanager_impl.h:38: };
private: DISALLOW?

[Elliot Glaysher, 2015-04-15 23:22:40]

The CQ bit was checked by erg@chromium.org

[Elliot Glaysher, 2015-04-15 23:22:40]

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

[commit-bot: I haz the power, 2015-04-15 23:23:29]

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

[commit-bot: I haz the power, 2015-04-15 23:49:13]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-04-15 23:54:59]

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