Closed Bug 1476846 Opened 6 years ago Closed 6 years ago

Reduce latency of applying async images of video

Categories

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

enhancement

Tracking

()

RESOLVED FIXED
mozilla63
Tracking Status
firefox63 --- fixed

People

(Reporter: sotaro, Assigned: sotaro)

References

(Blocks 1 open bug)

Details

Attachments

(2 files, 1 obsolete file)

async images of video is applied with latency with async scene build. It is better to reduce the latency.
Depends on: 1475187
Assignee: nobody → sotaro.ikeda.g
Blocks: 1467768
Attachment #8993219 - Flags: review?(nical.bugzilla)
Attachment #8993219 - Flags: review?(nical.bugzilla) → review+
Rebased.
Attachment #8993219 - Attachment is obsolete: true
Attachment #8995854 - Flags: review+
Priority: -- → P3
Pushed by sikeda@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/738e88d3b78e Reduce latency of applying async images of video r=nical
Comment on attachment 8995854 [details] [diff] [review] patch - Reduce latency of applying async images of video Review of attachment 8995854 [details] [diff] [review]: ----------------------------------------------------------------- ::: gfx/layers/wr/AsyncImagePipelineManager.cpp @@ +355,5 @@ > // We may, however, have updated some resources. > + > + // Use transaction of scene builder thread to notify epoch. > + // It is for making epoc update consisitent. > + aMaybeFastTxn.UpdateEpoch(aPipelineId, aEpoch); This part doesn't seem right. The comment says you're using the scene builder transaction, but the code is using the fast transaction which goes on the RB thread. Also typos: s/epoc/epoch/ and s/consisitent/consistent/
^
Flags: needinfo?(sotaro.ikeda.g)
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla63
Thanks for checking!
Flags: needinfo?(sotaro.ikeda.g)
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Attachment #8998381 - Flags: review+
Status: REOPENED → RESOLVED
Closed: 6 years ago6 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: