Refactor to remove need for infiniteIntRect when computing rects in pre-paint

third_party/WebKit/Source/platform/graphics/paint/GeometryMapper.cpp

 // Copyright 2016 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 "platform/graphics/paint/GeometryMapper.h"
 
 #include "platform/RuntimeEnabledFeatures.h"
 #include "platform/geometry/LayoutRect.h"
 
 namespace blink {
@@ skipping 131 matching lines @@
       continue;
 
     PropertyTreeState transformAndClipState(effect->localTransformSpace(),
                                             effect->outputClip(), nullptr);
     bool hasRadius = result.hasRadius();
     result = sourceToDestinationVisualRectInternal(
         result.rect(), lastTransformAndClipState, transformAndClipState,
         success);
     hasRadius |= result.hasRadius();
     if (!success) {
-      result.setHasRadius(hasRadius);
+      if (hasRadius)
+        result.setHasRadius();
       return result;
     }
 
     result = effect->mapRect(result.rect());
-    result.setHasRadius(hasRadius);
+    if (hasRadius)
+      result.setHasRadius();
     lastTransformAndClipState = transformAndClipState;
   }
 
   PropertyTreeState finalTransformAndClipState(ancestorState.transform(),
                                                ancestorState.clip(), nullptr);
   bool hasRadius = result.hasRadius();
   result = sourceToDestinationVisualRectInternal(
       result.rect(), lastTransformAndClipState, finalTransformAndClipState,
       success);
-  result.setHasRadius(hasRadius || result.hasRadius());
+  if (hasRadius || result.hasRadius())
+    result.setHasRadius();
   return result;
 }
 
 FloatRect GeometryMapper::localToAncestorRect(
     const FloatRect& rect,
     const TransformPaintPropertyNode* localTransformNode,
     const TransformPaintPropertyNode* ancestorTransformNode) {
   bool success = false;
   FloatRect result = localToAncestorRectInternal(
       rect, localTransformNode, ancestorTransformNode, success);
@@ skipping 163 matching lines @@
   for (auto it = intermediateNodes.rbegin(); it != intermediateNodes.rend();
        ++it) {
     success = false;
     const TransformationMatrix& transformMatrix = localToAncestorMatrixInternal(
         (*it)->localTransformSpace(), ancestorTransform, success);
     if (!success)
       return clip;
     FloatRect mappedRect = transformMatrix.mapRect((*it)->clipRect().rect());
     clip.intersect(mappedRect);
     if ((*it)->clipRect().isRounded())
-      clip.setHasRadius(true);
+      clip.setHasRadius();
     clipCache.set(*it, clip);
   }
 
   success = true;
   return clipCache.find(descendant)->value;
 }
 
 const TransformationMatrix& GeometryMapper::localToAncestorMatrix(
     const TransformPaintPropertyNode* localTransformNode,
     const TransformPaintPropertyNode* ancestorTransformNode) {
@@ skipping 105 matching lines @@
     const TransformPaintPropertyNode*,
     const TransformPaintPropertyNode*);
 template const ClipPaintPropertyNode* GeometryMapper::lowestCommonAncestor(
     const ClipPaintPropertyNode*,
     const ClipPaintPropertyNode*);
 template const ScrollPaintPropertyNode* GeometryMapper::lowestCommonAncestor(
     const ScrollPaintPropertyNode*,
     const ScrollPaintPropertyNode*);
 
 }  // namespace blink