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

Unified Diff: Source/core/paint/LayerClipRecorder.h

Issue 757183003: Revert of Enable fast/images with slimming paint (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 6 years, 1 month 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
« no previous file with comments | « Source/core/paint/HTMLCanvasPainter.cpp ('k') | Source/core/paint/LayerClipRecorder.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/core/paint/LayerClipRecorder.h
diff --git a/Source/core/paint/LayerClipRecorder.h b/Source/core/paint/LayerClipRecorder.h
deleted file mode 100644
index e03160f8ab214e90a97012be4f078ad8db51fbff..0000000000000000000000000000000000000000
--- a/Source/core/paint/LayerClipRecorder.h
+++ /dev/null
@@ -1,50 +0,0 @@
-// Copyright 2014 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.
-
-#ifndef LayerClipRecorder_h
-#define LayerClipRecorder_h
-
-#include "core/paint/ViewDisplayList.h"
-#include "core/rendering/LayerPaintingInfo.h"
-#include "core/rendering/PaintPhase.h"
-#include "wtf/Vector.h"
-
-namespace blink {
-
-class ClipRect;
-class GraphicsContext;
-class RenderLayerModelObject;
-
-class LayerClipRecorder {
-public:
-
- enum BorderRadiusClippingRule { IncludeSelfForBorderRadius, DoNotIncludeSelfForBorderRadius };
-
- // Set rounded clip rectangles defined by border radii all the way from the LayerPaintingInfo
- // "root" layer down to the specified layer (or the parent of said layer, in case
- // BorderRadiusClippingRule says to skip self). fragmentOffset is used for multicol, to specify
- // the translation required to get from flow thread coordinates to visual coordinates for a
- // certain column.
- // FIXME: The BorderRadiusClippingRule parameter is really useless now. If we want to skip self,
- // why not just supply the parent layer as the first parameter instead?
- // FIXME: The ClipRect passed is in visual coordinates (not flow thread coordinates), but at the
- // same time we pass a fragmentOffset, so that we can translate from flow thread coordinates to
- // visual coordinates. This may look rather confusing/redundant, but it is needed for rounded
- // border clipping. Would be nice to clean up this.
- explicit LayerClipRecorder(const RenderLayerModelObject*, GraphicsContext*, DisplayItem::Type, const ClipRect&, const LayerPaintingInfo* localPaintingInfo, const LayoutPoint& fragmentOffset, PaintLayerFlags, BorderRadiusClippingRule = IncludeSelfForBorderRadius);
-
- ~LayerClipRecorder();
-
-private:
-
- void collectRoundedRectClips(RenderLayer&, const LayerPaintingInfo& localPaintingInfo, GraphicsContext*, const LayoutPoint& fragmentOffset, PaintLayerFlags,
- BorderRadiusClippingRule, Vector<RoundedRect>& roundedRectClips);
-
- GraphicsContext* m_graphicsContext;
- const RenderLayerModelObject* m_renderer;
-};
-
-} // namespace blink
-
-#endif // LayerDisplayList_h
« no previous file with comments | « Source/core/paint/HTMLCanvasPainter.cpp ('k') | Source/core/paint/LayerClipRecorder.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698