Closed Bug 1294339 Opened 4 years ago Closed 4 years ago

Don't use nsIGfxInfo in CompositorD3D11

Categories

(Core :: Graphics, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla51
Tracking Status
firefox51 --- fixed

People

(Reporter: dvander, Assigned: dvander)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

No description provided.
Attached patch patchSplinter Review
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.
Attachment #8779971 - Flags: review?(matt.woodrow)
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?
Attachment #8779971 - Flags: review?(matt.woodrow) → review+
Pushed by danderson@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/6e3fb17d70ad
Don't use nsIGfxInfo in CompositorD3D11. (bug 1294339, r=mattwoodrow)
https://hg.mozilla.org/mozilla-central/rev/6e3fb17d70ad
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla51
You need to log in before you can comment on or make changes to this bug.