Revert "Replace security state workaround in CanAccessDataForOrigin()"

This reverts commit 263ce44bb356617e098d2cbad147146e0a36fc64.

Reason for revert: There are still renderer process terminations due to
this CL, see details in

Original change's description:
> Replace security state workaround in CanAccessDataForOrigin()
> - Replace workaround with code that is more strict about enforcing
>   security policy during child process shutdown. The old code would
>   always allow data access for IDs not in the security_state_ map. The
>   new code adds a pending map so we can deal with UI/IO thread races
>   during child process removal AND rejects any unknown IDs.
> - Fixed a test that depended on the old behavior where unknown IDs
>   always allowed access.
> Bug: 898281, 600441, 915203
> Change-Id: I26ca1e48536672b05d2310d8a17be47d5b6ef5c7
> Reviewed-on:
> Commit-Queue: Aaron Colwell <>
> Reviewed-by: Tom Sepez <>
> Reviewed-by: Alex Moshchuk <>
> Cr-Commit-Position: refs/heads/master@{#617937},,

# Not skipping CQ checks because original CL landed > 1 day ago.

Bug: 898281, 600441, 915203
Change-Id: I3fa9efdede12c4403ace0df0678049358009e4e4
Reviewed-by: Nasko Oskov <>
Commit-Queue: Nasko Oskov <>
Cr-Commit-Position: refs/heads/master@{#618700}
7 files changed