Backed out changeset 8611e9d0bdaa (bug 1319378) for Win8 reftest failures in table-overflowed-by-animation.html
authorPhil Ringnalda <philringnalda@gmail.com>
Mon, 28 Nov 2016 22:06:06 -0800
changeset 324595 9d051cf1d568b06dada2fe2cda2b9844a5bc1b25
parent 324594 716be76c9a2f05cf595f05f3a366d54c5bf9df41
child 324596 2281f43b043b0ac5b842b5bcae7fc0995737fb19
push id24
push usermaklebus@msu.edu
push dateTue, 20 Dec 2016 03:11:33 +0000
bugs1319378
milestone53.0a1
backs out8611e9d0bdaab5204a120a23b971b0c1d3a9f1b4
Backed out changeset 8611e9d0bdaa (bug 1319378) for Win8 reftest failures in table-overflowed-by-animation.html
layout/generic/nsFrame.cpp
layout/reftests/transform/reftest.list
layout/reftests/transform/table-overflowed-by-animation-ref.html
layout/reftests/transform/table-overflowed-by-animation.html
--- a/layout/generic/nsFrame.cpp
+++ b/layout/generic/nsFrame.cpp
@@ -550,18 +550,17 @@ nsFrame::Init(nsIContent*       aContent
 
     if (HasAnyStateBits(NS_FRAME_IN_POPUP) && TrackingVisibility()) {
       // Assume all frames in popups are visible.
       IncApproximateVisibleCount();
     }
   }
   const nsStyleDisplay *disp = StyleDisplay();
   if (disp->HasTransform(this) ||
-      (IsFrameOfType(eSupportsCSSTransforms) &&
-       nsLayoutUtils::HasAnimationOfProperty(this, eCSSProperty_transform))) {
+      nsLayoutUtils::HasAnimationOfProperty(this, eCSSProperty_transform)) {
     // The frame gets reconstructed if we toggle the -moz-transform
     // property, so we can set this bit here and then ignore it.
     mState |= NS_FRAME_MAY_BE_TRANSFORMED;
   }
   if (disp->mPosition == NS_STYLE_POSITION_STICKY &&
       !aPrevInFlow &&
       !(mState & NS_FRAME_IS_NONDISPLAY) &&
       !disp->IsInnerTableStyle()) {
--- a/layout/reftests/transform/reftest.list
+++ b/layout/reftests/transform/reftest.list
@@ -134,9 +134,8 @@ pref(svg.transform-box.enabled,true) == 
 == transform-origin-svg-2a.svg transform-origin-svg-2-ref.svg
 == transform-origin-svg-2b.svg transform-origin-svg-2-ref.svg
 # Bug 1122526
 == animate-layer-scale-inherit-1.html animate-layer-scale-inherit-1-ref.html
 == animate-layer-scale-inherit-2.html animate-layer-scale-inherit-2-ref.html
 == animate-layer-scale-inherit-3.html animate-layer-scale-inherit-1-ref.html
 # Bug 1301500
 == dynamic-add-without-change-cb-1.html dynamic-add-without-change-cb-1-ref.html
-== table-overflowed-by-animation.html table-overflowed-by-animation-ref.html
deleted file mode 100644
--- a/layout/reftests/transform/table-overflowed-by-animation-ref.html
+++ /dev/null
@@ -1,7 +0,0 @@
-<!DOCTYPE html>
-<html>
-<table style="width: 100px; height: 100px;">
-  <td style="transform: rotateZ(45deg); background-color: rgb(212, 61, 188);">
-  </td>
-</table>
-</html>
deleted file mode 100644
--- a/layout/reftests/transform/table-overflowed-by-animation.html
+++ /dev/null
@@ -1,29 +0,0 @@
-<!DOCTYPE html>
-<html class="reftest-wait">
-<style>
-@keyframes anim {
-  /*
-   * We need to use different transform functions to produce
-   * UpdatePostTransformOverflow change hint, also these functions have to be
-   * the same matrix to being the same position while running reftest.
-   */
-  from {
-    transform: rotate(0deg);
-  }
-  to {
-    transform: scale(1);
-  }
-}
-</style>
-<table id="test" style="width: 100px; height: 100px; animation: anim 1s infinite;">
-  <td style="transform: rotateZ(45deg); background-color: rgb(212, 61, 188);">
-  </td>
-</table>
-<script>
-document.getElementById("test").addEventListener("animationstart", () => {
-  requestAnimationFrame(() => {
-    document.documentElement.classList.remove("reftest-wait");
-  });
-}, false);
-</script>
-</html>