DescriptionRevert of Check for already existing entry when adding to SiteProcessMap. (patchset #7 id:160001 of https://codereview.chromium.org/2739193004/ )
Reason for revert:
Speculative revert to see a bot failure.
SubresourceFilterBrowserTest.FailedProvisionalLoadInMainframe got flaky around this CL, and this is relatively more suspicious in the failing range.
BUG=701663
Original issue's description:
> Check for already existing entry when adding to SiteProcessMap.
>
> The SiteProcessMap::RegisterProcess can overwrite existing entries
> in the map, which can lead to incorrect process being mapped to
> a site in process-per-site mode. This CL checks for existence and
> avoids the overwrite.
>
> BUG=700514
> CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
>
> Review-Url: https://codereview.chromium.org/2739193004
> Cr-Commit-Position: refs/heads/master@{#456920}
> Committed: https://chromium.googlesource.com/chromium/src/+/0e9d619f88ffabc3c237cbdf75603df8af9925e6
TBR=creis@chromium.org,rdevlin.cronin@chromium.org,nasko@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=700514
Review-Url: https://codereview.chromium.org/2749543004
Cr-Commit-Position: refs/heads/master@{#457010}
Committed: https://chromium.googlesource.com/chromium/src/+/96bb282f55edfa739e4e5f6633bb5fce61583e87
Patch Set 1 #
Messages
Total messages: 9 (4 generated)
|