Backed out changeset 3531847a7419 (bug 1285174) for Reftest failures
authorIris Hsiao <ihsiao@mozilla.com>
Thu, 28 Jul 2016 11:34:18 +0800
changeset 349088 5484e1694dc559510ff4679890b08a59eb78bad9
parent 349087 bcb2a29ee460a790e11ee9178dd62ceddf7f79fe
child 349089 8da1ed0adf15e8244e57f22c581bbee072a118b9
push id1230
push userjlund@mozilla.com
push dateMon, 31 Oct 2016 18:13:35 +0000
treeherdermozilla-release@5e06e3766db2 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
bugs1285174
milestone50.0a1
backs out3531847a7419ec572bce11d97dfb8de81b7a0177
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
Backed out changeset 3531847a7419 (bug 1285174) for Reftest failures
layout/base/nsDisplayList.cpp
--- a/layout/base/nsDisplayList.cpp
+++ b/layout/base/nsDisplayList.cpp
@@ -5448,23 +5448,16 @@ nsDisplayTransform::SetReferenceFrameToA
 }
 
 void
 nsDisplayTransform::Init(nsDisplayListBuilder* aBuilder)
 {
   mHasBounds = false;
   mStoredList.SetClip(aBuilder, DisplayItemClip::NoClip());
   mStoredList.SetVisibleRect(mChildrenVisibleRect);
-  // If we made this item fully visible using ShouldPrerenderTransformedContent in
-  // BuildDisplayListForStackingContext then it won't have been applied to the
-  // post-transform visible region. Do that now.
-  if (mIsFullyVisible) {
-    bool snap;
-    mVisibleRect = GetBounds(aBuilder, &snap);
-  }
 }
 
 nsDisplayTransform::nsDisplayTransform(nsDisplayListBuilder* aBuilder,
                                        nsIFrame *aFrame, nsDisplayList *aList,
                                        const nsRect& aChildrenVisibleRect,
                                        uint32_t aIndex,
                                        bool aIsFullyVisible)
   : nsDisplayItem(aBuilder, aFrame)