Use LayoutRect during addFocusRingRects to avoid loss of precision

Source/core/rendering/RenderBox.cpp

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 1999 Antti Koivisto (koivisto@kde.org)
  *           (C) 2005 Allan Sandfeld Jensen (kde@carewolf.com)
  *           (C) 2005, 2006 Samuel Weinig (sam.weinig@gmail.com)
  * Copyright (C) 2005, 2006, 2007, 2008, 2009, 2010 Apple Inc. All rights reserved.
  * Copyright (C) 2013 Adobe Systems Incorporated. All rights reserved.
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
@@ skipping 558 matching lines @@
     rect.move(absPos.x(), absPos.y());
     return rect;
 }
 
 FloatQuad RenderBox::absoluteContentQuad() const
 {
     LayoutRect rect = contentBoxRect();
     return localToAbsoluteQuad(FloatRect(rect));
 }
 
-void RenderBox::addFocusRingRects(Vector<IntRect>& rects, const LayoutPoint& additionalOffset, const RenderLayerModelObject*) const
+void RenderBox::addFocusRingRects(Vector<LayoutRect>& rects, const LayoutPoint& additionalOffset, const RenderLayerModelObject*) const
 {
     if (!size().isEmpty())
-        rects.append(pixelSnappedIntRect(additionalOffset, size()));
+        rects.append(LayoutRect(additionalOffset, size()));
 }
 
 bool RenderBox::canResize() const
 {
     // We need a special case for <iframe> because they never have
     // hasOverflowClip(). However, they do "implicitly" clip their contents, so
     // we want to allow resizing them also.
     return (hasOverflowClip() || isRenderIFrame()) && style()->resize() != RESIZE_NONE;
 }
 
@@ skipping 3604 matching lines @@
         top = std::max(top, borderOutsets.top());
         right = std::max(right, borderOutsets.right());
         bottom = std::max(bottom, borderOutsets.bottom());
         left = std::max(left, borderOutsets.left());
     }
 
     RenderStyle* outlineStyle = this->outlineStyle();
     if (outlineStyle->hasOutline()) {
         if (outlineStyle->outlineStyleIsAuto()) {
             // The result focus ring rects are in coordinates of this object's border box.
-            Vector<IntRect> focusRingRects;
+            Vector<LayoutRect> focusRingRects;
             addFocusRingRects(focusRingRects, LayoutPoint(), this);
-            IntRect rect = unionRect(focusRingRects);
+            LayoutRect rect = unionRect(focusRingRects);
 
             int outlineSize = GraphicsContext::focusRingOutsetExtent(outlineStyle->outlineOffset(), outlineStyle->outlineWidth());
-            top = std::max<LayoutUnit>(top, -rect.y() + outlineSize);
-            right = std::max<LayoutUnit>(right, rect.maxX() - width() + outlineSize);
-            bottom = std::max<LayoutUnit>(bottom, rect.maxY() - height() + outlineSize);
-            left = std::max<LayoutUnit>(left, -rect.x() + outlineSize);
+            top = std::max(top, -rect.y() + outlineSize);
+            right = std::max(right, rect.maxX() - width() + outlineSize);
+            bottom = std::max(bottom, rect.maxY() - height() + outlineSize);
+            left = std::max(left, -rect.x() + outlineSize);
         } else {
             LayoutUnit outlineSize = outlineStyle->outlineSize();
             top = std::max(top, outlineSize);
             right = std::max(right, outlineSize);
             bottom = std::max(bottom, outlineSize);
             left = std::max(left, outlineSize);
         }
     }
 
     return LayoutBoxExtent(top, right, bottom, left);
@@ skipping 534 matching lines @@
 RenderBox::BoxDecorationData::BoxDecorationData(const RenderStyle& style)
 {
     backgroundColor = style.visitedDependentColor(CSSPropertyBackgroundColor);
     hasBackground = backgroundColor.alpha() || style.hasBackgroundImage();
     ASSERT(hasBackground == style.hasBackground());
     hasBorder = style.hasBorder();
     hasAppearance = style.hasAppearance();
 }
 
 } // namespace blink