We should not create CompositorVsyncScheduler in CompositorBridgeParent::Initialize().

WebRenderBridgeParent::WebRenderBridgeParent creates CompositorVsyncScheduler when webrender is enabled.

Comment on attachment 8814341 [details] [diff] [review] patch - Suppress CompositorVsyncScheduler creation Review of attachment 8814341 [details] [diff] [review]: ----------------------------------------------------------------- Maybe it would make sense to check the webrender pref instead of using #ifdefs? I assume we already have this pref, otherwise we'll have to add it since we'll need to be able to switch WR on and off without rebuilding firefox.

(In reply to Nicolas Silva [:nical] from comment #3) > > Maybe it would make sense to check the webrender pref instead of using > #ifdefs? I assume we already have this pref, otherwise we'll have to add it > since we'll need to be able to switch WR on and off without rebuilding > firefox. Oh, I forgot about it.

Pushed by sikeda@mozilla.com: https://hg.mozilla.org/projects/graphics/rev/53a19f05f274 Switch WebRender by Pref r=nical?

Please address vlad's comment at https://gfx.tacowolf.com/rQ53a19f05f274

Sorry, I am going to address it.

Pushed by sikeda@mozilla.com: https://hg.mozilla.org/projects/graphics/rev/ef45f3844967 Remove some MOZ_ENABLE_WEBRENDER r=nical?

Comment on attachment 8818477 [details] [diff] [review] patch - Remove some MOZ_ENABLE_WEBRENDER Moved a review to Phabricator.