Move CanvasRenderingContext2D timer to frame-specific TaskRunnerTimer.

Move CanvasRenderingContext2D timer to frame-specific TaskRunnerTimer.

Move CanvasRenderingContext2D timers to frame-specific TaskRunnerTimer.
This associates it with the frame's CanvasBlobSerialization timer task queue.

The spec link: 
https://html.spec.whatwg.org/multipage/scripting.html#canvasrenderingcontext2d

This CL is migrating tasks/timers from a per-renderer scheduler to a per-frame
scheduler:
https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/80dsbEEOcWY

BUG=624694
Review-Url: https://codereview.chromium.org/2649783002
Cr-Commit-Position: refs/heads/master@{#448268}
Committed: https://chromium.googlesource.com/chromium/src/+/0a80c26dfe96dc2ac2233c0840fe10bbb3c67dad

[Byoungkwon Ko, 2017-01-22 21:28:48]

The CQ bit was checked by gogag2@gmail.com to run a CQ dry run

[commit-bot: I haz the power, 2017-01-22 21:28:52]

Dry run: 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-22 23:16:33]

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

[commit-bot: I haz the power, 2017-01-22 23:16:34]

Dry run: This issue passed the CQ dry run.

[Byoungkwon Ko, 2017-02-05 17:09:47]

Description was changed from

==========
Move CanvasRenderingContext2D timer to frame-specific TaskRunnerTimer.

Move CanvasRenderingContext2D timers to frame-specific TaskRunnerTimer.
This associates it with the frame's ----- timer task queue.
-----[1]

[1] The spec link: -----

This CL is migrating tasks/timers from a per-renderer scheduler to a per-frame
scheduler:
https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/80dsbEEOcWY

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

to

==========
Move CanvasRenderingContext2D timer to frame-specific TaskRunnerTimer.

Move CanvasRenderingContext2D timers to frame-specific TaskRunnerTimer.
This associates it with the frame's CanvasBlobSerialization timer task queue.

The spec link:
https://html.spec.whatwg.org/multipage/scripting.html#canvasrenderingcontext2d

This CL is migrating tasks/timers from a per-renderer scheduler to a per-frame
scheduler:
https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/80dsbEEOcWY

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

[Byoungkwon Ko, 2017-02-05 17:11:37]

The CQ bit was checked by gogag2@gmail.com to run a CQ dry run

[Byoungkwon Ko, 2017-02-05 17:11:46]

Description was changed from

==========
Move CanvasRenderingContext2D timer to frame-specific TaskRunnerTimer.

Move CanvasRenderingContext2D timers to frame-specific TaskRunnerTimer.
This associates it with the frame's CanvasBlobSerialization timer task queue.

The spec link:
https://html.spec.whatwg.org/multipage/scripting.html#canvasrenderingcontext2d

This CL is migrating tasks/timers from a per-renderer scheduler to a per-frame
scheduler:
https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/80dsbEEOcWY

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

to

==========
Move CanvasRenderingContext2D timer to frame-specific TaskRunnerTimer.

Move CanvasRenderingContext2D timers to frame-specific TaskRunnerTimer.
This associates it with the frame's CanvasBlobSerialization timer task queue.

The spec link: 
https://html.spec.whatwg.org/multipage/scripting.html#canvasrenderingcontext2d

This CL is migrating tasks/timers from a per-renderer scheduler to a per-frame
scheduler:
https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/80dsbEEOcWY

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

[commit-bot: I haz the power, 2017-02-05 17:11:50]

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

[Byoungkwon Ko, 2017-02-05 17:13:22]

gogag2@gmail.com changed reviewers:
+ haraken@chromium.org

[Byoungkwon Ko, 2017-02-05 17:13:23]

PTAL Haraken@

[commit-bot: I haz the power, 2017-02-05 18:47:26]

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

[commit-bot: I haz the power, 2017-02-05 18:47:27]

Dry run: This issue passed the CQ dry run.

[haraken, 2017-02-06 00:37:20]

I think we should use MiscPlatformAPI for this. With that change, LGTM.

- As far as I understand a discussion in
https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/CQIwXxKpO_Q,
the task source of the context lost/restore events is not yet defined in the
spec.

- CanvasBlobSerialization is a task source used in the toBlob() algorithm. It
looks wrong to use it for the context lost/restore events.

[Byoungkwon Ko, 2017-02-06 14:41:36]

On 2017/02/06 00:37:20, haraken wrote:
> I think we should use MiscPlatformAPI for this. With that change, LGTM.
> 
> - As far as I understand a discussion in
> https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/CQIwXxKpO_Q,
> the task source of the context lost/restore events is not yet defined in the
> spec.
> 
> - CanvasBlobSerialization is a task source used in the toBlob() algorithm. It
> looks wrong to use it for the context lost/restore events.

I changed it to use MiscPlatformAPI.
Thanks.

[Byoungkwon Ko, 2017-02-06 14:42:20]

The CQ bit was checked by gogag2@gmail.com

[Byoungkwon Ko, 2017-02-06 14:42:20]

The patchset sent to the CQ was uploaded after l-g-t-m from haraken@chromium.org
Link to the patchset: https://codereview.chromium.org/2649783002/#ps40001
(title: "Move CanvasRenderingContext2D timer to frame-specific
TaskRunnerTimer.")

[commit-bot: I haz the power, 2017-02-06 14:42:27]

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-02-06 16:23:52]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1486392140216440,
"parent_rev": "b81ab36ca2c7626767bf0fc13fb5960cea764a47", "commit_rev":
"0a80c26dfe96dc2ac2233c0840fe10bbb3c67dad"}

[commit-bot: I haz the power, 2017-02-06 16:24:21]

Description was changed from

==========
Move CanvasRenderingContext2D timer to frame-specific TaskRunnerTimer.

Move CanvasRenderingContext2D timers to frame-specific TaskRunnerTimer.
This associates it with the frame's CanvasBlobSerialization timer task queue.

The spec link: 
https://html.spec.whatwg.org/multipage/scripting.html#canvasrenderingcontext2d

This CL is migrating tasks/timers from a per-renderer scheduler to a per-frame
scheduler:
https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/80dsbEEOcWY

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

to

==========
Move CanvasRenderingContext2D timer to frame-specific TaskRunnerTimer.

Move CanvasRenderingContext2D timers to frame-specific TaskRunnerTimer.
This associates it with the frame's CanvasBlobSerialization timer task queue.

The spec link: 
https://html.spec.whatwg.org/multipage/scripting.html#canvasrenderingcontext2d

This CL is migrating tasks/timers from a per-renderer scheduler to a per-frame
scheduler:
https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/80dsbEEOcWY

BUG=624694

Review-Url: https://codereview.chromium.org/2649783002
Cr-Commit-Position: refs/heads/master@{#448268}
Committed:
https://chromium.googlesource.com/chromium/src/+/0a80c26dfe96dc2ac2233c0840fe...
==========

[commit-bot: I haz the power, 2017-02-06 16:24:22]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/0a80c26dfe96dc2ac2233c0840fe...