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

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

Issue 1987943002: [wip] unprefix filter Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 6 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 129 matching lines...) Expand 10 before | Expand all | Expand 10 after
140 if (!SVGMaskPainter(*masker).prepareEffect(m_object, paintInfo().context )) 140 if (!SVGMaskPainter(*masker).prepareEffect(m_object, paintInfo().context ))
141 return false; 141 return false;
142 m_masker = masker; 142 m_masker = masker;
143 } 143 }
144 return true; 144 return true;
145 } 145 }
146 146
147 bool SVGPaintContext::applyFilterIfNecessary(SVGResources* resources) 147 bool SVGPaintContext::applyFilterIfNecessary(SVGResources* resources)
148 { 148 {
149 if (!resources) { 149 if (!resources) {
150 if (m_object.style()->svgStyle().hasFilter()) 150 if (m_object.style()->hasFilter())
151 return false; 151 return false;
152 } else if (LayoutSVGResourceFilter* filter = resources->filter()) { 152 } else if (LayoutSVGResourceFilter* filter = resources->filter()) {
153 m_filterRecordingContext = adoptPtr(new SVGFilterRecordingContext(paintI nfo().context)); 153 m_filterRecordingContext = adoptPtr(new SVGFilterRecordingContext(paintI nfo().context));
154 m_filter = filter; 154 m_filter = filter;
155 GraphicsContext* filterContext = SVGFilterPainter(*filter).prepareEffect (m_object, *m_filterRecordingContext); 155 GraphicsContext* filterContext = SVGFilterPainter(*filter).prepareEffect (m_object, *m_filterRecordingContext);
156 if (!filterContext) 156 if (!filterContext)
157 return false; 157 return false;
158 158
159 // Because the filter needs to cache its contents we replace the context 159 // Because the filter needs to cache its contents we replace the context
160 // during filtering with the filter's context. 160 // during filtering with the filter's context.
(...skipping 61 matching lines...) Expand 10 before | Expand all | Expand 10 after
222 // though. 222 // though.
223 // Additionally, it's not really safe/guaranteed to be correct, as 223 // Additionally, it's not really safe/guaranteed to be correct, as
224 // something down the paint pipe may want to farther tweak the color 224 // something down the paint pipe may want to farther tweak the color
225 // filter, which could yield incorrect results. (Consider just using 225 // filter, which could yield incorrect results. (Consider just using
226 // saveLayer() w/ this color filter explicitly instead.) 226 // saveLayer() w/ this color filter explicitly instead.)
227 paint.setColorFilter(sk_ref_sp(paintInfo.context.colorFilter())); 227 paint.setColorFilter(sk_ref_sp(paintInfo.context.colorFilter()));
228 return true; 228 return true;
229 } 229 }
230 230
231 } // namespace blink 231 } // namespace blink
OLDNEW
« no previous file with comments | « third_party/WebKit/Source/core/paint/PaintLayer.cpp ('k') | third_party/WebKit/Source/core/style/SVGComputedStyle.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698