Closed
Bug 1900632
Opened 8 months ago
Closed 8 months ago
Implement `VideoFrame.copyTo` for surfaces
Categories
(Core :: Audio/Video: Web Codecs, defect)
Core
Audio/Video: Web Codecs
Tracking
()
RESOLVED
FIXED
129 Branch
Tracking | Status | |
---|---|---|
firefox129 | --- | fixed |
People
(Reporter: padenot, Assigned: chunmin)
References
Details
Attachments
(4 files)
No description provided.
Assignee | ||
Comment 1•8 months ago
|
||
Assignee | ||
Comment 2•8 months ago
|
||
Depends on D212633
Assignee | ||
Comment 3•8 months ago
|
||
Depends on D212634
Assignee | ||
Comment 4•8 months ago
|
||
Previously, the copyto method for RGB types was only implemented for
images in the ImageFormat::MOZ2D_SURFACE
format. However, this
implementation can be applied to other formats as long as a
DataSourceSurface
can be obtained from the stored image. This patch
removes the the format check for MOZ2D_SURFACE
, allowing the
implementation to be used for other image types, such as
GPUVideoImage
.
Depends on D212635
Updated•8 months ago
|
Attachment #9405614 -
Attachment description: WIP: Bug 1900632 - Add error-level log macro in VideoFrFrame → WIP: Bug 1900632 - Add error-level log macro in VideoFrame
Updated•8 months ago
|
Attachment #9405616 -
Attachment description: WIP: Bug 1900632 - Generalize copyto implementation for RGB type → WIP: Bug 1900632 - Enable copyto for more image formats
Updated•8 months ago
|
Assignee: nobody → cchang
Attachment #9405613 -
Attachment description: WIP: Bug 1900632 - Expose GetSourceSurface from ImageConversion → Bug 1900632 - Expose GetSourceSurface from ImageConversion
Status: NEW → ASSIGNED
Updated•8 months ago
|
Attachment #9405614 -
Attachment description: WIP: Bug 1900632 - Add error-level log macro in VideoFrame → Bug 1900632 - Add error-level log macro in VideoFrame
Updated•8 months ago
|
Attachment #9405615 -
Attachment description: WIP: Bug 1900632 - Sort the include header in VideoFrame.cpp per coding style → Bug 1900632 - Sort the include header in VideoFrame.cpp per coding style
Updated•8 months ago
|
Attachment #9405616 -
Attachment description: WIP: Bug 1900632 - Enable copyto for more image formats → Bug 1900632 - Enable copyto for more image formats
Pushed by cchang@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/0e94a0eb26fd
Expose GetSourceSurface from ImageConversion r=media-playback-reviewers,padenot
https://hg.mozilla.org/integration/autoland/rev/f36150fbb0c5
Add error-level log macro in VideoFrame r=media-playback-reviewers,padenot
https://hg.mozilla.org/integration/autoland/rev/31876ded17ca
Sort the include header in VideoFrame.cpp per coding style r=media-playback-reviewers,padenot
https://hg.mozilla.org/integration/autoland/rev/9af665a32dd8
Enable copyto for more image formats r=media-playback-reviewers,padenot
Comment 6•8 months ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/0e94a0eb26fd
https://hg.mozilla.org/mozilla-central/rev/f36150fbb0c5
https://hg.mozilla.org/mozilla-central/rev/31876ded17ca
https://hg.mozilla.org/mozilla-central/rev/9af665a32dd8
Status: ASSIGNED → RESOLVED
Closed: 8 months ago
status-firefox129:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 129 Branch
You need to log in
before you can comment on or make changes to this bug.
Description
•