From e6cb5dba57482665ed1980d614c7db99cb83c1f2 Mon Sep 17 00:00:00 2001 From: Randall Barker Date: Wed, 3 Feb 2016 13:47:47 -0800 Subject: [PATCH] Bug 1245285 - part 3, Remove unused repaint_interval prefs r=botond --- gfx/layers/apz/src/AsyncPanZoomController.cpp | 14 -------------- gfx/thebes/gfxPrefs.h | 3 --- modules/libpref/init/all.js | 5 ----- 3 files changed, 22 deletions(-) diff --git a/gfx/layers/apz/src/AsyncPanZoomController.cpp b/gfx/layers/apz/src/AsyncPanZoomController.cpp index 1f44da531df..fb723fce17d 100644 --- a/gfx/layers/apz/src/AsyncPanZoomController.cpp +++ b/gfx/layers/apz/src/AsyncPanZoomController.cpp @@ -214,11 +214,6 @@ using mozilla::gfx::PointTyped; * value of this pref, and (t1 - t0) is the amount of time, in milliseconds, * that has elapsed between the two samples. * - * \li\b apz.fling_repaint_interval - * Maximum amount of time flinging before sending a viewport change. This will - * asynchronously repaint the page.\n - * Units: milliseconds - * * \li\b apz.fling_stop_on_tap_threshold * When flinging, if the velocity is above this number, then a tap on the * screen will stop the fling without dispatching a tap to content. If the @@ -282,18 +277,9 @@ using mozilla::gfx::PointTyped; * Units: screen pixels (for distance) * screen pixels per millisecond (for velocity) * - * \li\b apz.pan_repaint_interval - * Maximum amount of time while panning before sending a viewport change. This - * will asynchronously repaint the page. It is also forced when panning stops. - * * \li\b apz.record_checkerboarding * Whether or not to record detailed info on checkerboarding events. * - * \li\b apz.smooth_scroll_repaint_interval - * Maximum amount of time doing a smooth scroll before sending a viewport - * change. This will asynchronously repaint the page.\n - * Units: milliseconds - * * \li\b apz.test.logging_enabled * Enable logging of APZ test data (see bug 961289). * diff --git a/gfx/thebes/gfxPrefs.h b/gfx/thebes/gfxPrefs.h index 95af19c2485..a3f096fe76d 100644 --- a/gfx/thebes/gfxPrefs.h +++ b/gfx/thebes/gfxPrefs.h @@ -160,7 +160,6 @@ private: DECL_GFX_PREF(Once, "apz.fling_curve_function_y2", APZCurveFunctionY2, float, 1.0f); DECL_GFX_PREF(Live, "apz.fling_curve_threshold_inches_per_ms", APZCurveThreshold, float, -1.0f); DECL_GFX_PREF(Live, "apz.fling_friction", APZFlingFriction, float, 0.002f); - DECL_GFX_PREF(Live, "apz.fling_repaint_interval", APZFlingRepaintInterval, int32_t, 75); DECL_GFX_PREF(Live, "apz.fling_stop_on_tap_threshold", APZFlingStopOnTapThreshold, float, 0.05f); DECL_GFX_PREF(Live, "apz.fling_stopped_threshold", APZFlingStoppedThreshold, float, 0.01f); DECL_GFX_PREF(Live, "apz.highlight_checkerboarded_areas", APZHighlightCheckerboardedAreas, bool, false); @@ -175,10 +174,8 @@ private: DECL_GFX_PREF(Live, "apz.overscroll.stop_distance_threshold", APZOverscrollStopDistanceThreshold, float, 5.0f); DECL_GFX_PREF(Live, "apz.overscroll.stop_velocity_threshold", APZOverscrollStopVelocityThreshold, float, 0.01f); DECL_GFX_PREF(Live, "apz.overscroll.stretch_factor", APZOverscrollStretchFactor, float, 0.5f); - DECL_GFX_PREF(Live, "apz.pan_repaint_interval", APZPanRepaintInterval, int32_t, 250); DECL_GFX_PREF(Live, "apz.printtree", APZPrintTree, bool, false); DECL_GFX_PREF(Live, "apz.record_checkerboarding", APZRecordCheckerboarding, bool, false); - DECL_GFX_PREF(Live, "apz.smooth_scroll_repaint_interval", APZSmoothScrollRepaintInterval, int32_t, 75); DECL_GFX_PREF(Live, "apz.test.logging_enabled", APZTestLoggingEnabled, bool, false); DECL_GFX_PREF(Live, "apz.touch_move_tolerance", APZTouchMoveTolerance, float, 0.0); DECL_GFX_PREF(Live, "apz.touch_start_tolerance", APZTouchStartTolerance, float, 1.0f/4.5f); diff --git a/modules/libpref/init/all.js b/modules/libpref/init/all.js index e9f8b12c1bd..7ad0d7082f5 100644 --- a/modules/libpref/init/all.js +++ b/modules/libpref/init/all.js @@ -578,7 +578,6 @@ pref("apz.fling_curve_function_x2", "1.0"); pref("apz.fling_curve_function_y2", "1.0"); pref("apz.fling_curve_threshold_inches_per_ms", "-1.0"); pref("apz.fling_friction", "0.002"); -pref("apz.fling_repaint_interval", 16); pref("apz.fling_stop_on_tap_threshold", "0.05"); pref("apz.fling_stopped_threshold", "0.01"); pref("apz.highlight_checkerboarded_areas", false); @@ -593,7 +592,6 @@ pref("apz.overscroll.spring_stiffness", "0.0018"); pref("apz.overscroll.stop_distance_threshold", "5.0"); pref("apz.overscroll.stop_velocity_threshold", "0.01"); pref("apz.overscroll.stretch_factor", "0.35"); -pref("apz.pan_repaint_interval", 16); // Whether to print the APZC tree for debugging pref("apz.printtree", false); @@ -603,7 +601,6 @@ pref("apz.record_checkerboarding", true); #else pref("apz.record_checkerboarding", false); #endif -pref("apz.smooth_scroll_repaint_interval", 16); pref("apz.test.logging_enabled", false); pref("apz.touch_start_tolerance", "0.1"); pref("apz.touch_move_tolerance", "0.03"); @@ -621,8 +618,6 @@ pref("apz.zoom_animation_duration_ms", 250); // Mobile prefs pref("apz.allow_zooming", true); pref("apz.enlarge_displayport_when_clipped", true); -pref("apz.fling_repaint_interval", 75); -pref("apz.smooth_scroll_repaint_interval", 75); pref("apz.x_skate_size_multiplier", "1.25"); pref("apz.y_skate_size_multiplier", "1.5"); pref("apz.x_stationary_size_multiplier", "1.5");