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

Side by Side Diff: third_party/WebKit/Source/core/paint/SVGShapePainter.cpp

Issue 2690583002: Make cc/paint have concrete types (Closed)
Patch Set: PaintRecord as typedef, fixup playback calls 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 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 #include "core/paint/SVGShapePainter.h" 5 #include "core/paint/SVGShapePainter.h"
6 6
7 #include "core/layout/svg/LayoutSVGResourceMarker.h" 7 #include "core/layout/svg/LayoutSVGResourceMarker.h"
8 #include "core/layout/svg/LayoutSVGShape.h" 8 #include "core/layout/svg/LayoutSVGShape.h"
9 #include "core/layout/svg/SVGLayoutSupport.h" 9 #include "core/layout/svg/SVGLayoutSupport.h"
10 #include "core/layout/svg/SVGMarkerData.h" 10 #include "core/layout/svg/SVGMarkerData.h"
(...skipping 210 matching lines...) Expand 10 before | Expand all | Expand 10 after
221 markerPosition.type, markerStart, markerMid, markerEnd)) { 221 markerPosition.type, markerStart, markerMid, markerEnd)) {
222 PaintRecordBuilder builder(boundingBox, nullptr, &paintInfo.context); 222 PaintRecordBuilder builder(boundingBox, nullptr, &paintInfo.context);
223 PaintInfo markerPaintInfo(builder.context(), paintInfo); 223 PaintInfo markerPaintInfo(builder.context(), paintInfo);
224 224
225 // It's expensive to track the transformed paint cull rect for each 225 // It's expensive to track the transformed paint cull rect for each
226 // marker so just disable culling. The shape paint call will already 226 // marker so just disable culling. The shape paint call will already
227 // be culled if it is outside the paint info cull rect. 227 // be culled if it is outside the paint info cull rect.
228 markerPaintInfo.m_cullRect.m_rect = LayoutRect::infiniteIntRect(); 228 markerPaintInfo.m_cullRect.m_rect = LayoutRect::infiniteIntRect();
229 229
230 paintMarker(markerPaintInfo, *marker, markerPosition, strokeWidth); 230 paintMarker(markerPaintInfo, *marker, markerPosition, strokeWidth);
231 builder.endRecording()->playback(paintInfo.context.canvas()); 231 paintInfo.context.canvas()->drawPicture(builder.endRecording());
232 } 232 }
233 } 233 }
234 } 234 }
235 235
236 void SVGShapePainter::paintMarker(const PaintInfo& paintInfo, 236 void SVGShapePainter::paintMarker(const PaintInfo& paintInfo,
237 const LayoutSVGResourceMarker& marker, 237 const LayoutSVGResourceMarker& marker,
238 const MarkerPosition& position, 238 const MarkerPosition& position,
239 float strokeWidth) { 239 float strokeWidth) {
240 if (!marker.shouldPaint()) 240 if (!marker.shouldPaint())
241 return; 241 return;
242 242
243 TransformRecorder transformRecorder( 243 TransformRecorder transformRecorder(
244 paintInfo.context, marker, 244 paintInfo.context, marker,
245 marker.markerTransformation(position.origin, position.angle, 245 marker.markerTransformation(position.origin, position.angle,
246 strokeWidth)); 246 strokeWidth));
247 Optional<FloatClipRecorder> clipRecorder; 247 Optional<FloatClipRecorder> clipRecorder;
248 if (SVGLayoutSupport::isOverflowHidden(&marker)) 248 if (SVGLayoutSupport::isOverflowHidden(&marker))
249 clipRecorder.emplace(paintInfo.context, marker, paintInfo.phase, 249 clipRecorder.emplace(paintInfo.context, marker, paintInfo.phase,
250 marker.viewport()); 250 marker.viewport());
251 SVGContainerPainter(marker).paint(paintInfo); 251 SVGContainerPainter(marker).paint(paintInfo);
252 } 252 }
253 253
254 } // namespace blink 254 } // namespace blink
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698