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)

enhancement

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: nobody → sotaro.ikeda.g
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
Status: NEW → RESOLVED
Closed: 11 months ago
Resolution: --- → FIXED
Target Milestone: --- → 126 Branch
Blocks: 1526207
Regressions: 1888384

(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
Blocks: 1889275
Regressions: 1890852
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: