1294343 - Remove fallback-to-WARP in the compositor

Status: RESOLVED FIXED

(Core :: Graphics, defect)

Description

[David Anderson [:dvander] - inactive, e-mail if emergency]

Fallback-to-WARP is a little annoying to keep working with the GPU process. It's extra state to pass around everywhere, and the dynamism of it complicates decision making. We have never tested this configuration (it has shipped without any coverage, AFAIK) and it recently got disabled due to not performing well with video. WARP fallback was never more than about 1% of our D3D11 users anyway.

I'd like to just remove the fallback. For Gecko developers, we can keep a pref to force it on. Once the GPU process work has stabilized it might make sense to revisit WARP.

Comment 1

[David Anderson [:dvander] - inactive, e-mail if emergency]

Attachment: patch

Comment 2

[Jeff Muizelaar [:jrmuizel]]

Comment on attachment 8779973 [details] [diff] [review]
patch

Review of attachment 8779973 [details] [diff] [review]:
-----------------------------------------------------------------

The code changes are fine with me. If you haven't already checked with Bas, you should to make sure he has no objections to removing this code.

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/8d1cf3481e5f
Remove fallback to WARP, but keep the force pref for testing. (bug 1294343, r=jrmuizel)

Comment 4

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/8d1cf3481e5f