GitLab

The HasSecurityState() check was introduced as a temporary workaround
to eliminate crashes caused by Mojo services trying to perform
ChildProcessSecurityPolicy security checks after the RenderProcessHost
has been destroyed.  HasSecurityState() was used to detect that the
process was gone and just fail open.

Now, we have a better solution in place with
ChildProcessSecurityPolicyImpl::Handle, which extends the lifetime of
a process's security state even after the RPH has gone away.  Switch
BroadcastChannelProvider to use this approach instead.

Bug: 943887
Change-Id: Id83c27364506ff9a349a10c5f54193c2aa0f369b
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2031924Reviewed-by: Marijn Kruisselbrink <mek@chromium.org>
Commit-Queue: Alex Moshchuk <alexmos@chromium.org>
Cr-Commit-Position: refs/heads/master@{#737179}