Revert the merge of CrossSiteResourceHandler and NavigationResourceThrottle.

Revert the merge of CrossSiteResourceHandler and NavigationResourceThrottle.

The merge between those two objects was done over more than one CL. The
goal of this CL is to revert all of the code at once, so it can be easily
relanded once the branch cut happens.

This CL reverts the following two:

Revert "Merge CrossSiteResourceHandler and NavigationResourceThrottle"
This reverts commit f40a0349f8575ccd544d6844dccd56119b87170c.

Revert "Prevent RFH from attempting to transfer after it's been swapped out"
This reverts commit 152381c57c5c04d07b9ebb2496e369ba239a98f7.

BUG=621856, 651503
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
R=creis@chromium.org
TBR=sky@chromium.org

Committed: https://chromium.googlesource.com/chromium/src/+/42ac3af0382dff192c3bc9f90f37a3982822bbb0

[nasko, 2016-10-05 20:51:32]

Description was changed from

==========
Revert the merge of CrossSiteResourceHandler and NavigationResourceThrottle.

The merge between those two objects was done over more than one CL. The
goal of this CL is to revert all of the code at once, so it can be easily
relanded once the branch cut happens.

This CL reverts the following two:

Revert "Merge CrossSiteResourceHandler and NavigationResourceThrottle"
This reverts commit f40a0349f8575ccd544d6844dccd56119b87170c.

Revert "Prevent RFH from attempting to transfer after it's been swapped out"
This reverts commit 152381c57c5c04d07b9ebb2496e369ba239a98f7.

BUG=621856, 651503
==========

to

==========
Revert the merge of CrossSiteResourceHandler and NavigationResourceThrottle.

The merge between those two objects was done over more than one CL. The
goal of this CL is to revert all of the code at once, so it can be easily
relanded once the branch cut happens.

This CL reverts the following two:

Revert "Merge CrossSiteResourceHandler and NavigationResourceThrottle"
This reverts commit f40a0349f8575ccd544d6844dccd56119b87170c.

Revert "Prevent RFH from attempting to transfer after it's been swapped out"
This reverts commit 152381c57c5c04d07b9ebb2496e369ba239a98f7.

BUG=621856, 651503
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
==========

[nasko, 2016-10-05 20:52:06]

The CQ bit was checked by nasko@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-10-05 20:52:34]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[nasko, 2016-10-05 20:54:59]

nasko@chromium.org changed reviewers:
+ creis@chromium.org, sky@chromium.org

[nasko, 2016-10-05 20:55:00]

Hey Charlie and Scott,
Can you review the revert of the CSRH removal work?
Thanks in advance!
Nasko

[nasko, 2016-10-05 22:21:08]

The CQ bit was checked by nasko@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-10-05 22:21:58]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Charlie Reis, 2016-10-05 22:29:33]

creis@chromium.org changed reviewers:
+ clamy@chromium.org

[Charlie Reis, 2016-10-05 22:29:34]

[+clamy for FYI]

LGTM, thanks!  We should be able to revert this CL right after the branch cut to
resume where we left off (i.e., fixing https://crbug.com/651503).  That'll give
this large functional change more time to bake before going to beta in M56,
which I think will be worthwhile.

https://codereview.chromium.org/2397893002/diff/1/content/browser/frame_host/...
File content/browser/frame_host/navigation_handle_impl.cc (left):

https://codereview.chromium.org/2397893002/diff/1/content/browser/frame_host/...
content/browser/frame_host/navigation_handle_impl.cc:89:
started_from_context_menu_(started_from_context_menu),
I think we need to keep this line.  (I see that you've fixed in PS2.  Thanks!)

[nasko, 2016-10-05 23:49:22]

Description was changed from

==========
Revert the merge of CrossSiteResourceHandler and NavigationResourceThrottle.

The merge between those two objects was done over more than one CL. The
goal of this CL is to revert all of the code at once, so it can be easily
relanded once the branch cut happens.

This CL reverts the following two:

Revert "Merge CrossSiteResourceHandler and NavigationResourceThrottle"
This reverts commit f40a0349f8575ccd544d6844dccd56119b87170c.

Revert "Prevent RFH from attempting to transfer after it's been swapped out"
This reverts commit 152381c57c5c04d07b9ebb2496e369ba239a98f7.

BUG=621856, 651503
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
==========

to

==========
Revert the merge of CrossSiteResourceHandler and NavigationResourceThrottle.

The merge between those two objects was done over more than one CL. The
goal of this CL is to revert all of the code at once, so it can be easily
relanded once the branch cut happens.

This CL reverts the following two:

Revert "Merge CrossSiteResourceHandler and NavigationResourceThrottle"
This reverts commit f40a0349f8575ccd544d6844dccd56119b87170c.

Revert "Prevent RFH from attempting to transfer after it's been swapped out"
This reverts commit 152381c57c5c04d07b9ebb2496e369ba239a98f7.

BUG=621856, 651503
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
TBR=sky@chromium.org
==========

[nasko, 2016-10-05 23:50:47]

The CQ bit was unchecked by nasko@chromium.org

[commit-bot: I haz the power, 2016-10-05 23:55:15]

Description was changed from

==========
Revert the merge of CrossSiteResourceHandler and NavigationResourceThrottle.

The merge between those two objects was done over more than one CL. The
goal of this CL is to revert all of the code at once, so it can be easily
relanded once the branch cut happens.

This CL reverts the following two:

Revert "Merge CrossSiteResourceHandler and NavigationResourceThrottle"
This reverts commit f40a0349f8575ccd544d6844dccd56119b87170c.

Revert "Prevent RFH from attempting to transfer after it's been swapped out"
This reverts commit 152381c57c5c04d07b9ebb2496e369ba239a98f7.

BUG=621856, 651503
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
TBR=sky@chromium.org
==========

to

==========
Revert the merge of CrossSiteResourceHandler and NavigationResourceThrottle.

The merge between those two objects was done over more than one CL. The
goal of this CL is to revert all of the code at once, so it can be easily
relanded once the branch cut happens.

This CL reverts the following two:

Revert "Merge CrossSiteResourceHandler and NavigationResourceThrottle"
This reverts commit f40a0349f8575ccd544d6844dccd56119b87170c.

Revert "Prevent RFH from attempting to transfer after it's been swapped out"
This reverts commit 152381c57c5c04d07b9ebb2496e369ba239a98f7.

BUG=621856, 651503
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
R=creis@chromium.org
TBR=sky@chromium.org

Committed: https://crrev.com/42ac3af0382dff192c3bc9f90f37a3982822bbb0
Cr-Commit-Position: refs/heads/master@{#423343}
==========

[commit-bot: I haz the power, 2016-10-05 23:55:17]

Patchset 2 (id:??) landed as
https://crrev.com/42ac3af0382dff192c3bc9f90f37a3982822bbb0
Cr-Commit-Position: refs/heads/master@{#423343}

[nasko, 2016-10-05 23:56:17]

Description was changed from

==========
Revert the merge of CrossSiteResourceHandler and NavigationResourceThrottle.

The merge between those two objects was done over more than one CL. The
goal of this CL is to revert all of the code at once, so it can be easily
relanded once the branch cut happens.

This CL reverts the following two:

Revert "Merge CrossSiteResourceHandler and NavigationResourceThrottle"
This reverts commit f40a0349f8575ccd544d6844dccd56119b87170c.

Revert "Prevent RFH from attempting to transfer after it's been swapped out"
This reverts commit 152381c57c5c04d07b9ebb2496e369ba239a98f7.

BUG=621856, 651503
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
R=creis@chromium.org
TBR=sky@chromium.org

Committed: https://crrev.com/42ac3af0382dff192c3bc9f90f37a3982822bbb0
Cr-Commit-Position: refs/heads/master@{#423343}
==========

to

==========
Revert the merge of CrossSiteResourceHandler and NavigationResourceThrottle.

The merge between those two objects was done over more than one CL. The
goal of this CL is to revert all of the code at once, so it can be easily
relanded once the branch cut happens.

This CL reverts the following two:

Revert "Merge CrossSiteResourceHandler and NavigationResourceThrottle"
This reverts commit f40a0349f8575ccd544d6844dccd56119b87170c.

Revert "Prevent RFH from attempting to transfer after it's been swapped out"
This reverts commit 152381c57c5c04d07b9ebb2496e369ba239a98f7.

BUG=621856, 651503
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
R=creis@chromium.org
TBR=sky@chromium.org

Committed:
https://chromium.googlesource.com/chromium/src/+/42ac3af0382dff192c3bc9f90f37...
==========

[nasko, 2016-10-05 23:56:19]

Committed patchset #2 (id:20001) manually as
42ac3af0382dff192c3bc9f90f37a3982822bbb0 (presubmit successful).