Bug 1361639 - Part 3. Remove unused EffectOffsets::offsetToBoundingBoxInDevPx. draft
authorcku <cku@mozilla.com>
Wed, 30 Aug 2017 11:25:39 +0800
changeset 656361 672ce1aec9dbeaf0e12b2ff0b3dd1dbdc1261e00
parent 656360 f7360810085667c6759e6a80ae5a7f54fba544c7
child 656387 19bf32ea976745e2a8062ca76f3b932c97055e9b
child 656409 022686227d08cf1477d495f944e8de99a93af153
child 656465 67cdb2391e9f54d0341a8edbc6a11792d8ebfe61
push id77183
push userbmo:cku@mozilla.com
push dateThu, 31 Aug 2017 03:28:47 +0000
bugs1361639
milestone57.0a1
Bug 1361639 - Part 3. Remove unused EffectOffsets::offsetToBoundingBoxInDevPx. MozReview-Commit-ID: DRdA5FAeu4Q
layout/svg/nsSVGIntegrationUtils.cpp
--- a/layout/svg/nsSVGIntegrationUtils.cpp
+++ b/layout/svg/nsSVGIntegrationUtils.cpp
@@ -640,19 +640,16 @@ ValidateSVGFrame(nsIFrame* aFrame)
   return true;
 }
 
 struct EffectOffsets {
   // The offset between the reference frame and the bounding box of the
   // target frame in app unit.
   nsPoint  offsetToBoundingBox;
   // The offset between the reference frame and the bounding box of the
-  // target frame in device unit.
-  gfxPoint offsetToBoundingBoxInDevPx;
-  // The offset between the reference frame and the bounding box of the
   // target frame in app unit.
   nsPoint  offsetToUserSpace;
   // The offset between the reference frame and the bounding box of the
   // target frame in device unit.
   gfxPoint offsetToUserSpaceInDevPx;
 };
 
 EffectOffsets
@@ -695,19 +692,16 @@ ComputeEffectOffset(nsIFrame* aFrame, co
   NS_ASSERTION(hasSVGLayout ||
                result.offsetToBoundingBox == result.offsetToUserSpace,
                "For non-SVG frames there shouldn't be any additional offset");
 #endif
 
   result.offsetToUserSpaceInDevPx =
     nsLayoutUtils::PointToGfxPoint(result.offsetToUserSpace,
                                    aFrame->PresContext()->AppUnitsPerDevPixel());
-  result.offsetToBoundingBoxInDevPx =
-    nsLayoutUtils::PointToGfxPoint(result.offsetToBoundingBox,
-                                   aFrame->PresContext()->AppUnitsPerDevPixel());
 
   return result;
 }
 
 /**
  * Setup transform matrix of a gfx context by a specific frame. Move the
  * origin of aParams.ctx to the user space of aFrame.
  */