Commit 4ec17567 authored by Ray Kraesig's avatar Ray Kraesig
Browse files

Bug 1830792 - [1/4] Revert default to DXGI_SWAP_EFFECT_FLIP_SEQUENTIAL r=sotaro 

This reverts Mercurial commit ec948153333a957e3ef5302b5b0f469eda0d48a8
(introduced as part of bug 1820066) due to reports of tearing while
playing full-screen video.

Differential Revision: https://phabricator.services.mozilla.com/D176841
parent ec938bc5

gfx/thebes/gfxPlatform.cpp

+1 −9
Original line number Diff line number Diff line
@@ -2808,19 +2808,11 @@ void gfxPlatform::InitWebRenderConfig() { 
    gfxVars::SetReuseDecoderDevice(true);

  }



  // SWAP_EFFECT_FLIP_SEQUENTIAL is strongly recommended on Win10+ machines --

  // DXGI warnings are emitted by the Direct3D11 debug layer if it's not used.

  //

  // This avoids bug 1763981, which is only exhibited on Windows 11. (This may

  // be due to some sort of raciness between the parent and GPU processes when

  // no separate compositor-HWND is created. Alternatively, it may be a bug in

  // Win11 DWM's handling of SWAP_EFFECT_SEQUENTIAL.)

  if (Preferences::GetBool("gfx.webrender.flip-sequential", IsWin10OrLater())) {

  if (Preferences::GetBool("gfx.webrender.flip-sequential", false)) {

    if (gfxVars::UseWebRenderANGLE()) {

      gfxVars::SetUseWebRenderFlipSequentialWin(true);

    }

  }



  if (Preferences::GetBool("gfx.webrender.triple-buffering.enabled", false)) {

    if (gfxVars::UseWebRenderDCompWin() ||

        gfxVars::UseWebRenderFlipSequentialWin()) {

modules/libpref/init/all.js

+1 −1
Original line number Diff line number Diff line
@@ -457,7 +457,7 @@ pref("gfx.downloadable_fonts.disable_cache", false); 
pref("gfx.content.azure.backends", "skia");



#ifdef XP_WIN

  // pref("gfx.webrender.flip-sequential", false);  // no default

  pref("gfx.webrender.flip-sequential", false);

  pref("gfx.webrender.dcomp-win.enabled", true);

  pref("gfx.webrender.triple-buffering.enabled", true);

#endif