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

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

Issue 2342913003: Replace narrowPrecisionToFloat with clampTo<float> (Closed)
Patch Set: Created 4 years, 3 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 /* 1 /*
2 * Copyright (C) 2007, 2008 Rob Buis <buis@kde.org> 2 * Copyright (C) 2007, 2008 Rob Buis <buis@kde.org>
3 * Copyright (C) 2007 Nikolas Zimmermann <zimmermann@kde.org> 3 * Copyright (C) 2007 Nikolas Zimmermann <zimmermann@kde.org>
4 * Copyright (C) 2007 Eric Seidel <eric@webkit.org> 4 * Copyright (C) 2007 Eric Seidel <eric@webkit.org>
5 * Copyright (C) 2009 Google, Inc. All rights reserved. 5 * Copyright (C) 2009 Google, Inc. All rights reserved.
6 * Copyright (C) 2009 Dirk Schulze <krit@webkit.org> 6 * Copyright (C) 2009 Dirk Schulze <krit@webkit.org>
7 * Copyright (C) Research In Motion Limited 2009-2010. All rights reserved. 7 * Copyright (C) Research In Motion Limited 2009-2010. All rights reserved.
8 * 8 *
9 * This library is free software; you can redistribute it and/or 9 * This library is free software; you can redistribute it and/or
10 * modify it under the terms of the GNU Library General Public 10 * modify it under the terms of the GNU Library General Public
(...skipping 13 matching lines...) Expand all
24 24
25 #include "core/paint/SVGPaintContext.h" 25 #include "core/paint/SVGPaintContext.h"
26 26
27 #include "core/frame/FrameHost.h" 27 #include "core/frame/FrameHost.h"
28 #include "core/layout/svg/LayoutSVGResourceFilter.h" 28 #include "core/layout/svg/LayoutSVGResourceFilter.h"
29 #include "core/layout/svg/LayoutSVGResourceMasker.h" 29 #include "core/layout/svg/LayoutSVGResourceMasker.h"
30 #include "core/layout/svg/SVGLayoutSupport.h" 30 #include "core/layout/svg/SVGLayoutSupport.h"
31 #include "core/layout/svg/SVGResources.h" 31 #include "core/layout/svg/SVGResources.h"
32 #include "core/layout/svg/SVGResourcesCache.h" 32 #include "core/layout/svg/SVGResourcesCache.h"
33 #include "core/paint/SVGMaskPainter.h" 33 #include "core/paint/SVGMaskPainter.h"
34 #include "platform/FloatConversion.h"
35 #include "wtf/PtrUtil.h" 34 #include "wtf/PtrUtil.h"
36 35
37 namespace blink { 36 namespace blink {
38 37
39 SVGPaintContext::~SVGPaintContext() 38 SVGPaintContext::~SVGPaintContext()
40 { 39 {
41 if (m_filter) { 40 if (m_filter) {
42 ASSERT(SVGResourcesCache::cachedResourcesForLayoutObject(&m_object)); 41 ASSERT(SVGResourcesCache::cachedResourcesForLayoutObject(&m_object));
43 ASSERT(SVGResourcesCache::cachedResourcesForLayoutObject(&m_object)->fil ter() == m_filter); 42 ASSERT(SVGResourcesCache::cachedResourcesForLayoutObject(&m_object)->fil ter() == m_filter);
44 ASSERT(m_filterRecordingContext); 43 ASSERT(m_filterRecordingContext);
(...skipping 168 matching lines...) Expand 10 before | Expand all | Expand 10 after
213 // though. 212 // though.
214 // Additionally, it's not really safe/guaranteed to be correct, as 213 // Additionally, it's not really safe/guaranteed to be correct, as
215 // something down the paint pipe may want to farther tweak the color 214 // something down the paint pipe may want to farther tweak the color
216 // filter, which could yield incorrect results. (Consider just using 215 // filter, which could yield incorrect results. (Consider just using
217 // saveLayer() w/ this color filter explicitly instead.) 216 // saveLayer() w/ this color filter explicitly instead.)
218 paint.setColorFilter(sk_ref_sp(paintInfo.context.getColorFilter())); 217 paint.setColorFilter(sk_ref_sp(paintInfo.context.getColorFilter()));
219 return true; 218 return true;
220 } 219 }
221 220
222 } // namespace blink 221 } // namespace blink
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698