1455715 - Fix missing double-bounce calls

Status: RESOLVED FIXED

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

Description

[Kartikaya Gupta (email:[e-mail])]

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 1

[Kartikaya Gupta (email:[e-mail])]

Attachment: Bug 1455715 - Fix some controller thread tasks that were bypassing the updater thread.

These tasks have an implicit ordering with other tasks that are
dispatched from the compositor thread to the updater thread, and so they
need to be bounced through the updater thread before we run them on the
controller thread.

Review commit: https://reviewboard.mozilla.org/r/238556/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/238556/

Comment 2

[Botond Ballo [:botond]]

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

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/fa90dad2faac
Fix some controller thread tasks that were bypassing the updater thread. r=botond

Comment 4

[Cosmin Sabou [:CosminS]]

https://hg.mozilla.org/mozilla-central/rev/fa90dad2faac