Closed Bug 1414727 Opened 8 years ago Closed 8 years ago

Reduce DisplayList update in AsyncImagePipelineManager

Categories

(Core :: Graphics: WebRender, enhancement, P1)

Product:
Core
Component:
Graphics: WebRender
Type:
enhancement

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla59
Tracking Flags:
Tracking Status
firefox57 --- unaffected
firefox58 --- unaffected
firefox59 --- fixed

People

(Reporter: sotaro, Assigned: sotaro)

References

Details

(Whiteboard: [wr-mvp])

Attachments

(1 file)

patch - Change AsyncImagePipeline::mIsChanged flag handling as to reduce DisplayList update
2.50 KB, patch
nical
: review+
Details | Diff | Splinter Review
Bug 1412755 fixed video and canvas layout related bug, but with the fix, the change increased updating display lists. It is nice if we could reduce the updating display list if it is not necessary.
Whiteboard: [wr-mvp] [triage]
Blocks: stage-wr-trains
status-firefox57: --- → unaffected
status-firefox58: --- → unaffected
Whiteboard: [wr-mvp] [triage] → [wr-mvp] [triage][wr-reserve-candidate]
Whiteboard: [wr-mvp] [triage][wr-reserve-candidate] → [wr-reserve]
Priority: -- → P3
Blocks: 1412755
It is low hanging fruit and we could reduce unnecessary webrender task.
Status: NEW → ASSIGNED
Priority: P3 → P1
Whiteboard: [wr-reserve] → [wr-mvp]
Attachment #8928050 - Flags: review?(nical.bugzilla)
Attachment #8928050 - Flags: review?(nical.bugzilla) → review+
Pushed by sikeda@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/84426b967192 Change AsyncImagePipeline::mIsChanged flag handling as to reduce DisplayList update r=nical
https://hg.mozilla.org/mozilla-central/rev/84426b967192
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
status-firefox59: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla59
Depends on: 1418877
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: