Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1773)

Unified Diff: third_party/WebKit/Source/platform/graphics/paint/GeometryMapper.h

Issue 2604083002: Remove infinite clip in PaintArtifactCompositor. (Closed)
Patch Set: none Created 4 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
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(
+ 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(

Powered by Google App Engine
This is Rietveld 408576698