Bug 1168245 - Remove some code that appears to make no sense from nsDisplayTransform::GetDeltaToTransformOrigin. r=mattwoodrow
authorJonathan Watt <jwatt@jwatt.org>
Wed, 13 May 2015 13:22:15 +0100
changeset 245656 3c9ee60d1ca5cb1c5236c52553afba9466f272ed
parent 245655 1c34d99961d4080149389d6d38fe16c493273d48
child 245657 143b620b98aafeb0b6977c891e2c7c19c916d4a0
push id28814
push userkwierso@gmail.com
push dateTue, 26 May 2015 23:16:29 +0000
treeherdermozilla-central@ff2e07228041 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersmattwoodrow
bugs1168245
milestone41.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 1168245 - Remove some code that appears to make no sense from nsDisplayTransform::GetDeltaToTransformOrigin. r=mattwoodrow
layout/base/nsDisplayList.cpp
--- a/layout/base/nsDisplayList.cpp
+++ b/layout/base/nsDisplayList.cpp
@@ -4691,21 +4691,16 @@ nsDisplayTransform::GetDeltaToTransformO
       nscoord offset =
         (index == 0) ? aFrame->GetPosition().x : aFrame->GetPosition().y;
       coords[index] -= NSAppUnitsToFloatPixels(offset, aAppUnitsPerPixel);
     }
   }
 
   coords[2] = NSAppUnitsToFloatPixels(display->mTransformOrigin[2].GetCoordValue(),
                                       aAppUnitsPerPixel);
-  if (aBoundsOverride) {
-    // Adjust based on the origin of the override:
-    coords[0] += NSAppUnitsToFloatPixels(aBoundsOverride->x, aAppUnitsPerPixel);
-    coords[1] += NSAppUnitsToFloatPixels(aBoundsOverride->y, aAppUnitsPerPixel);
-  }
 
   return Point3D(coords[0], coords[1], coords[2]);
 }
 
 /* Returns the delta specified by the -moz-perspective-origin property.
  * This is a positive delta, meaning that it indicates the direction to move
  * to get from (0, 0) of the frame to the perspective origin. This function is
  * called off the main thread.