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

Unified Diff: third_party/WebKit/Source/platform/graphics/compositing/PaintArtifactCompositor.h

Issue 2793993003: Make GeometryMapper fully static (Closed)
Patch Set: Created 3 years, 9 months 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/compositing/PaintArtifactCompositor.h
diff --git a/third_party/WebKit/Source/platform/graphics/compositing/PaintArtifactCompositor.h b/third_party/WebKit/Source/platform/graphics/compositing/PaintArtifactCompositor.h
index 6254454c902b8df319153afcacd7406652962b45..f427f91a87157514e626a85e5bab1bc6aee530ba 100644
--- a/third_party/WebKit/Source/platform/graphics/compositing/PaintArtifactCompositor.h
+++ b/third_party/WebKit/Source/platform/graphics/compositing/PaintArtifactCompositor.h
@@ -25,7 +25,6 @@ class Vector2dF;
namespace blink {
-class GeometryMapper;
class JSONObject;
class PaintArtifact;
class WebLayer;
@@ -56,8 +55,7 @@ class PLATFORM_EXPORT PaintArtifactCompositor {
void update(
const PaintArtifact&,
RasterInvalidationTrackingMap<const PaintChunk>* paintChunkInvalidations,
- bool storeDebugInfo,
- GeometryMapper&);
+ bool storeDebugInfo);
// The root layer of the tree managed by this object.
cc::Layer* rootLayer() const { return m_rootLayer.get(); }
@@ -97,14 +95,14 @@ class PLATFORM_EXPORT PaintArtifactCompositor {
// chunks after chunks in this layer, with appropriate space conversion
// applied. The merged layer must have a property tree state that's deeper
// than this layer, i.e. can "upcast" to this layer's state.
- void merge(const PendingLayer& guest, GeometryMapper&);
+ void merge(const PendingLayer& guest);
bool canMerge(const PendingLayer& guest) const;
// Mutate this layer's property tree state to a more general (shallower)
// state, thus the name "upcast". The concrete effect of this is to
// "decomposite" some of the properties, so that fewer properties will be
// applied by the compositor, and more properties will be applied internally
// to the chunks as Skia commands.
- void upcast(const PropertyTreeState&, GeometryMapper&);
+ void upcast(const PropertyTreeState&);
FloatRect bounds;
Vector<const PaintChunk*> paintChunks;
bool knownToBeOpaque;
@@ -120,8 +118,7 @@ class PLATFORM_EXPORT PaintArtifactCompositor {
// Collects the PaintChunks into groups which will end up in the same
// cc layer. This is the entry point of the layerization algorithm.
void collectPendingLayers(const PaintArtifact&,
- Vector<PendingLayer>& pendingLayers,
- GeometryMapper&);
+ Vector<PendingLayer>& pendingLayers);
// This is the internal recursion of collectPendingLayers. This function
// loops over the list of paint chunks, scoped by an isolated group
// (i.e. effect node). Inside of the loop, chunks are tested for overlap
@@ -141,12 +138,9 @@ class PLATFORM_EXPORT PaintArtifactCompositor {
// can be satisfied (and the effect node has no direct reason).
static void layerizeGroup(const PaintArtifact&,
Vector<PendingLayer>& pendingLayers,
- GeometryMapper&,
const EffectPaintPropertyNode&,
Vector<PaintChunk>::const_iterator& chunkCursor);
- static bool mightOverlap(const PendingLayer&,
- const PendingLayer&,
- GeometryMapper&);
+ static bool mightOverlap(const PendingLayer&, const PendingLayer&);
static bool canDecompositeEffect(const EffectPaintPropertyNode*,
const PendingLayer&);
@@ -162,8 +156,7 @@ class PLATFORM_EXPORT PaintArtifactCompositor {
gfx::Vector2dF& layerOffset,
Vector<std::unique_ptr<ContentLayerClientImpl>>& newContentLayerClients,
RasterInvalidationTrackingMap<const PaintChunk>*,
- bool storeDebugInfo,
- GeometryMapper&);
+ bool storeDebugInfo);
// Finds a client among the current vector of clients that matches the paint
// chunk's id, or otherwise allocates a new one.

Powered by Google App Engine
This is Rietveld 408576698