Don't assume that the permission manager exists.

Don't assume that the permission manager exists.

It's valid for a browser context's permission manager to return NULL.
Handle this case in Push Messaging by immediately rejecting the request
instead.

BUG=695418
Review-Url: https://codereview.chromium.org/2706243007
Cr-Commit-Position: refs/heads/master@{#452489}
Committed: https://chromium.googlesource.com/chromium/src/+/ea60d1f2365d4715d25c4bdfb13c6749a2302a8f

[Sami, 2017-02-23 14:09:39]

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

[Sami, 2017-02-23 14:09:39]

lgtm

[Peter Beverloo, 2017-02-23 14:10:13]

The CQ bit was checked by peter@chromium.org

[commit-bot: I haz the power, 2017-02-23 14:10:25]

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-02-23 15:14:25]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1487859013030350, "parent_rev":
"2647340cca7d59e89a0e8392bf427135e478df31", "commit_rev":
"ea60d1f2365d4715d25c4bdfb13c6749a2302a8f"}

[commit-bot: I haz the power, 2017-02-23 15:15:02]

Description was changed from

==========
Don't assume that the permission manager exists.

It's valid for a browser context's permission manager to return NULL.
Handle this case in Push Messaging by immediately rejecting the request
instead.

BUG=695418
==========

to

==========
Don't assume that the permission manager exists.

It's valid for a browser context's permission manager to return NULL.
Handle this case in Push Messaging by immediately rejecting the request
instead.

BUG=695418

Review-Url: https://codereview.chromium.org/2706243007
Cr-Commit-Position: refs/heads/master@{#452489}
Committed:
https://chromium.googlesource.com/chromium/src/+/ea60d1f2365d4715d25c4bdfb13c...
==========

[commit-bot: I haz the power, 2017-02-23 15:15:03]

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