Backed out changeset ea2c1cabc853 (bug 1774849) for causing crashtest failures on 944353.jpg.

layout/base/PresShell.cpp

@@ -36,7 +36,6 @@
 #include "mozilla/StaticPrefs_apz.h"
 #include "mozilla/StaticPrefs_dom.h"
 #include "mozilla/StaticPrefs_font.h"
-#include "mozilla/StaticPrefs_image.h"
 #include "mozilla/StaticPrefs_layout.h"
 #include "mozilla/TextEvents.h"
 #include "mozilla/TimeStamp.h"
@@ -6421,10 +6420,8 @@ void PresShell::PaintInternal(nsView* aViewToPaint, PaintInternalFlags aFlags) {
 
   // We force sync-decode for printing / print-preview (printing already does
   // this from nsPageSequenceFrame::PrintNextSheet).
-  // We also force sync-decoding via pref for reftests.
   if (aFlags & PaintInternalFlags::PaintSyncDecodeImages ||
-      mDocument->IsStaticDocument() ||
-      StaticPrefs::image_decode_sync_enabled()) {
+      mDocument->IsStaticDocument()) {
     flags |= PaintFrameFlags::SyncDecodeImages;
   }
   if (renderer->GetBackendType() == layers::LayersBackend::LAYERS_WR) {

modules/libpref/init/StaticPrefList.yaml

@@ -6265,12 +6265,6 @@
   value: false
   mirror: always
 
-# Decode all images automatically on load, ignoring our normal heuristics.
-- name: image.decode-sync.enabled
-  type: bool
-  value: false
-  mirror: always
-
 # Whether we attempt to downscale images during decoding.
 - name: image.downscale-during-decode.enabled
   type: RelaxedAtomicBool

testing/profiles/reftest/user.js

@@ -45,9 +45,6 @@ user_pref("gfx.color_management.mode", 2);
 user_pref("gfx.logging.level", 1);
 // Disable downscale-during-decode, since it makes reftests more difficult.
 user_pref("image.downscale-during-decode.enabled", false);
-// We do the capturing the reftest results with sync decoding
-// so we want to be consistent.
-user_pref("image.decode-sync.enabled", true);
 // Disable interruptible reflow since (1) it's normally not going to
 // happen, but (2) it might happen if we somehow end up with both
 // pending user events and clock skew.  So to avoid having to change