Recreate MediaPermissionDispatcher's service connection on navigation

Recreate MediaPermissionDispatcher's service connection on navigation

In r470992 we started closing PermissionService connections on
navigation however the lifetime of a MediaPermissionDispatcher is that
of the frame it is attached to. Since frames can be navigated the
service connection must be re-established after it is closed.

To prevent a race between the browser-side close and the navigation
committing in the renderer we close the connection from that side as
well.

Bug: 725038
Change-Id: Id5d9d03537c7b7ee0997d77da4bafed055686521
Reviewed-on: https://chromium-review.googlesource.com/513387
Reviewed-by: Daniel Cheng <dcheng@chromium.org>;
Reviewed-by: Nick Carter <nick@chromium.org>;
Commit-Queue: Reilly Grant <reillyg@chromium.org>;
Cr-Original-Commit-Position: refs/heads/master@{#477363}
Review-Url: https://codereview.chromium.org/2931513003 .
Cr-Commit-Position: refs/branch-heads/3112@{#231}
Cr-Branched-From: b6460e24cf59f429d69de255538d0fc7a425ccf9-refs/heads/master@{#474897}
Committed: https://chromium.googlesource.com/chromium/src/+/4d2aa441f353f892cda7029b5bbd908777536010

[Reilly Grant (use Gerrit), 2017-06-07 19:58:40]

Description was changed from

==========
Recreate MediaPermissionDispatcher's service connection on navigation

In r470992 we started closing PermissionService connections on
navigation however the lifetime of a MediaPermissionDispatcher is that
of the frame it is attached to. Since frames can be navigated the
service connection must be re-established after it is closed.

To prevent a race between the browser-side close and the navigation
committing in the renderer we close the connection from that side as
well.

Bug: 725038
Change-Id: Id5d9d03537c7b7ee0997d77da4bafed055686521
Reviewed-on: https://chromium-review.googlesource.com/513387
Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Reviewed-by: Nick Carter <nick@chromium.org>
Commit-Queue: Reilly Grant <reillyg@chromium.org>
Cr-Commit-Position: refs/heads/master@{#477363}
(cherry picked from commit 355be367c161e11c42991b47f13ba4a915c9e283)
==========

to

==========
Recreate MediaPermissionDispatcher's service connection on navigation

In r470992 we started closing PermissionService connections on
navigation however the lifetime of a MediaPermissionDispatcher is that
of the frame it is attached to. Since frames can be navigated the
service connection must be re-established after it is closed.

To prevent a race between the browser-side close and the navigation
committing in the renderer we close the connection from that side as
well.

Bug: 725038
Change-Id: Id5d9d03537c7b7ee0997d77da4bafed055686521
Reviewed-on: https://chromium-review.googlesource.com/513387
Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Reviewed-by: Nick Carter <nick@chromium.org>
Commit-Queue: Reilly Grant <reillyg@chromium.org>
Cr-Original-Commit-Position: refs/heads/master@{#477363}
Review-Url: https://codereview.chromium.org/2931513003 .
Cr-Commit-Position: refs/branch-heads/3112@{#231}
Cr-Branched-From:
b6460e24cf59f429d69de255538d0fc7a425ccf9-refs/heads/master@{#474897}
Committed:
https://chromium.googlesource.com/chromium/src/+/4d2aa441f353f892cda7029b5bbd...
==========

[Reilly Grant (use Gerrit), 2017-06-07 19:58:41]

Committed patchset #1 (id:1) manually as
4d2aa441f353f892cda7029b5bbd908777536010.