Bug 1356751 - Remove unused pref layout.frame_rate.precise. r?mchang draft
authorMarkus Stange <mstange@themasta.com>
Sat, 15 Apr 2017 00:23:08 -0400
changeset 563175 2db91197138fafcab1bcb3094fe367145a143e2d
parent 563174 06589ba25f99cff2ba36e6a9cbaf0946b349c4e7
child 563196 883c6228fb8ad9748574794f2adf300aebd1775c
child 563221 3ed18ed014252fdb71ee1f45acfac69f11a2fb42
child 563222 17f74d2a87ca9fe19b0e5e625e06fa5d9974c527
push id54219
push userbmo:mstange@themasta.com
push dateSat, 15 Apr 2017 04:23:43 +0000
reviewersmchang
bugs1356751
milestone55.0a1
Bug 1356751 - Remove unused pref layout.frame_rate.precise. r?mchang MozReview-Commit-ID: 2QjinjJHzUh
b2g/app/b2g.js
modules/libpref/init/all.js
--- a/b2g/app/b2g.js
+++ b/b2g/app/b2g.js
@@ -409,25 +409,16 @@ pref("dom.mozApps.single_variant_sourced
 
 // controls if we want camera support
 pref("device.camera.enabled", true);
 pref("media.realtime_decoder.enabled", true);
 
 // TCPSocket
 pref("dom.mozTCPSocket.enabled", true);
 
-// "Preview" landing of bug 710563, which is bogged down in analysis
-// of talos regression.  This is a needed change for higher-framerate
-// CSS animations, and incidentally works around an apparent bug in
-// our handling of requestAnimationFrame() listeners, which are
-// supposed to enable this REPEATING_PRECISE_CAN_SKIP behavior.  The
-// secondary bug isn't really worth investigating since it's obseleted
-// by bug 710563.
-pref("layout.frame_rate.precise", true);
-
 // Handle hardware buttons in the b2g chrome package
 pref("b2g.keys.menu.enabled", true);
 
 // Display simulator software buttons
 pref("b2g.software-buttons", false);
 
 // Screen timeout in seconds
 pref("power.screen.timeout", 60);
--- a/modules/libpref/init/all.js
+++ b/modules/libpref/init/all.js
@@ -2895,28 +2895,16 @@ pref("layout.interruptible-reflow.enable
 // automatically based on knowledge of the platform (or 60Hz if no platform-
 // specific information is available).
 pref("layout.frame_rate", -1);
 
 // pref to dump the display list to the log. Useful for debugging drawing.
 pref("layout.display-list.dump", false);
 pref("layout.display-list.dump-content", false);
 
-// pref to control precision of the frame rate timer. When true,
-// we use a "precise" timer, which means each notification fires
-// Nms after the start of the last notification. That means if the
-// processing of the notification is slow, the timer can fire immediately
-// after we've just finished processing the last notification, which might
-// lead to starvation problems.
-// When false, we use a "slack" timer which fires Nms after the *end*
-// of the last notification. This can give less tight frame rates
-// but provides more time for other operations when the browser is
-// heavily loaded.
-pref("layout.frame_rate.precise", false);
-
 // pref to control whether layout warnings that are hit quite often are enabled
 pref("layout.spammy_warnings.enabled", false);
 
 // Should we fragment floats inside CSS column layout?
 pref("layout.float-fragments-inside-column.enabled", true);
 
 // The number of frames times the frame rate is the time required to
 // pass without painting used to guess that we'll not paint again soon