Fix application of OBB paint-servers when device scale != 1

Source/core/rendering/svg/RenderSVGResourceGradient.cpp

Index: Source/core/rendering/svg/RenderSVGResourceGradient.cpp
diff --git a/Source/core/rendering/svg/RenderSVGResourceGradient.cpp b/Source/core/rendering/svg/RenderSVGResourceGradient.cpp
index 1c026e5201a7768c6c29b0651366f010ec73cc61..6f44a65b116012aa881e069ad581c6b6c7ebcfcf 100644
--- a/Source/core/rendering/svg/RenderSVGResourceGradient.cpp
+++ b/Source/core/rendering/svg/RenderSVGResourceGradient.cpp
@@ -85,8 +85,6 @@ bool RenderSVGResourceGradient::applyResource(RenderObject* object, RenderStyle*
     if (!gradientData)
         gradientData = adoptPtr(new GradientData);
 
-    bool isPaintingText = resourceMode & ApplyToTextMode;
-
     // Create gradient object
     if (!gradientData->gradient) {
         buildGradient(gradientData.get());
@@ -101,35 +99,28 @@ bool RenderSVGResourceGradient::applyResource(RenderObject* object, RenderStyle*
         calculateGradientTransform(gradientTransform);
 
         gradientData->userspaceTransform *= gradientTransform;
-        if (isPaintingText) {
-            // Depending on font scaling factor, we may need to rescale the gradient here since
-            // text painting removes the scale factor from the context.
-            AffineTransform additionalTextTransform;
-            if (shouldTransformOnTextPainting(object, additionalTextTransform))
-                gradientData->userspaceTransform *= additionalTextTransform;
-        }
-        gradientData->gradient->setGradientSpaceTransform(gradientData->userspaceTransform);
     }
 
     if (!gradientData->gradient)
         return false;
 
+    const SVGRenderStyle* svgStyle = style->svgStyle();
+    ASSERT(svgStyle);
+
+    AffineTransform computedGradientSpaceTransform = computeResourceSpaceTransform(object, gradientData->userspaceTransform, svgStyle, resourceMode);

[f(malita), 2014/04/09 16:26:15]

Isn't (resourceMode & ApplyToTextMode) invariant for any given *object (which is
the key for m_gradientMap)? We should be able to include the additional text
transform in the cached based transform, and only adjust the non-scaling stroke
component on applyResource(), no?

[fs, 2014/04/09 16:59:13]

It's not invariant ATM - decorations are not painted with that flag set. I
suppose it would be possible to just pass/set that flag in that case too, and
hence be able to cache it as part of the "base" transform.
It does seem a bit unnecessary to tie in additional dependencies on the CTM if
it can be avoided - or?

[f(malita), 2014/04/09 17:46:51]

Ugh. I find all that SVGInlineTextBox::m_paintingResourceMode shuffling utterly
confusing.
Wouldn't that produce different results though, as we're potentially tweaking
the transform? Maybe that's the correct thing to do for decorations anyway?

[fs, 2014/04/10 08:16:41]

Agreed. (Good that you took care of it! *removes from TODO*)
 
Well, if you start weighing in things like <textPath> it becomes less obvious
surely. In that case the additional per-glyph transform shouldn't affect the
gradient (hmm, need to test that...).
What's to be considered "correct" is just a matter of defining the semantics
though. (It's probably also worth noting that using non-solidcolor fills on
decorations is likely broken ATM for scale!=1 [hint: look at
paintDecorationWithStyle].)
Personally I'd lean towards the side that decouples instance-data as much as
possible (i.e. consider the possibility of moving the bbox-transform out as
well.)

[f(malita), 2014/04/10 12:57:48]

SGTM.

+    gradientData->gradient->setGradientSpaceTransform(computedGradientSpaceTransform);
+
     // Draw gradient
     context->save();
 
-    if (isPaintingText)
+    if (resourceMode & ApplyToTextMode)
         context->setTextDrawingMode(resourceMode & ApplyToFillMode ? TextModeFill : TextModeStroke);
 
-    const SVGRenderStyle* svgStyle = style->svgStyle();
-    ASSERT(svgStyle);
-
     if (resourceMode & ApplyToFillMode) {
         context->setAlphaAsFloat(svgStyle->fillOpacity());
         context->setFillGradient(gradientData->gradient);
         context->setFillRule(svgStyle->fillRule());
     } else if (resourceMode & ApplyToStrokeMode) {
-        if (svgStyle->vectorEffect() == VE_NON_SCALING_STROKE)
-            gradientData->gradient->setGradientSpaceTransform(transformOnNonScalingStroke(object, gradientData->userspaceTransform));
         context->setAlphaAsFloat(svgStyle->strokeOpacity());
         context->setStrokeGradient(gradientData->gradient);
         SVGRenderSupport::applyStrokeStyleToContext(context, style, object);