Closed Bug 1455715 Opened 6 years ago Closed 6 years ago

Fix missing double-bounce calls

Categories

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

Other Branch
enhancement

Tracking

()

RESOLVED FIXED
mozilla61
Tracking Status
firefox61 --- fixed

People

(Reporter: kats, Assigned: kats)

References

Details

(Whiteboard: [gfx-noted])

Attachments

(1 file)

While debugging I realized that I missed a couple of places in [1] where we need to do the "double bounce". (Since that patch landed the sampler thread got split into the sampler and updater threads, and the double bounce now refers to bouncing off the updater thread onto the controller thread, so the commit message needs to be read with that in mind.)

[1] https://hg.mozilla.org/mozilla-central/rev/cac0b3a18cf94a2d3f2bd2223bef33009375235d
Comment on attachment 8969749 [details]
Bug 1455715 - Fix some controller thread tasks that were bypassing the updater thread.

https://reviewboard.mozilla.org/r/238556/#review244410
Attachment #8969749 - Flags: review?(botond) → review+
Pushed by kgupta@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/fa90dad2faac
Fix some controller thread tasks that were bypassing the updater thread. r=botond
https://hg.mozilla.org/mozilla-central/rev/fa90dad2faac
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla61
You need to log in before you can comment on or make changes to this bug.