Bug 1753565 - Default disable smooth scrolls if prefers-reduced-motion. r=botond

If the user has the prefers-reduced-motion system setting enabled, do
not enable smooth scrolls by default.

Differential Revision: https://phabricator.services.mozilla.com/D172446

UltraBlame original commit: a2f066123f3c6b76e43c5f09b154d838f1ba798b

Author: marco-c

gfx/layers/apz/test/gtest/TestBasic.cpp

@@ -310,6 +310,7 @@ TEST_F(APZCBasicTester, RelativeScrollOffset) {
 }
 
 TEST_F(APZCBasicTester, MultipleSmoothScrollsSmooth) {
+  SCOPED_GFX_PREF_BOOL("general.smoothScroll", true);
 
 
 
@@ -450,11 +451,13 @@ class APZCSmoothScrollTester : public APZCBasicTester {
 };
 
 TEST_F(APZCSmoothScrollTester, SmoothScrollDestinationUpdateBezier) {
+  SCOPED_GFX_PREF_BOOL("general.smoothScroll", true);
   SCOPED_GFX_PREF_BOOL("general.smoothScroll.msdPhysics.enabled", false);
   TestSmoothScrollDestinationUpdate();
 }
 
 TEST_F(APZCSmoothScrollTester, SmoothScrollDestinationUpdateMsd) {
+  SCOPED_GFX_PREF_BOOL("general.smoothScroll", true);
   SCOPED_GFX_PREF_BOOL("general.smoothScroll.msdPhysics.enabled", true);
   TestSmoothScrollDestinationUpdate();
 }

layout/base/nsLayoutUtils.cpp

@@ -85,6 +85,7 @@
 #include "mozilla/StaticPrefs_apz.h"
 #include "mozilla/StaticPrefs_dom.h"
 #include "mozilla/StaticPrefs_font.h"
+#include "mozilla/StaticPrefs_general.h"
 #include "mozilla/StaticPrefs_gfx.h"
 #include "mozilla/StaticPrefs_image.h"
 #include "mozilla/StaticPrefs_layers.h"
@@ -7770,6 +7771,22 @@ size_t nsLayoutUtils::SizeOfTextRunsForFrames(nsIFrame* aFrame,
 }
 
 
+void nsLayoutUtils::RecomputeSmoothScrollDefault() {
+  
+  
+  
+  Preferences::SetBool(
+      "general.smoothScroll",
+      !LookAndFeel::GetInt(LookAndFeel::IntID::PrefersReducedMotion, 0),
+      PrefValueKind::Default);
+}
+
+
+bool nsLayoutUtils::IsSmoothScrollingEnabled() {
+  return StaticPrefs::general_smoothScroll();
+}
+
+
 void nsLayoutUtils::Initialize() {
   nsComputedDOMStyle::RegisterPrefChangeCallbacks();
 }

layout/base/nsLayoutUtils.h

@@ -3039,6 +3039,19 @@ class nsLayoutUtils {
 
   static nsIFrame* GetNearestOverflowClipFrame(nsIFrame* aFrame);
 
+  
+
+
+  static bool IsSmoothScrollingEnabled();
+
+  
+
+
+
+
+
+  static void RecomputeSmoothScrollDefault();
+
  private:
 
 

layout/generic/nsGfxScrollFrame.cpp

@@ -2235,11 +2235,6 @@ void nsHTMLScrollFrame::AsyncScroll::InitSmoothScroll(
 }
 
 
-bool nsHTMLScrollFrame::IsSmoothScrollingEnabled() {
-  return StaticPrefs::general_smoothScroll();
-}
-
-
 
 
 
@@ -2560,7 +2555,8 @@ void nsHTMLScrollFrame::ScrollToWithOrigin(nsPoint aScrollPosition,
     mAsyncScroll->SetRefreshObserver(this);
   }
 
-  const bool isSmoothScroll = aParams.IsSmooth() && IsSmoothScrollingEnabled();
+  const bool isSmoothScroll =
+      aParams.IsSmooth() && nsLayoutUtils::IsSmoothScrollingEnabled();
   if (isSmoothScroll) {
     mAsyncScroll->InitSmoothScroll(now, GetScrollPosition(), mDestination,
                                    aParams.mOrigin, range, currentVelocity);
@@ -7903,7 +7899,7 @@ bool nsHTMLScrollFrame::IsSmoothScroll(dom::ScrollBehavior aBehavior) const {
 
 
   if (aBehavior == dom::ScrollBehavior::Instant ||
-      !nsHTMLScrollFrame::IsSmoothScrollingEnabled()) {
+      !nsLayoutUtils::IsSmoothScrollingEnabled()) {
     return false;
   }
 

layout/generic/nsGfxScrollFrame.h

@@ -538,8 +538,6 @@ class nsHTMLScrollFrame : public nsContainerFrame,
   void PostScrolledAreaEvent();
   MOZ_CAN_RUN_SCRIPT void FireScrolledAreaEvent();
 
-  static bool IsSmoothScrollingEnabled();
-
 
 
 

widget/nsXPLookAndFeel.cpp

@@ -17,6 +17,7 @@
 #include "nsFont.h"
 #include "nsIFrame.h"
 #include "nsIXULRuntime.h"
+#include "nsLayoutUtils.h"
 #include "Theme.h"
 #include "SurfaceCacheUtils.h"
 #include "mozilla/dom/ContentParent.h"
@@ -549,6 +550,10 @@ void nsXPLookAndFeel::Init() {
 
   RecomputeColorSchemes();
 
+  if (XRE_IsParentProcess()) {
+    nsLayoutUtils::RecomputeSmoothScrollDefault();
+  }
+
 
 
 
@@ -1170,8 +1175,9 @@ void nsXPLookAndFeel::RefreshImpl() {
   sIntCache.Clear();
   RecomputeColorSchemes();
 
-  
   if (XRE_IsParentProcess()) {
+    nsLayoutUtils::RecomputeSmoothScrollDefault();
+    
     widget::RemoteLookAndFeel::ClearCachedData();
   }
 }