Move WebAssociatedURLLoaderImpl's error timer to the frame's loading task runner.

Move WebAssociatedURLLoaderImpl's error timer to the frame's loading task runner.

If some user turns out to need these callbacks even when the frame is
suspended or destroyed, it should be moved instead to the general-purpose
loading task runner.

BUG=624694
Review-Url: https://codereview.chromium.org/2646893002
Cr-Commit-Position: refs/heads/master@{#444963}
Committed: https://chromium.googlesource.com/chromium/src/+/2ca43b1c99640f0578ba96397f820e1a0cfe7485

[jbroman, 2017-01-20 01:22:25]

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

[commit-bot: I haz the power, 2017-01-20 01:23:38]

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

[jbroman, 2017-01-20 01:54:15]

jbroman@chromium.org changed reviewers:
+ haraken@chromium.org

[jbroman, 2017-01-20 01:54:15]

I _think_ this ought to be fine, but some of the uses are sufficiently indirect
that I'm not 100% certain that there's no case that the error call is expected
when the frame is suspended/destroyed. If that bug is ever reported, though,
changing the task runner here should be straightforward.

WDYT?

[haraken, 2017-01-20 02:19:04]

First, this should be an Unspeced task. Second, UnspecedLoading makes more sense
than UnspecedTimer for this task. Hence, LGTM.

[jbroman, 2017-01-20 02:22:43]

The CQ bit was unchecked by jbroman@chromium.org

[jbroman, 2017-01-20 02:22:44]

The CQ bit was checked by jbroman@chromium.org

[commit-bot: I haz the power, 2017-01-20 02:22:51]

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-01-20 03:12:30]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1484878964790320, "parent_rev":
"5b078604227b8dd8864657954161097abc3091ee", "commit_rev":
"2ca43b1c99640f0578ba96397f820e1a0cfe7485"}

[commit-bot: I haz the power, 2017-01-20 03:13:07]

Description was changed from

==========
Move WebAssociatedURLLoaderImpl's error timer to the frame's loading task
runner.

If some user turns out to need these callbacks even when the frame is
suspended or destroyed, it should be moved instead to the general-purpose
loading task runner.

BUG=624694
==========

to

==========
Move WebAssociatedURLLoaderImpl's error timer to the frame's loading task
runner.

If some user turns out to need these callbacks even when the frame is
suspended or destroyed, it should be moved instead to the general-purpose
loading task runner.

BUG=624694

Review-Url: https://codereview.chromium.org/2646893002
Cr-Commit-Position: refs/heads/master@{#444963}
Committed:
https://chromium.googlesource.com/chromium/src/+/2ca43b1c99640f0578ba96397f82...
==========

[commit-bot: I haz the power, 2017-01-20 03:13:07]

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