Remove infinite clip in PaintArtifactCompositor.

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

Index: third_party/WebKit/Source/platform/graphics/paint/GeometryMapper.h
diff --git a/third_party/WebKit/Source/platform/graphics/paint/GeometryMapper.h b/third_party/WebKit/Source/platform/graphics/paint/GeometryMapper.h
index b12dcffd4cced61d6b76ee6fe2182fc08e62942a..a39c75d1eca5c60a8def570548b692ada677e558 100644
--- a/third_party/WebKit/Source/platform/graphics/paint/GeometryMapper.h
+++ b/third_party/WebKit/Source/platform/graphics/paint/GeometryMapper.h
@@ -125,6 +125,13 @@ class PLATFORM_EXPORT GeometryMapper {
       const PropertyTreeState& ancestorState,
       bool& success);
 
+  // Returns the "clip visual rect" between |localTransformState| and
+  // |ancestorState|. See above for the definition of "clip visual rect".
+  FloatRect localToAncestorClipRect(

[wkorman, 2016/12/29 19:20:40]

nit: this is only unit tested indirectly, may be worth adding a few specific
test(s) as we expose it as a public method, but not a big deal. 

[chrishtr, 2016/12/29 19:34:21]

Good idea. Done.

+      const PropertyTreeState& localTransformState,
+      const PropertyTreeState& ancestorState,
+      bool& success);
+
  private:
   // Used by mapToVisualRectInDestinationSpace() after fast mapping (assuming
   // destination is an ancestor of source) failed.
@@ -142,13 +149,6 @@ class PLATFORM_EXPORT GeometryMapper {
       const PropertyTreeState& destinationState,
       bool& success);
 
-  // Returns the "clip visual rect" between |localTransformState| and
-  // |ancestorState|. See above for the definition of "clip visual rect".
-  FloatRect localToAncestorClipRect(
-      const PropertyTreeState& localTransformState,
-      const PropertyTreeState& ancestorState,
-      bool& success);
-
   // Returns the precomputed data if already set, or adds and memoizes a new
   // PrecomputedDataForAncestor otherwise.
   PrecomputedDataForAncestor& getPrecomputedDataForAncestor(