Revert of Allow MojoAsyncResourceHandler::OnWillRead to complete asyncronously

Revert of Allow MojoAsyncResourceHandler::OnWillRead to complete asyncronously (patchset #5 id:80001 of https://codereview.chromium.org/2738973002/ )

Reason for revert:
This CL leads to a layout test crash.
virtual/mojo-loading/http/tests/serviceworker/chromium/request-body-blob-crash.html

Original issue's description:
> when a Mojo buffer can't be allocated synchronously.
> 
> The class still needs to create its own intermediary buffer
> when the Mojo buffer is too small to meet the
> MimeSniffingResourceHandler's needs, but once that's fixed,
> the class can be made significantly simpler.
> 
> BUG=687240
> 
> Review-Url: https://codereview.chromium.org/2738973002
> Cr-Commit-Position: refs/heads/master@{#458093}
> Committed: https://chromium.googlesource.com/chromium/src/+/2ba831790eccdf8fa9178eef72b8479a12a2f8aa

TBR=rdsmith@chromium.org,mmenke@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=687240, 703471
Review-Url: https://codereview.chromium.org/2760223002
Cr-Commit-Position: refs/heads/master@{#458297}
Committed: https://chromium.googlesource.com/chromium/src/+/4fede0f13bb7bd62971e98b966548953b56059aa

[yhirano, 2017-03-21 03:29:53]

The CQ bit was checked by yhirano@chromium.org

[yhirano, 2017-03-21 03:29:53]

Created Revert of Allow MojoAsyncResourceHandler::OnWillRead to complete
asyncronously

[yhirano, 2017-03-21 03:29:59]

The CQ bit was unchecked by yhirano@chromium.org

[yhirano, 2017-03-21 03:30:20]

Description was changed from

==========
Revert of Allow MojoAsyncResourceHandler::OnWillRead to complete asyncronously
(patchset #5 id:80001 of https://codereview.chromium.org/2738973002/ )

Reason for revert:
This CL leads to a layout test crash.
virtual/mojo-loading/http/tests/serviceworker/chromium/request-body-blob-crash.html

Original issue's description:
> when a Mojo buffer can't be allocated synchronously.
> 
> The class still needs to create its own intermediary buffer
> when the Mojo buffer is too small to meet the
> MimeSniffingResourceHandler's needs, but once that's fixed,
> the class can be made significantly simpler.
> 
> BUG=687240
> 
> Review-Url: https://codereview.chromium.org/2738973002
> Cr-Commit-Position: refs/heads/master@{#458093}
> Committed:
https://chromium.googlesource.com/chromium/src/+/2ba831790eccdf8fa9178eef72b8...

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

to

==========
Revert of Allow MojoAsyncResourceHandler::OnWillRead to complete asyncronously
(patchset #5 id:80001 of https://codereview.chromium.org/2738973002/ )

Reason for revert:
This CL leads to a layout test crash.
virtual/mojo-loading/http/tests/serviceworker/chromium/request-body-blob-crash.html

Original issue's description:
> when a Mojo buffer can't be allocated synchronously.
> 
> The class still needs to create its own intermediary buffer
> when the Mojo buffer is too small to meet the
> MimeSniffingResourceHandler's needs, but once that's fixed,
> the class can be made significantly simpler.
> 
> BUG=687240
> 
> Review-Url: https://codereview.chromium.org/2738973002
> Cr-Commit-Position: refs/heads/master@{#458093}
> Committed:
https://chromium.googlesource.com/chromium/src/+/2ba831790eccdf8fa9178eef72b8...

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

[yhirano, 2017-03-21 03:30:33]

The CQ bit was checked by yhirano@chromium.org

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

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-21 03:33:19]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1490067033010480, "parent_rev":
"fc06580c35d2418fb842879214e5732a5b8a32b7", "commit_rev":
"55a1b43db74d3bd40f93b5a1f71f26de0aedb667"}

[commit-bot: I haz the power, 2017-03-21 03:37:53]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1490067033010480, "parent_rev":
"c0689312b5fd92ae2325bdc4b219a931b0d5d831", "commit_rev":
"4fede0f13bb7bd62971e98b966548953b56059aa"}

[commit-bot: I haz the power, 2017-03-21 03:39:22]

Description was changed from

==========
Revert of Allow MojoAsyncResourceHandler::OnWillRead to complete asyncronously
(patchset #5 id:80001 of https://codereview.chromium.org/2738973002/ )

Reason for revert:
This CL leads to a layout test crash.
virtual/mojo-loading/http/tests/serviceworker/chromium/request-body-blob-crash.html

Original issue's description:
> when a Mojo buffer can't be allocated synchronously.
> 
> The class still needs to create its own intermediary buffer
> when the Mojo buffer is too small to meet the
> MimeSniffingResourceHandler's needs, but once that's fixed,
> the class can be made significantly simpler.
> 
> BUG=687240
> 
> Review-Url: https://codereview.chromium.org/2738973002
> Cr-Commit-Position: refs/heads/master@{#458093}
> Committed:
https://chromium.googlesource.com/chromium/src/+/2ba831790eccdf8fa9178eef72b8...

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

to

==========
Revert of Allow MojoAsyncResourceHandler::OnWillRead to complete asyncronously
(patchset #5 id:80001 of https://codereview.chromium.org/2738973002/ )

Reason for revert:
This CL leads to a layout test crash.
virtual/mojo-loading/http/tests/serviceworker/chromium/request-body-blob-crash.html

Original issue's description:
> when a Mojo buffer can't be allocated synchronously.
> 
> The class still needs to create its own intermediary buffer
> when the Mojo buffer is too small to meet the
> MimeSniffingResourceHandler's needs, but once that's fixed,
> the class can be made significantly simpler.
> 
> BUG=687240
> 
> Review-Url: https://codereview.chromium.org/2738973002
> Cr-Commit-Position: refs/heads/master@{#458093}
> Committed:
https://chromium.googlesource.com/chromium/src/+/2ba831790eccdf8fa9178eef72b8...

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

Review-Url: https://codereview.chromium.org/2760223002
Cr-Commit-Position: refs/heads/master@{#458297}
Committed:
https://chromium.googlesource.com/chromium/src/+/4fede0f13bb7bd62971e98b96654...
==========

[commit-bot: I haz the power, 2017-03-21 03:39:23]

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