Revert of Move ResourceHandler deferred actions ahead of external protocol handling.

Revert of Move ResourceHandler deferred actions ahead of external protocol handling. (patchset #3 id:40001 of https://codereview.chromium.org/2574063003/ )

Reason for revert:
This patch made two layout tests TIMEOUT:

virtual/mojo-loading/http/tests/misc/redirect-to-about-blank.html
http/tests/misc/redirect-to-about-blank.html

Original issue's description:
> Move ResourceHandler deferred actions ahead of external protocol handling.
>
> While working on moving mixed-content checks to the browser
> (https://codereview.chromium.org/1905033002) I found out that external protocol
> handling happened too early in the processing of request start and redirects
> done by ResourceLoader; namely before ResourceHandler implementations are
> notified and able to do their work in deferred tasks.
>
> This CL changes that order to allow the upcoming mixed-content
> NavigationThrottle implementation to properly check the navigation start or
> redirect ahead of external protocol dispatching.
>
> For the record, the failing layout test that brought this change to light was:
> http/tests/security/mixedContent/nonwebby-scheme-in-iframe-allowed.https.html
>
> BUG=576270
>
> Committed: https://crrev.com/94bd59f28dcf81289f09c39d189128254631b275
> Cr-Commit-Position: refs/heads/master@{#439923}

TBR=jam@chromium.org,mmenke@chromium.org,carlosk@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=576270
Committed: https://crrev.com/1a7071bf34b7ee2375432347b4bda11ab1fed835
Cr-Commit-Position: refs/heads/master@{#439957}

[shimazu, 2016-12-21 00:47:28]

The CQ bit was checked by shimazu@chromium.org

[shimazu, 2016-12-21 00:47:29]

Created Revert of Move ResourceHandler deferred actions ahead of external
protocol handling.

[commit-bot: I haz the power, 2016-12-21 00:48:12]

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, 2016-12-21 00:49:33]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1482281248924580, "parent_rev":
"34700d7330e88ecbeb6165100e9861946226ee92", "commit_rev":
"ae0f5eecbec1a70fc88b432957877816c2a2006e"}

[commit-bot: I haz the power, 2016-12-21 00:50:02]

Description was changed from

==========
Revert of Move ResourceHandler deferred actions ahead of external protocol
handling. (patchset #3 id:40001 of https://codereview.chromium.org/2574063003/ )

Reason for revert:
This patch made two layout tests TIMEOUT:

virtual/mojo-loading/http/tests/misc/redirect-to-about-blank.html
http/tests/misc/redirect-to-about-blank.html

Original issue's description:
> Move ResourceHandler deferred actions ahead of external protocol handling.
> 
> While working on moving mixed-content checks to the browser
> (https://codereview.chromium.org/1905033002) I found out that external
protocol
> handling happened too early in the processing of request start and redirects
> done by ResourceLoader; namely before ResourceHandler implementations are
> notified and able to do their work in deferred tasks.
> 
> This CL changes that order to allow the upcoming mixed-content
> NavigationThrottle implementation to properly check the navigation start or
> redirect ahead of external protocol dispatching.
> 
> For the record, the failing layout test that brought this change to light was:
> http/tests/security/mixedContent/nonwebby-scheme-in-iframe-allowed.https.html
> 
> BUG=576270
> 
> Committed: https://crrev.com/94bd59f28dcf81289f09c39d189128254631b275
> Cr-Commit-Position: refs/heads/master@{#439923}

TBR=jam@chromium.org,mmenke@chromium.org,carlosk@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=576270
==========

to

==========
Revert of Move ResourceHandler deferred actions ahead of external protocol
handling. (patchset #3 id:40001 of https://codereview.chromium.org/2574063003/ )

Reason for revert:
This patch made two layout tests TIMEOUT:

virtual/mojo-loading/http/tests/misc/redirect-to-about-blank.html
http/tests/misc/redirect-to-about-blank.html

Original issue's description:
> Move ResourceHandler deferred actions ahead of external protocol handling.
> 
> While working on moving mixed-content checks to the browser
> (https://codereview.chromium.org/1905033002) I found out that external
protocol
> handling happened too early in the processing of request start and redirects
> done by ResourceLoader; namely before ResourceHandler implementations are
> notified and able to do their work in deferred tasks.
> 
> This CL changes that order to allow the upcoming mixed-content
> NavigationThrottle implementation to properly check the navigation start or
> redirect ahead of external protocol dispatching.
> 
> For the record, the failing layout test that brought this change to light was:
> http/tests/security/mixedContent/nonwebby-scheme-in-iframe-allowed.https.html
> 
> BUG=576270
> 
> Committed: https://crrev.com/94bd59f28dcf81289f09c39d189128254631b275
> Cr-Commit-Position: refs/heads/master@{#439923}

TBR=jam@chromium.org,mmenke@chromium.org,carlosk@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=576270

Review-Url: https://codereview.chromium.org/2597563002
==========

[commit-bot: I haz the power, 2016-12-21 00:50:02]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-12-21 00:53:25]

Description was changed from

==========
Revert of Move ResourceHandler deferred actions ahead of external protocol
handling. (patchset #3 id:40001 of https://codereview.chromium.org/2574063003/ )

Reason for revert:
This patch made two layout tests TIMEOUT:

virtual/mojo-loading/http/tests/misc/redirect-to-about-blank.html
http/tests/misc/redirect-to-about-blank.html

Original issue's description:
> Move ResourceHandler deferred actions ahead of external protocol handling.
> 
> While working on moving mixed-content checks to the browser
> (https://codereview.chromium.org/1905033002) I found out that external
protocol
> handling happened too early in the processing of request start and redirects
> done by ResourceLoader; namely before ResourceHandler implementations are
> notified and able to do their work in deferred tasks.
> 
> This CL changes that order to allow the upcoming mixed-content
> NavigationThrottle implementation to properly check the navigation start or
> redirect ahead of external protocol dispatching.
> 
> For the record, the failing layout test that brought this change to light was:
> http/tests/security/mixedContent/nonwebby-scheme-in-iframe-allowed.https.html
> 
> BUG=576270
> 
> Committed: https://crrev.com/94bd59f28dcf81289f09c39d189128254631b275
> Cr-Commit-Position: refs/heads/master@{#439923}

TBR=jam@chromium.org,mmenke@chromium.org,carlosk@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=576270

Review-Url: https://codereview.chromium.org/2597563002
==========

to

==========
Revert of Move ResourceHandler deferred actions ahead of external protocol
handling. (patchset #3 id:40001 of https://codereview.chromium.org/2574063003/ )

Reason for revert:
This patch made two layout tests TIMEOUT:

virtual/mojo-loading/http/tests/misc/redirect-to-about-blank.html
http/tests/misc/redirect-to-about-blank.html

Original issue's description:
> Move ResourceHandler deferred actions ahead of external protocol handling.
>
> While working on moving mixed-content checks to the browser
> (https://codereview.chromium.org/1905033002) I found out that external
protocol
> handling happened too early in the processing of request start and redirects
> done by ResourceLoader; namely before ResourceHandler implementations are
> notified and able to do their work in deferred tasks.
>
> This CL changes that order to allow the upcoming mixed-content
> NavigationThrottle implementation to properly check the navigation start or
> redirect ahead of external protocol dispatching.
>
> For the record, the failing layout test that brought this change to light was:
> http/tests/security/mixedContent/nonwebby-scheme-in-iframe-allowed.https.html
>
> BUG=576270
>
> Committed: https://crrev.com/94bd59f28dcf81289f09c39d189128254631b275
> Cr-Commit-Position: refs/heads/master@{#439923}

TBR=jam@chromium.org,mmenke@chromium.org,carlosk@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=576270

Committed: https://crrev.com/1a7071bf34b7ee2375432347b4bda11ab1fed835
Cr-Commit-Position: refs/heads/master@{#439957}
==========

[commit-bot: I haz the power, 2016-12-21 00:53:26]

Patchset 1 (id:??) landed as
https://crrev.com/1a7071bf34b7ee2375432347b4bda11ab1fed835
Cr-Commit-Position: refs/heads/master@{#439957}