1178753 - Remove the possibility to disable async-video

Status: RESOLVED FIXED

(Core :: Graphics: Layers, defect)

Description

[Nicolas Silva [:nical]]

We never test nor ship OMTC without async-video. By now there's a bunch of stuff that will break if we disable async-video, some important stuff in bug 1143575 seems to also rely on async-video (let alone canvas workers, etc.) let's just remove the pref and simplify the cod ethat depends on it.

Comment 1

[Milan Sreckovic [:milan] (needinfo for best results)]

You mean, tie the async video to OMTC usage, right?  So, we can still "disable" the async video by turning off OMTC.

Comment 2

[Nicolas Silva [:nical]]

(In reply to Milan Sreckovic [:milan] from comment #1)
> You mean, tie the async video to OMTC usage, right?  So, we can still
> "disable" the async video by turning off OMTC.

Yes, although that would be quite the risky thing to do at this point.

Comment 3

[Nicolas Silva [:nical]]

Attachment: remove the pref.

Comment 4

[Milan Sreckovic [:milan] (needinfo for best results)]

Release Note Request (optional, but appreciated)
I forget - do we release note preference removal?  This was not really something we encouraged people to change, and we have no telemetry as to how many people would have used it, but I thought I'd ask.

Comment 5

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/11ac079454e5

Comment 6

[Nicolas Silva [:nical]]

I don't think that this is relnote worthy, it hasn't changed the default configuration.

Comment 7

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/11ac079454e5

Comment 8

[Ritu Kothari (:ritu) (Inactive, please n-i to RyanVM, jcristau, or pascal)]

Rejecting relnote nom based on comment 6. Milan, please let me know if you have a concern.