Fix repeat background position problem

third_party/WebKit/Source/core/paint/BackgroundImageGeometry.cpp

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "core/paint/BackgroundImageGeometry.h"
 
 #include "core/frame/FrameView.h"
 #include "core/layout/LayoutBox.h"
 #include "core/layout/LayoutBoxModelObject.h"
 #include "core/layout/LayoutTableCell.h"
@@ skipping 207 matching lines @@
                                          LayoutUnit extraOffset,
                                          LayoutUnit offsetForCell) {
   // We would like to identify the phase as a fraction of the image size in the
   // absence of snapping, then re-apply it to the snapped values. This is to
   // handle large positions.
   if (unsnappedTileWidth) {
     LayoutUnit computedXPosition =
         roundedMinimumValueForLength(fillLayer.xPosition(),
                                      unsnappedAvailableWidth) -
         offsetForCell;
+    float numberOfTilesInPosition;
     if (fillLayer.backgroundXOrigin() == RightEdge) {
-      float numberOfTilesInPosition =
+      numberOfTilesInPosition =
           (snappedAvailableWidth - computedXPosition + extraOffset).toFloat() /
           unsnappedTileWidth.toFloat();
-      float fractionalPositionWithinTile =
-          numberOfTilesInPosition - truncf(numberOfTilesInPosition);
-      setPhaseX(LayoutUnit(
-          roundf(fractionalPositionWithinTile * tileSize().width())));
     } else {
-      float numberOfTilesInPosition =
-          (computedXPosition + extraOffset).toFloat() /
-          unsnappedTileWidth.toFloat();
-      float fractionalPositionWithinTile =
-          1.0f - (numberOfTilesInPosition - truncf(numberOfTilesInPosition));
-      setPhaseX(LayoutUnit(
-          roundf(fractionalPositionWithinTile * tileSize().width())));
+      numberOfTilesInPosition = (computedXPosition + extraOffset).toFloat() /
+                                unsnappedTileWidth.toFloat();
     }
+    float fractionalPositionWithinTile =
+        1.0f - (numberOfTilesInPosition - truncf(numberOfTilesInPosition));
+    setPhaseX(
+        LayoutUnit(roundf(fractionalPositionWithinTile * tileSize().width())));
   } else {
     setPhaseX(LayoutUnit());
   }
   setSpaceSize(LayoutSize(LayoutUnit(), spaceSize().height()));
 }
 
 void BackgroundImageGeometry::setRepeatY(const FillLayer& fillLayer,
                                          LayoutUnit unsnappedTileHeight,
                                          LayoutUnit snappedAvailableHeight,
                                          LayoutUnit unsnappedAvailableHeight,
                                          LayoutUnit extraOffset,
                                          LayoutUnit offsetForCell) {
   // We would like to identify the phase as a fraction of the image size in the
   // absence of snapping, then re-apply it to the snapped values. This is to
   // handle large positions.
   if (unsnappedTileHeight) {
     LayoutUnit computedYPosition =
         roundedMinimumValueForLength(fillLayer.yPosition(),
                                      unsnappedAvailableHeight) -
         offsetForCell;
+    float numberOfTilesInPosition;
     if (fillLayer.backgroundYOrigin() == BottomEdge) {
-      float numberOfTilesInPosition =
+      numberOfTilesInPosition =
           (snappedAvailableHeight - computedYPosition + extraOffset).toFloat() /
           unsnappedTileHeight.toFloat();
-      float fractionalPositionWithinTile =
-          numberOfTilesInPosition - truncf(numberOfTilesInPosition);
-      setPhaseY(LayoutUnit(
-          roundf(fractionalPositionWithinTile * tileSize().height())));
     } else {
-      float numberOfTilesInPosition =
-          (computedYPosition + extraOffset).toFloat() /
-          unsnappedTileHeight.toFloat();
-      float fractionalPositionWithinTile =
-          1.0f - (numberOfTilesInPosition - truncf(numberOfTilesInPosition));
-      setPhaseY(LayoutUnit(
-          roundf(fractionalPositionWithinTile * tileSize().height())));
+      numberOfTilesInPosition = (computedYPosition + extraOffset).toFloat() /
+                                unsnappedTileHeight.toFloat();
     }
+    float fractionalPositionWithinTile =
+        1.0f - (numberOfTilesInPosition - truncf(numberOfTilesInPosition));
+    setPhaseY(
+        LayoutUnit(roundf(fractionalPositionWithinTile * tileSize().height())));
   } else {
     setPhaseY(LayoutUnit());
   }
   setSpaceSize(LayoutSize(spaceSize().width(), LayoutUnit()));
 }
 
 void BackgroundImageGeometry::setSpaceX(LayoutUnit space,
                                         LayoutUnit availableWidth,
                                         LayoutUnit extraOffset) {
   LayoutUnit computedXPosition =
@@ skipping 347 matching lines @@
     useFixedAttachment(paintRect.location());
 
   // Clip the final output rect to the paint rect
   m_destRect.intersect(paintRect);
 
   // Snap as-yet unsnapped values.
   setDestRect(LayoutRect(pixelSnappedIntRect(m_destRect)));
 }
 
 }  // namespace blink