Make FileReader.abort() (synchronously) follow the spec.

Make FileReader.abort() (synchronously) follow the spec.

It is problematic to cancel a ThreadableLoader (by way of FileReaderLoader)
while it is on the stack, which is one of the steps involved when
abort()ing a reader (as part of the "terminate" step.) To avoid such
potential trouble, the loader termination is done asynchronously.

However, there's no good reason to delay performing the other (user
visible) abort() steps, so arrange for that to happen and align with
the spec & others.

R=jsbell
BUG=288349
Committed: https://crrev.com/4557e0a6d697821badf2e8adb95d356a7cd85e62
Cr-Commit-Position: refs/heads/master@{#431639}

[sof, 2016-11-11 09:26:43]

The CQ bit was checked by sigbjornf@opera.com to run a CQ dry run

[commit-bot: I haz the power, 2016-11-11 09:26:58]

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

[sof, 2016-11-11 10:19:33]

sigbjornf@opera.com changed reviewers:
+ jsbell@chromium.org, nhiroki@chromium.org

[sof, 2016-11-11 10:19:33]

please take a look.

[commit-bot: I haz the power, 2016-11-11 10:39:26]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-11-11 10:39:27]

Dry run: This issue passed the CQ dry run.

[jsbell, 2016-11-11 16:56:05]

lgtm

[sof, 2016-11-11 20:43:46]

Description was changed from

==========
Make FileReader.abort() (synchronously) follow the spec.

It is problematic to cancel a ThreadableLoader (by way of FileReaderLoader)
while it is on the stack, which is one of the steps involved when
abort()ing a reader (as part of the "terminate" step.) To avoid such
potential trouble, the loader termination is done asynchronously.

However, there's no good reason to delay performing the other (user
visible) abort() steps, so arrange for that to happen and align with
the spec & others.

R=
BUG=288349
==========

to

==========
Make FileReader.abort() (synchronously) follow the spec.

It is problematic to cancel a ThreadableLoader (by way of FileReaderLoader)
while it is on the stack, which is one of the steps involved when
abort()ing a reader (as part of the "terminate" step.) To avoid such
potential trouble, the loader termination is done asynchronously.

However, there's no good reason to delay performing the other (user
visible) abort() steps, so arrange for that to happen and align with
the spec & others.

R=jsbell
BUG=288349
==========

[sof, 2016-11-11 20:43:52]

The CQ bit was checked by sigbjornf@opera.com

[commit-bot: I haz the power, 2016-11-11 20:44:47]

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-11-11 20:51:49]

Description was changed from

==========
Make FileReader.abort() (synchronously) follow the spec.

It is problematic to cancel a ThreadableLoader (by way of FileReaderLoader)
while it is on the stack, which is one of the steps involved when
abort()ing a reader (as part of the "terminate" step.) To avoid such
potential trouble, the loader termination is done asynchronously.

However, there's no good reason to delay performing the other (user
visible) abort() steps, so arrange for that to happen and align with
the spec & others.

R=jsbell
BUG=288349
==========

to

==========
Make FileReader.abort() (synchronously) follow the spec.

It is problematic to cancel a ThreadableLoader (by way of FileReaderLoader)
while it is on the stack, which is one of the steps involved when
abort()ing a reader (as part of the "terminate" step.) To avoid such
potential trouble, the loader termination is done asynchronously.

However, there's no good reason to delay performing the other (user
visible) abort() steps, so arrange for that to happen and align with
the spec & others.

R=jsbell
BUG=288349
==========

[commit-bot: I haz the power, 2016-11-11 20:51:51]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-11-11 21:00:46]

Description was changed from

==========
Make FileReader.abort() (synchronously) follow the spec.

It is problematic to cancel a ThreadableLoader (by way of FileReaderLoader)
while it is on the stack, which is one of the steps involved when
abort()ing a reader (as part of the "terminate" step.) To avoid such
potential trouble, the loader termination is done asynchronously.

However, there's no good reason to delay performing the other (user
visible) abort() steps, so arrange for that to happen and align with
the spec & others.

R=jsbell
BUG=288349
==========

to

==========
Make FileReader.abort() (synchronously) follow the spec.

It is problematic to cancel a ThreadableLoader (by way of FileReaderLoader)
while it is on the stack, which is one of the steps involved when
abort()ing a reader (as part of the "terminate" step.) To avoid such
potential trouble, the loader termination is done asynchronously.

However, there's no good reason to delay performing the other (user
visible) abort() steps, so arrange for that to happen and align with
the spec & others.

R=jsbell
BUG=288349

Committed: https://crrev.com/4557e0a6d697821badf2e8adb95d356a7cd85e62
Cr-Commit-Position: refs/heads/master@{#431639}
==========

[commit-bot: I haz the power, 2016-11-11 21:00:47]

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