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

Side by Side Diff: third_party/WebKit/Source/core/paint/LayerClipRecorder.h

Issue 2194273002: Fix border radius on composited children. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Update the comment on why we say an empty div can paint 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 unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef LayerClipRecorder_h 5 #ifndef LayerClipRecorder_h
6 #define LayerClipRecorder_h 6 #define LayerClipRecorder_h
7 7
8 #include "core/CoreExport.h" 8 #include "core/CoreExport.h"
9 #include "core/paint/PaintLayerPaintingInfo.h" 9 #include "core/paint/PaintLayerPaintingInfo.h"
10 #include "core/paint/PaintPhase.h" 10 #include "core/paint/PaintPhase.h"
(...skipping 28 matching lines...) Expand all
39 // FIXME: The ClipRect passed is in visual coordinates (not flow thread 39 // FIXME: The ClipRect passed is in visual coordinates (not flow thread
40 // coordinates), but at the same time we pass a fragmentOffset, so that we can 40 // coordinates), but at the same time we pass a fragmentOffset, so that we can
41 // translate from flow thread coordinates to visual coordinates. This may look 41 // translate from flow thread coordinates to visual coordinates. This may look
42 // rather confusing/redundant, but it is needed for rounded border clipping. 42 // rather confusing/redundant, but it is needed for rounded border clipping.
43 // Would be nice to clean up this. 43 // Would be nice to clean up this.
44 explicit LayerClipRecorder( 44 explicit LayerClipRecorder(
45 GraphicsContext&, 45 GraphicsContext&,
46 const LayoutBoxModelObject&, 46 const LayoutBoxModelObject&,
47 DisplayItem::Type, 47 DisplayItem::Type,
48 const ClipRect&, 48 const ClipRect&,
49 const PaintLayerPaintingInfo* localPaintingInfo, 49 const PaintLayer* clipRoot,
50 const LayoutPoint& fragmentOffset, 50 const LayoutPoint& fragmentOffset,
51 PaintLayerFlags, 51 PaintLayerFlags,
52 BorderRadiusClippingRule = IncludeSelfForBorderRadius); 52 BorderRadiusClippingRule = IncludeSelfForBorderRadius);
53 53
54 ~LayerClipRecorder(); 54 ~LayerClipRecorder();
55 55
56 private: 56 private:
57 void collectRoundedRectClips(PaintLayer&, 57 void collectRoundedRectClips(PaintLayer&,
58 const PaintLayerPaintingInfo& localPaintingInfo, 58 const PaintLayer* clipRoot,
59 GraphicsContext&, 59 GraphicsContext&,
60 const LayoutPoint& fragmentOffset, 60 const LayoutPoint& fragmentOffset,
61 PaintLayerFlags, 61 PaintLayerFlags,
62 BorderRadiusClippingRule, 62 BorderRadiusClippingRule,
63 Vector<FloatRoundedRect>& roundedRectClips); 63 Vector<FloatRoundedRect>& roundedRectClips);
64 64
65 GraphicsContext& m_graphicsContext; 65 GraphicsContext& m_graphicsContext;
66 const LayoutBoxModelObject& m_layoutObject; 66 const LayoutBoxModelObject& m_layoutObject;
67 DisplayItem::Type m_clipType; 67 DisplayItem::Type m_clipType;
68 }; 68 };
69 69
70 } // namespace blink 70 } // namespace blink
71 71
72 #endif // LayerClipRecorder_h 72 #endif // LayerClipRecorder_h
OLDNEW
« no previous file with comments | « third_party/WebKit/Source/core/paint/FilterPainter.cpp ('k') | third_party/WebKit/Source/core/paint/LayerClipRecorder.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698