Implement ImageDecoder
Categories
(Core :: Audio/Video: Web Codecs, enhancement, P1)
Tracking
()
Tracking | Status | |
---|---|---|
firefox130 | --- | fixed |
People
(Reporter: chunmin, Assigned: aosmond)
References
(Depends on 1 open bug, Blocks 2 open bugs)
Details
Attachments
(2 files)
Reporter | ||
Updated•2 years ago
|
Updated•2 years ago
|
Reporter | ||
Updated•1 year ago
|
Reporter | ||
Updated•1 year ago
|
Assignee | ||
Comment 2•3 months ago
|
||
Update, 2/3 of tests passing, patches coming soon.
Assignee | ||
Updated•3 months ago
|
Assignee | ||
Comment 3•3 months ago
|
||
This patch adds convenience methods to throw on ErrorResult and
reject on Promise directly from a MediaResult which has an nsresult
and a message.
Assignee | ||
Comment 4•3 months ago
|
||
This leverages the promise based image decoding API added in bug
1901078 to remap that onto the WebCodecs specification.
Pushed by aosmond@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/0b609b8fcff3 Part 1. Expose methods to throw/reject based on a MediaResult. r=media-playback-reviewers,padenot https://hg.mozilla.org/integration/autoland/rev/0c7ce075a24f Part 2. Implement WebCodecs image decoding API. r=webidl,media-playback-reviewers,smaug,padenot
Comment 6•27 days ago
•
|
||
Backed out for causing interface related mochitests failures.
- Backout link
- Push with failures
- Failure Log
- Failure line: TEST-UNEXPECTED-FAIL | dom/workers/test/test_worker_interfaces_secureContext.html | undefined: If this is failing: DANGER, are you sure you want to expose the new interface ImageDecoder to all webpages as a property of self? Do not make a change to this file without a review from a DOM peer for that specific change!!! (or a JS peer for changes to ecmaGlobals)
Also, please check these mochitests too.
Assignee | ||
Updated•27 days ago
|
Pushed by aosmond@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/6aa4e953e922 Part 1. Expose methods to throw/reject based on a MediaResult. r=media-playback-reviewers,padenot https://hg.mozilla.org/integration/autoland/rev/40ba80bacf94 Part 2. Implement WebCodecs image decoding API. r=webidl,media-playback-reviewers,smaug,padenot
Comment 8•27 days ago
•
|
||
Backed out 3 changesets (Bug 1901078, Bug 1749048) for causing bustages in ImageDecoder.cpp
Push with failures: https://treeherder.mozilla.org/jobs?repo=autoland&group_state=expanded&resultStatus=testfailed%2Cbusted%2Cexception&searchStr=linux%2Cx64%2Cdebug%2Chazard-linux64-haz%2Fdebug%2Ch&revision=cbe411e8ee43bc08773dcaead3ca9b64c3843189&selectedTaskRun=ZsrmJATMRPWba3Gi7W0OOw.0
Please check previous comment with the Later Edit
Log: https://treeherder.mozilla.org/logviewer?job_id=467642705&repo=autoland&lineNumber=108613
Backout: https://hg.mozilla.org/integration/autoland/rev/9aaf78863f45f028304432737b4e28c63794e109
Assignee | ||
Comment 9•27 days ago
|
||
The easiest way to work around this for now is to just switch to ProcessFixedData
. It incurs an extra copy but we already do it in other gfx contexts. I can do a follow up later to improve this.
Assignee | ||
Comment 10•27 days ago
|
||
Comment 11•27 days ago
|
||
Pushed by aosmond@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/38ee84f0cb41 Part 1. Expose methods to throw/reject based on a MediaResult. r=media-playback-reviewers,padenot https://hg.mozilla.org/integration/autoland/rev/ad42f737c988 Part 2. Implement WebCodecs image decoding API. r=webidl,media-playback-reviewers,smaug,padenot
Comment 12•27 days ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/38ee84f0cb41
https://hg.mozilla.org/mozilla-central/rev/ad42f737c988
Description
•