1776530 - Minor APZUpdater cleanup

Status: RESOLVED FIXED

(Core :: Panning and Zooming, task, P3)

Comment 1

[Botond Ballo [:botond]]

Attachment: Bug 1776530 - Rename APZUpdater::UsingWebRenderUpdaterThread() to IsConnectedToWebRender(). r=tnikkel

The earlier name dates back to a time when we could use WebRender
but not necessarily use a WebRender thread as the updater thread.

Also add a comment to list the remaining situatins in which this
function can return false.

Comment 2

[Botond Ballo [:botond]]

Attachment: Bug 1776530 - Assert that APZUpdater::RunOnUpdaterThread() is not called directly from the updater thread. r=tnikkel

With the exception of the egde case scenarion in which IsUsingWebRender()
return false.

Also add a comment warning that if RunOnUpdaterThread() were to be called
directly from the updater thread when using WebRender, the implementation
would be incorrect.

Depends on D150308

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/6f6fcfd3c0f5
Rename APZUpdater::UsingWebRenderUpdaterThread() to IsConnectedToWebRender(). r=tnikkel
https://hg.mozilla.org/integration/autoland/rev/ec0bd8f37b9a
Assert that APZUpdater::RunOnUpdaterThread() is not called directly from the updater thread. r=tnikkel

Comment 4

[Natalia Csoregi [:nataliaCs]]

https://hg.mozilla.org/mozilla-central/rev/6f6fcfd3c0f5
https://hg.mozilla.org/mozilla-central/rev/ec0bd8f37b9a