Source/platform/graphics/skia/NativeImageSkia.cpp - Issue 223673003: Improve computation of ctmScale{X,Y} in NativeImageSkia::drawPattern

Keyboard Shortcuts

	File
u :	up to issue
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line

	Issue
u :	up to list of issues
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue

Unified Diff: Source/platform/graphics/skia/NativeImageSkia.cpp

Issue 223673003: Improve computation of ctmScale{X,Y} in NativeImageSkia::drawPattern (Closed) Base URL: svn://svn.chromium.org/blink/trunk

Patch Set: Use GC::getCTM(). Created 6 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:

Index: Source/platform/graphics/skia/NativeImageSkia.cpp

diff --git a/Source/platform/graphics/skia/NativeImageSkia.cpp b/Source/platform/graphics/skia/NativeImageSkia.cpp

index f9ed9b6beb100e1c1dbb5323251aac4327af9e69..87133033551e0ef46d2deea75435487fbe9e6cbe 100644

--- a/Source/platform/graphics/skia/NativeImageSkia.cpp

+++ b/Source/platform/graphics/skia/NativeImageSkia.cpp

@@ -451,8 +451,9 @@ void NativeImageSkia::drawPattern(

return; // nothing to draw

SkMatrix totalMatrix = context->getTotalMatrix();

- SkScalar ctmScaleX = totalMatrix.getScaleX();

- SkScalar ctmScaleY = totalMatrix.getScaleY();

+ AffineTransform ctm = context->getCTM();

+ SkScalar ctmScaleX = ctm.xScale();

+ SkScalar ctmScaleY = ctm.yScale();

totalMatrix.preScale(scale.width(), scale.height());

// Figure out what size the bitmap will be in the destination. The