Bug 1436642 - Make target element size larger to be unable to send transform animations to the compositor. r?jaws draft
authorHiroyuki Ikezoe <hikezoe@mozilla.com>
Sat, 10 Feb 2018 06:52:23 +0900
changeset 753257 e8ae1a530c2d6c65ff34b2d607ade0c07942ef97
parent 752727 c5120bcaf7bdcb5cdb06a02b60bd5bfe6a867d06
push id98537
push userhikezoe@mozilla.com
push dateFri, 09 Feb 2018 21:52:45 +0000
reviewersjaws
bugs1436642
milestone60.0a1
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