Bug 1241692 - Part 2: We should observe stylings only for animations. r=dholbert
authorHiroyuki Ikezoe <hiikezoe@mozilla-japan.org>
Sat, 06 Feb 2016 20:38:24 +0900
changeset 283395 69f7ba132363aa783954f185a5c88b16582da924
parent 283394 df927d7001cdb96045f0fb4cc2f7a4415895e002
child 283396 7d6e0141de4fab594b48d54663a4cc90fc78e248
push id71510
push userphilringnalda@gmail.com
push dateSun, 07 Feb 2016 06:50:06 +0000
treeherdermozilla-inbound@69f7ba132363 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersdholbert
bugs1241692
milestone47.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 1241692 - Part 2: We should observe stylings only for animations. r=dholbert We don't need observe restylings other than animations. If those restylings happen, it's just noise for this test. We should drop them.
dom/animation/test/chrome/test_restyles.html
--- a/dom/animation/test/chrome/test_restyles.html
+++ b/dom/animation/test/chrome/test_restyles.html
@@ -41,17 +41,19 @@ function observeStyling(frameCount, onFr
   docShell.recordProfileTimelineMarkers = true;
   docShell.popProfileTimelineMarkers();
 
   return new Promise(function(resolve) {
     return waitForAnimationFrames(frameCount, onFrame).then(function() {
       var markers = docShell.popProfileTimelineMarkers();
       docShell.recordProfileTimelineMarkers = false;
       var stylingMarkers = markers.filter(function(marker, index) {
-        return marker.name == 'Styles';
+        return marker.name == 'Styles' &&
+               (marker.restyleHint == 'eRestyle_CSSAnimations' ||
+                marker.restyleHint == 'eRestyle_CSSTransitions');
       });
       resolve(stylingMarkers);
     });
   });
 }
 
 function ensureElementRemoval(aElement) {
   return new Promise(function(resolve) {