Closed
Bug 1887818
Opened 11 months ago
Closed 11 months ago
Prevent to call GPUVideoImage::GetAsSourceSurface() in CanvasRenderingContext2D::DrawImage() for RemoteDecoderVideoSubDescriptor::Tnull_t if CanvasTranslator is used
Categories
(Core :: Graphics: Canvas2D, enhancement)
Core
Graphics: Canvas2D
Tracking
()
RESOLVED
FIXED
126 Branch
Tracking | Status | |
---|---|---|
firefox126 | --- | fixed |
People
(Reporter: sotaro, Assigned: sotaro)
References
(Blocks 1 open bug)
Details
(Keywords: perf-alert)
Attachments
(1 file)
GPUVideoImage::GetAsSourceSurface() is heavy weight. It seems better to prevent to call it if CanvasTranslator is used.
Assignee | ||
Updated•11 months ago
|
Assignee: nobody → sotaro.ikeda.g
Assignee | ||
Comment 1•11 months ago
|
||
Updated•11 months ago
|
Attachment #9393214 -
Attachment description: WIP: Bug 1887818 - Prevent to call GPUVideoImage::GetAsSourceSurface() in CanvasRenderingContext2D::DrawImage() for RemoteDecoderVideoSubDescriptor::Tnull_t if CanvasTranslator is used → Bug 1887818 - Prevent to call GPUVideoImage::GetAsSourceSurface() in CanvasRenderingContext2D::DrawImage() for RemoteDecoderVideoSubDescriptor::Tnull_t if CanvasTranslator is used
Pushed by sikeda.birchill@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/5343a8186c58
Prevent to call GPUVideoImage::GetAsSourceSurface() in CanvasRenderingContext2D::DrawImage() for RemoteDecoderVideoSubDescriptor::Tnull_t if CanvasTranslator is used r=gfx-reviewers,lsalzman
Comment 3•11 months ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 11 months ago
status-firefox126:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 126 Branch
Comment 4•11 months ago
|
||
(In reply to Pulsebot from comment #2)
Pushed by sikeda.birchill@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/5343a8186c58
Prevent to call GPUVideoImage::GetAsSourceSurface() in
CanvasRenderingContext2D::DrawImage() for
RemoteDecoderVideoSubDescriptor::Tnull_t if CanvasTranslator is used
r=gfx-reviewers,lsalzman
== Change summary for alert #42085 (as of Mon, 01 Apr 2024 18:36:02 GMT) ==
Improvements:
Ratio | Test | Platform | Options | Absolute values (old vs new) |
---|---|---|---|---|
67% | canvas2dvideo Mean tick time across 100 ticks: | linux1804-64-shippable-qr | e10s fission stylo webgl-ipc webrender | 12.36 -> 4.04 |
67% | canvas2dvideo Mean tick time across 100 ticks: | linux1804-64-qr | e10s fission stylo webgl-ipc webrender | 12.39 -> 4.06 |
67% | canvas2dvideo Mean tick time across 100 ticks: | linux1804-64-qr | e10s fission stylo webgl-ipc webrender | 12.21 -> 4.06 |
66% | canvas2dvideo Mean tick time across 100 ticks: | linux1804-64-shippable-qr | e10s fission stylo webrender | 12.05 -> 4.05 |
For up to date results, see: https://treeherder.mozilla.org/perfherder/alerts?id=42085
Keywords: perf-alert
You need to log in
before you can comment on or make changes to this bug.
Description
•