Bug 1436642 - Make target element size larger to be unable to send transform animations to the compositor. r=jaws
authorHiroyuki Ikezoe <hikezoe@mozilla.com>
Sat, 10 Feb 2018 06:52:23 +0900
changeset 403320 f5282133fa2088f320e3fb67ffdf27762950cc64
parent 403319 9d1ff0a1233976037991ebfe3d096b5e4fced071
child 403321 57f10073ac604636a144eff33f32cafaab724cd0
push id59436
push userhikezoe@mozilla.com
push dateSat, 10 Feb 2018 22:03:58 +0000
treeherderautoland@f5282133fa20 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersjaws
bugs1436642
milestone60.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1436642 - Make target element size larger to be unable to send transform animations to the compositor. r=jaws The original value were too small for Android. MozReview-Commit-ID: 4V6qC8orYNJ
dom/animation/test/chrome/test_animation_performance_warning.html
--- a/dom/animation/test/chrome/test_animation_performance_warning.html
+++ b/dom/animation/test/chrome/test_animation_performance_warning.html
@@ -1172,17 +1172,17 @@ function start() {
                                      { class: 'compositable' },
                                      { transform: [ 'translate(0px)',
                                                     'translate(100px)'] },
                                      100 * MS_PER_SEC);
     return waitForPaints().then(() => {
       assert_animation_property_state_equals(
         animation.effect.getProperties(),
         [ { property: 'transform', runningOnCompositor: true } ]);
-      animation.effect.target.style = 'width: 5200px; height: 5200px';
+      animation.effect.target.style = 'width: 10000px; height: 10000px';
       return waitForFrame();
     }).then(() => {
       // viewport depends on test environment.
       var expectedWarning = new RegExp(
         "Animation cannot be run on the compositor because the area of the frame " +
         "\\(\\d+\\) is too large relative to the viewport " +
         "\\(larger than \\d+\\)");
       assert_animation_property_state_equals(
@@ -1207,23 +1207,23 @@ function start() {
                                      { class: 'compositable' },
                                      { transform: [ 'translate(0px)',
                                                     'translate(100px)'] },
                                      100 * MS_PER_SEC);
     return waitForPaints().then(() => {
       assert_animation_property_state_equals(
         animation.effect.getProperties(),
         [ { property: 'transform', runningOnCompositor: true } ]);
-      animation.effect.target.style = 'width: 5200px; height: 1px';
+      animation.effect.target.style = 'width: 20000px; height: 1px';
       return waitForFrame();
     }).then(() => {
       // viewport depends on test environment.
       var expectedWarning = new RegExp(
         "Animation cannot be run on the compositor because the frame size " +
-        "\\(5200, 1\\) is too large relative to the viewport " +
+        "\\(20000, 1\\) is too large relative to the viewport " +
         "\\(larger than \\(\\d+, \\d+\\)\\) or larger than the " +
         "maximum allowed value \\(\\d+, \\d+\\)");
       assert_animation_property_state_equals(
         animation.effect.getProperties(),
         [ {
           property: 'transform',
           runningOnCompositor: false,
           warning: expectedWarning