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

Unified Diff: third_party/WebKit/Source/core/svg/graphics/SVGImageForContainer.cpp

Issue 2559013002: Add ColorBehavior to blink::Image draw methods (Closed)
Patch Set: Rebase 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 side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/core/svg/graphics/SVGImageForContainer.cpp
diff --git a/third_party/WebKit/Source/core/svg/graphics/SVGImageForContainer.cpp b/third_party/WebKit/Source/core/svg/graphics/SVGImageForContainer.cpp
index 9751be631d27757281b07cdc62187f809401a058..0bd7db6c835973dd72735db55f24f8477cf794d4 100644
--- a/third_party/WebKit/Source/core/svg/graphics/SVGImageForContainer.cpp
+++ b/third_party/WebKit/Source/core/svg/graphics/SVGImageForContainer.cpp
@@ -42,7 +42,10 @@ void SVGImageForContainer::draw(SkCanvas* canvas,
const FloatRect& dstRect,
const FloatRect& srcRect,
RespectImageOrientationEnum,
- ImageClampingMode) {
+ ImageClampingMode,
+ const ColorBehavior& colorBehavior) {
+ // TODO(ccameron): This function should not ignore |colorBehavior|.
+ // https://crbug.com/667431
m_image->drawForContainer(canvas, paint, m_containerSize, m_zoom, dstRect,
srcRect, m_url);
}
@@ -54,12 +57,17 @@ void SVGImageForContainer::drawPattern(GraphicsContext& context,
SkBlendMode op,
const FloatRect& dstRect,
const FloatSize& repeatSpacing) {
+ // TODO(ccameron): This function should not ignore |context|'s color behavior.
+ // https://crbug.com/667431
m_image->drawPatternForContainer(context, m_containerSize, m_zoom, srcRect,
scale, phase, op, dstRect, repeatSpacing,
m_url);
}
-sk_sp<SkImage> SVGImageForContainer::imageForCurrentFrame() {
+sk_sp<SkImage> SVGImageForContainer::imageForCurrentFrame(
+ const ColorBehavior& colorBehavior) {
+ // TODO(ccameron): This function should not ignore |colorBehavior|.
+ // https://crbug.com/667431
return m_image->imageForCurrentFrameForContainer(m_url, size());
}

Powered by Google App Engine
This is Rietveld 408576698