New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 597240 link

Starred by 2 users

Issue metadata

Status: WontFix
Owner: ----
Closed: Aug 16
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 2
Type: Feature

Blocking:
issue 587871



Sign in to add a comment

Create a sub task runner interface for Blink

Project Member Reported by skyos...@chromium.org, Mar 23 2016

Issue description

Blink would like to have a sub task runner interface which:

- Is parented to a specific task queue, inheriting its settings.
- Can cancel all pending tasks.
- TBD: should we allow sub-sub-task runners?

This could be an instance of WebTaskRunner or a new interface which supports cancellation.

See https://codereview.chromium.org/1810513002/.
 

Comment 1 by phil...@opera.com, Mar 23 2016

Cc: mlamouri@chromium.org
This would allow us to replace GenericEventQueue in HTMLMediaElement and thereby get the correct ordering between events and other tasks. That's issue 587871.

Comment 2 by phil...@opera.com, Mar 23 2016

Blocking: 587871
It would be nice to use one of these for every task source in blink, which should help frame Blamer and potentially future user-agent interventions.
alexclarke@ and skyostill@, is this something your team might tackle? :)
Components: Blink>Media
Looks like this is on our Q2 list. Alex, it sounded like you were interested in owning this?
Labels: Needs-BlinkMediaTriage
Cc: -phil...@opera.com foolip@chromium.org
Labels: -Pri-3 -Needs-BlinkMediaTriage OS-All Pri-2
skyostill@, any news on this? :)
Not quite there yet, but some news:

1. There's now a TaskRunnerHelper which lets you get a MediaElementEvent task queue[1]. However for now that task queue falls back to a shared queue behind the covers. I'm imagining we'll add an API to create a task queue for a given task type next to this.

2. The scheduler is getting support for cancellable tasks any moment now, which is one of the requirements for a sub task queue.

3. The scheduler code has been moved into Blink, so making scheduler primitives oilpan'd is now tenable.

[1] https://cs.chromium.org/chromium/src/third_party/WebKit/Source/core/dom/TaskRunnerHelper.h?l=28&gs=cpp%253Ablink%253A%253Aenum-TaskType%253A%253AMediaElementEvent%2540chromium%252F..%252F..%252Fthird_party%252FWebKit%252FSource%252Fcore%252Fdom%252FTaskRunnerHelper.h%257Cdef&gsn=MediaElementEvent&ct=xref_usages
Project Member

Comment 10 by sheriffbot@chromium.org, Aug 21 2017

Labels: Hotlist-Recharge-Cold
Status: Untriaged (was: Available)
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue.

Sorry for the inconvenience if the bug really should have been left as Available. If you change it back, also remove the "Hotlist-Recharge-Cold" label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Status: Available (was: Untriaged)
I think we still want to consider this.
Cc: altimin@chromium.org
Status: WontFix (was: Available)
I don't think that we need this anymore — the part of comments #3 are addressed with capturing TaskTypes in the scheduler.

Sign in to add a comment