Revert of Check for already existing entry when adding to SiteProcessMap.

Revert of Check for already existing entry when adding to SiteProcessMap. (patchset #1 id:1 of https://codereview.chromium.org/2755643003/ )

Reason for revert:
Test is flaking again, reverting the reland.

Original issue's description:
> Reland of Check for already existing entry when adding to SiteProcessMap. (patchset #1 id:1 of https://codereview.chromium.org/2749543004/ )
> 
> Reason for revert:
> Reverting the revert to see if this was indeed the culprit for the flakiness.
> 
> Original issue's description:
> > Revert 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
> 
> TBR=creis@chromium.org,rdevlin.cronin@chromium.org,tzik@chromium.org
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=701663
> 
> Review-Url: https://codereview.chromium.org/2755643003
> Cr-Commit-Position: refs/heads/master@{#457110}
> Committed: https://chromium.googlesource.com/chromium/src/+/0bcbed7d67f9a2813e1a278aada6c047a26bbbff

TBR=creis@chromium.org,rdevlin.cronin@chromium.org,tzik@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=701663
Review-Url: https://codereview.chromium.org/2756573002
Cr-Commit-Position: refs/heads/master@{#457211}
Committed: https://chromium.googlesource.com/chromium/src/+/7542ac912821eb3d5c939f17de433621d60f7fab

[nasko, 2017-03-15 21:16:51]

The CQ bit was checked by nasko@chromium.org

[nasko, 2017-03-15 21:16:52]

Created Revert of Check for already existing entry when adding to
SiteProcessMap.

[commit-bot: I haz the power, 2017-03-15 21:17:43]

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-03-15 21:20:04]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1489612611572430, "parent_rev":
"3e36a03e908bfc7bf9a4335611ae4789280c1506", "commit_rev":
"7542ac912821eb3d5c939f17de433621d60f7fab"}

[commit-bot: I haz the power, 2017-03-15 21:21:54]

Description was changed from

==========
Revert of Check for already existing entry when adding to SiteProcessMap.
(patchset #1 id:1 of https://codereview.chromium.org/2755643003/ )

Reason for revert:
Test is flaking again, reverting the reland.

Original issue's description:
> Reland of Check for already existing entry when adding to SiteProcessMap.
(patchset #1 id:1 of https://codereview.chromium.org/2749543004/ )
> 
> Reason for revert:
> Reverting the revert to see if this was indeed the culprit for the flakiness.
> 
> Original issue's description:
> > Revert 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/+/0e9d619f88ffabc3c237cbdf7560...
> > 
> > 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/+/96bb282f55edfa739e4e5f6633bb...
> 
> TBR=creis@chromium.org,rdevlin.cronin@chromium.org,tzik@chromium.org
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=701663
> 
> Review-Url: https://codereview.chromium.org/2755643003
> Cr-Commit-Position: refs/heads/master@{#457110}
> Committed:
https://chromium.googlesource.com/chromium/src/+/0bcbed7d67f9a2813e1a278aada6...

TBR=creis@chromium.org,rdevlin.cronin@chromium.org,tzik@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=701663
==========

to

==========
Revert of Check for already existing entry when adding to SiteProcessMap.
(patchset #1 id:1 of https://codereview.chromium.org/2755643003/ )

Reason for revert:
Test is flaking again, reverting the reland.

Original issue's description:
> Reland of Check for already existing entry when adding to SiteProcessMap.
(patchset #1 id:1 of https://codereview.chromium.org/2749543004/ )
> 
> Reason for revert:
> Reverting the revert to see if this was indeed the culprit for the flakiness.
> 
> Original issue's description:
> > Revert 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/+/0e9d619f88ffabc3c237cbdf7560...
> > 
> > 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/+/96bb282f55edfa739e4e5f6633bb...
> 
> TBR=creis@chromium.org,rdevlin.cronin@chromium.org,tzik@chromium.org
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=701663
> 
> Review-Url: https://codereview.chromium.org/2755643003
> Cr-Commit-Position: refs/heads/master@{#457110}
> Committed:
https://chromium.googlesource.com/chromium/src/+/0bcbed7d67f9a2813e1a278aada6...

TBR=creis@chromium.org,rdevlin.cronin@chromium.org,tzik@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=701663

Review-Url: https://codereview.chromium.org/2756573002
Cr-Commit-Position: refs/heads/master@{#457211}
Committed:
https://chromium.googlesource.com/chromium/src/+/7542ac912821eb3d5c939f17de43...
==========

[commit-bot: I haz the power, 2017-03-15 21:21:56]

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