Enforce OffscreenCanvas.convertToBlob to terminate after idling for too long

Enforce OffscreenCanvas.convertToBlob to terminate after idling for too long

BUG=657102
Review-Url: https://codereview.chromium.org/2613733002
Cr-Commit-Position: refs/heads/master@{#442059}
Committed: https://chromium.googlesource.com/chromium/src/+/449b31c06863f9d26120345bdcdfb72196aeeb87

[xlai (Olivia), 2017-01-04 21:34:13]

xlai@chromium.org changed reviewers:
+ junov@chromium.org

[xlai (Olivia), 2017-01-04 21:34:13]

OffscreenCanvas.convertToBlob() on main thread is just exactly as same as that
for canvas.toBlob().

For worker thread, what we need to do is just to make sure
that we remember to do a CrossThreadBind when encoding is
complete (or failed) and send the task back to main thread
for blob creation and promise resolve.
OffscreenCanvas-convertToBlob-noIdleTask-worker.html is testing that part of
code.

[Justin Novosad, 2017-01-06 15:49:21]

lgtm with nit

https://codereview.chromium.org/2613733002/diff/20001/third_party/WebKit/Layo...
File
third_party/WebKit/LayoutTests/fast/canvas/OffscreenCanvas-convertToBlob-noIdleTask-worker.html
(right):

https://codereview.chromium.org/2613733002/diff/20001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/fast/canvas/OffscreenCanvas-convertToBlob-noIdleTask-worker.html:17:
// In WebKit Layout Tests, flag "--enable-threaded-compositing", which is
same here

https://codereview.chromium.org/2613733002/diff/20001/third_party/WebKit/Layo...
File
third_party/WebKit/LayoutTests/fast/canvas/OffscreenCanvas-convertToBlob-noIdleTask.html
(right):

https://codereview.chromium.org/2613733002/diff/20001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/fast/canvas/OffscreenCanvas-convertToBlob-noIdleTask.html:4:
// In WebKit Layout Tests, flag "--enable-threaded-compositing", which is
nit: WebKit -> blink

[xlai (Olivia), 2017-01-06 19:34:51]

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

[commit-bot: I haz the power, 2017-01-06 19:35:27]

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-06 21:24:58]

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

[commit-bot: I haz the power, 2017-01-06 21:24:59]

Dry run: This issue passed the CQ dry run.

[xlai (Olivia), 2017-01-06 21:26:45]

The CQ bit was checked by xlai@chromium.org

[xlai (Olivia), 2017-01-06 21:26:46]

The patchset sent to the CQ was uploaded after l-g-t-m from junov@chromium.org
Link to the patchset: https://codereview.chromium.org/2613733002/#ps40001
(title: "rebase and fix based on junov feedback")

[commit-bot: I haz the power, 2017-01-06 21:27:07]

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-06 21:34:27]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1483738005216500,
"parent_rev": "18a3a8709bd95e5a90aa5892962790688e3620a5", "commit_rev":
"449b31c06863f9d26120345bdcdfb72196aeeb87"}

[commit-bot: I haz the power, 2017-01-06 21:34:54]

Description was changed from

==========
Enforce OffscreenCanvas.convertToBlob to terminate after idling for too long

BUG=657102
==========

to

==========
Enforce OffscreenCanvas.convertToBlob to terminate after idling for too long

BUG=657102

Review-Url: https://codereview.chromium.org/2613733002
Cr-Commit-Position: refs/heads/master@{#442059}
Committed:
https://chromium.googlesource.com/chromium/src/+/449b31c06863f9d26120345bdcdf...
==========

[commit-bot: I haz the power, 2017-01-06 21:34:55]

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