1294339 - Don't use nsIGfxInfo in CompositorD3D11

Status: RESOLVED FIXED

(Core :: Graphics, defect)

Comment 1

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

Attachment: patch

We use nsIGfxInfo to check whether we can use partial presentation, by matching the vendor ID to NVIDIA. But this information can be grabbed, more accurately (I hope), from DXGI.

Comment 2

[Matt Woodrow (:mattwoodrow)]

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

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

::: gfx/layers/d3d11/CompositorD3D11.cpp
@@ +476,5 @@
> +  if (gfxPrefs::PartialPresent() < 0) {
> +    return false;
> +  }
> +  if (DeviceManagerD3D11::Get()->IsWARP()) {
> +    return false;

return true?

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/6e3fb17d70ad
Don't use nsIGfxInfo in CompositorD3D11. (bug 1294339, r=mattwoodrow)

Comment 4

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

https://hg.mozilla.org/mozilla-central/rev/6e3fb17d70ad