Right now we read various silk related prefs in multiple locations. Consolidate them so it's easier to ensure silk is on/off.

Refactor checks for gfxPrefs::VsyncAlignedCompositor() to just check if the mCompositorVsyncObserver exists. Also nulls out the mCompositorVsyncObserver during CompositorParent initialization.

Replaces checks for gfxPrefs::TouchResampling() with sTouchDispatcher->mResamplingEnabled.

Comment on attachment 8558202 [details] [diff] [review] Part 1: Refactor checks for gfxPrefs::VsyncAlignedCompositor Review of attachment 8558202 [details] [diff] [review]: ----------------------------------------------------------------- Is something missing in this patch? It seems to make things dead code.

Comment on attachment 8558202 [details] [diff] [review] Part 1: Refactor checks for gfxPrefs::VsyncAlignedCompositor Review of attachment 8558202 [details] [diff] [review]: ----------------------------------------------------------------- Ok, the patch context was just insufficient. Pulling up the code for mCompositorVsyncObserver makes this clear.

https://hg.mozilla.org/integration/mozilla-inbound/rev/74ec30a5de72 https://hg.mozilla.org/integration/mozilla-inbound/rev/7a9bb617e461

https://hg.mozilla.org/mozilla-central/rev/74ec30a5de72 https://hg.mozilla.org/mozilla-central/rev/7a9bb617e461