Introduce SVGPaintServer

Source/core/paint/SVGInlineTextBoxPainter.cpp

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "config.h"
 #include "core/paint/SVGInlineTextBoxPainter.h"
 
 #include "core/dom/DocumentMarkerController.h"
 #include "core/dom/RenderedDocumentMarker.h"
 #include "core/editing/Editor.h"
 #include "core/frame/LocalFrame.h"
 #include "core/paint/InlinePainter.h"
 #include "core/paint/InlineTextBoxPainter.h"
+#include "core/paint/SVGPaintServer.h"
 #include "core/rendering/PaintInfo.h"
 #include "core/rendering/RenderInline.h"
 #include "core/rendering/RenderTheme.h"
 #include "core/rendering/style/ShadowList.h"
 #include "core/rendering/svg/RenderSVGInlineText.h"
 #include "core/rendering/svg/RenderSVGResource.h"
-#include "core/rendering/svg/RenderSVGResourceSolidColor.h"
 #include "core/rendering/svg/SVGInlineTextBox.h"
 #include "core/rendering/svg/SVGRenderSupport.h"
 #include "core/rendering/svg/SVGResourcesCache.h"
 
 namespace blink {
 
 static inline bool textShouldBePainted(RenderSVGInlineText& textRenderer)
 {
     // Font::pixelSize(), returns FontDescription::computedPixelSize(), which returns "int(x + 0.5)".
     // If the absolute font size on screen is below x=0.5, don't render anything.
@@ skipping 100 matching lines @@
         // Spec: Line-through should be drawn after the text is filled and stroked; thus, the line-through is rendered on top of the text.
         if (decorations & TextDecorationLineThrough)
             paintDecoration(paintInfo.context, TextDecorationLineThrough, fragment);
     }
 
     // finally, paint the outline if any
     if (style->hasOutline() && parentRenderer.isRenderInline())
         InlinePainter(toRenderInline(parentRenderer)).paintOutline(paintInfo, paintOffset);
 }
 
-class PaintingResourceScope {
-public:
-    PaintingResourceScope(RenderObject& renderer)
-        : m_renderer(renderer)
-        , m_paintingResource(0)
-    {
-    }
-    ~PaintingResourceScope() { ASSERT(!m_paintingResource); }
-
-    bool acquirePaintingResource(GraphicsContext*&, RenderStyle*, RenderSVGResourceModeFlags);
-    void releasePaintingResource(GraphicsContext*&);
-
-private:
-    RenderObject& m_renderer;
-    RenderSVGResource* m_paintingResource;
-};
-
-bool PaintingResourceScope::acquirePaintingResource(GraphicsContext*& context, RenderStyle* style, RenderSVGResourceModeFlags resourceModeFlags)
-{
-    ASSERT(style);
-    RenderSVGResourceMode resourceMode = static_cast<RenderSVGResourceMode>(resourceModeFlags & (ApplyToFillMode | ApplyToStrokeMode));
-    ASSERT(resourceMode == ApplyToFillMode || resourceMode == ApplyToStrokeMode);
-
-    bool hasFallback = false;
-    m_paintingResource = RenderSVGResource::requestPaintingResource(resourceMode, &m_renderer, style, hasFallback);
-    if (!m_paintingResource)
-        return false;
-
-    if (!m_paintingResource->applyResource(&m_renderer, style, context, resourceModeFlags)) {
-        if (hasFallback) {
-            m_paintingResource = RenderSVGResource::sharedSolidPaintingResource();
-            m_paintingResource->applyResource(&m_renderer, style, context, resourceModeFlags);
-        }
-    }
-    return true;
-}
-
-void PaintingResourceScope::releasePaintingResource(GraphicsContext*& context)
-{
-    ASSERT(m_paintingResource);
-
-    m_paintingResource->postApplyResource(context);
-    m_paintingResource = 0;
-}
-
 void SVGInlineTextBoxPainter::paintSelectionBackground(PaintInfo& paintInfo)
 {
     if (m_svgInlineTextBox.renderer().style()->visibility() != VISIBLE)
         return;
 
     RenderObject& parentRenderer = m_svgInlineTextBox.parent()->renderer();
     ASSERT(!parentRenderer.document().printing());
 
     // Determine whether or not we're selected.
     bool paintSelectedTextOnly = paintInfo.phase == PaintPhaseSelection;
@@ skipping 125 matching lines @@
 
     if (fragment.width <= 0 && thickness <= 0)
         return;
 
     float decorationOffset = baselineOffsetForDecoration(decoration, scaledFont.fontMetrics(), thickness);
     FloatPoint decorationOrigin(fragment.x, fragment.y - decorationOffset / scalingFactor);
 
     Path path;
     path.addRect(FloatRect(decorationOrigin, FloatSize(fragment.width, thickness / scalingFactor)));
 
-    PaintingResourceScope resourceScope(*decorationRenderer);
-    if (resourceScope.acquirePaintingResource(context, decorationStyle, resourceMode)) {
-        SVGRenderSupport::fillOrStrokePath(context, resourceMode, path);
-        resourceScope.releasePaintingResource(context);
-    }
+    GraphicsContextStateSaver stateSaver(*context, false);
+    if (!SVGPaintServer::updateGraphicsContext(stateSaver, decorationStyle, *decorationRenderer, resourceMode))
+        return;
+    SVGRenderSupport::fillOrStrokePath(context, resourceMode, path);
 }
 
 void SVGInlineTextBoxPainter::paintTextWithShadows(GraphicsContext* context, RenderStyle* style,
     TextRun& textRun, const SVGTextFragment& fragment, int startPosition, int endPosition,
     RenderSVGResourceMode resourceMode)
 {
     RenderSVGInlineText& textRenderer = toRenderSVGInlineText(m_svgInlineTextBox.renderer());
 
     float scalingFactor = textRenderer.scalingFactor();
     ASSERT(scalingFactor);
 
     const Font& scaledFont = textRenderer.scaledFont();
     const ShadowList* shadowList = style->textShadow();
 
     // Text shadows are disabled when printing. http://crbug.com/258321
     bool hasShadow = shadowList && !context->printing();
 
     FloatPoint textOrigin(fragment.x, fragment.y);
     FloatSize textSize(fragment.width, fragment.height);
 
+    GraphicsContextStateSaver stateSaver(*context, false);
     if (scalingFactor != 1) {
         textOrigin.scale(scalingFactor, scalingFactor);
         textSize.scale(scalingFactor);
-        context->save();
+        stateSaver.save();
         context->scale(1 / scalingFactor, 1 / scalingFactor);
     }
 
+    if (!SVGPaintServer::updateGraphicsContext(stateSaver, style, m_svgInlineTextBox.parent()->renderer(), resourceMode | ApplyToTextMode))
+        return;
+
     if (hasShadow)
         context->setDrawLooper(shadowList->createDrawLooper(DrawLooperBuilder::ShadowRespectsAlpha));
 
-    PaintingResourceScope resourceScope(m_svgInlineTextBox.parent()->renderer());
-    if (resourceScope.acquirePaintingResource(context, style, resourceMode | ApplyToTextMode)) {
-        context->setTextDrawingMode(resourceMode == ApplyToFillMode ? TextModeFill : TextModeStroke);
+    context->setTextDrawingMode(resourceMode == ApplyToFillMode ? TextModeFill : TextModeStroke);
 
-        if (scalingFactor != 1 && resourceMode == ApplyToStrokeMode)
-            context->setStrokeThickness(context->strokeThickness() * scalingFactor);
+    if (scalingFactor != 1 && resourceMode == ApplyToStrokeMode)
+        context->setStrokeThickness(context->strokeThickness() * scalingFactor);
 
-        TextRunPaintInfo textRunPaintInfo(textRun);
-        textRunPaintInfo.from = startPosition;
-        textRunPaintInfo.to = endPosition;
+    TextRunPaintInfo textRunPaintInfo(textRun);
+    textRunPaintInfo.from = startPosition;
+    textRunPaintInfo.to = endPosition;
 
-        float baseline = scaledFont.fontMetrics().floatAscent();
-        textRunPaintInfo.bounds = FloatRect(textOrigin.x(), textOrigin.y() - baseline,
-            textSize.width(), textSize.height());
+    float baseline = scaledFont.fontMetrics().floatAscent();
+    textRunPaintInfo.bounds = FloatRect(textOrigin.x(), textOrigin.y() - baseline,
+        textSize.width(), textSize.height());
 
-        scaledFont.drawText(context, textRunPaintInfo, textOrigin);
-        resourceScope.releasePaintingResource(context);
-    }
+    scaledFont.drawText(context, textRunPaintInfo, textOrigin);
 
-    if (scalingFactor != 1)
-        context->restore();
-    else if (hasShadow)
+    if (hasShadow && !stateSaver.saved())

[f(malita), 2014/10/10 15:49:07]

Could we saveIfNeeded in the hasShadow conditional above instead, to reduce GC
state micromanagement?

[fs, 2014/10/10 16:10:23]

Done.

         context->clearShadow();
 }
 
 void SVGInlineTextBoxPainter::paintText(GraphicsContext* context, RenderStyle* style,
     RenderStyle* selectionStyle, const SVGTextFragment& fragment,
     RenderSVGResourceMode resourceMode, bool hasSelection, bool paintSelectedTextOnly)
 {
     ASSERT(style);
     ASSERT(selectionStyle);
 
@@ skipping 84 matching lines @@
 
             fragmentRect = fragmentTransform.mapRect(fragmentRect);
             markerRect.unite(fragmentRect);
         }
     }
 
     toRenderedDocumentMarker(marker)->setRenderedRect(textRenderer.localToAbsoluteQuad(markerRect).enclosingBoundingBox());
 }
 
 } // namespace blink