Find In Page hides the text when text color matches text search hightlight color.

third_party/WebKit/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/editing/Editor.h"
 #include "core/editing/markers/DocumentMarkerController.h"
 #include "core/editing/markers/RenderedDocumentMarker.h"
@@ skipping 47 matching lines @@
         return;
 
     DisplayItem::Type displayItemType = DisplayItem::paintPhaseToDrawingType(paintInfo.phase);
     if (!DrawingRecorder::useCachedDrawingIfPossible(paintInfo.context, m_svgInlineTextBox, displayItemType)) {
         LayoutObject& parentLayoutObject = *LineLayoutPaintShim::layoutObjectFrom(m_svgInlineTextBox.parent()->lineLayoutItem());
         const ComputedStyle& style = parentLayoutObject.styleRef();
 
         // TODO(chrishtr): passing the cull rect is incorrect.
         DrawingRecorder recorder(paintInfo.context, m_svgInlineTextBox, displayItemType, FloatRect(paintInfo.cullRect().m_rect));
         InlineTextBoxPainter(m_svgInlineTextBox).paintDocumentMarkers(
-            paintInfo.context, paintOffset, style,
+            paintInfo, paintOffset, style,
             textLayoutObject.scaledFont(), true);
 
         if (!m_svgInlineTextBox.textFragments().isEmpty())
             paintTextFragments(paintInfo, parentLayoutObject);
+
+        InlineTextBoxPainter(m_svgInlineTextBox).paintDocumentMarkers(
+            paintInfo, paintOffset, style,
+            textLayoutObject.scaledFont(), false);

[fs, 2015/12/17 10:10:55]

Maybe we should change this 'background' bool to an enum { MarkerForeground,
MarkerBackground } or some such. I guess we can punt on that though.

[ramya.v, 2015/12/18 03:43:33]

Done.

     }
 }
 
 void SVGInlineTextBoxPainter::paintTextFragments(const PaintInfo& paintInfo, LayoutObject& parentLayoutObject)
 {
     const ComputedStyle& style = parentLayoutObject.styleRef();
     const SVGComputedStyle& svgStyle = style.svgStyle();
 
     bool hasFill = svgStyle.hasFill();
     bool hasVisibleStroke = svgStyle.hasVisibleStroke();
@@ skipping 79 matching lines @@
 
     LayoutSVGInlineText& textLayoutObject = toLayoutSVGInlineText(*LineLayoutPaintShim::layoutObjectFrom(m_svgInlineTextBox.lineLayoutItem()));
     if (!textShouldBePainted(textLayoutObject))
         return;
 
     const ComputedStyle& style = m_svgInlineTextBox.parent()->lineLayoutItem().styleRef();
 
     int startPosition, endPosition;
     m_svgInlineTextBox.selectionStartEnd(startPosition, endPosition);
 
-    int fragmentStartPosition = 0;
-    int fragmentEndPosition = 0;
     AffineTransform fragmentTransform;
-    unsigned textFragmentsSize = m_svgInlineTextBox.textFragments().size();
+    Vector<SVGTextFragmentWithRange> fragmentInfoList = collectFragments(startPosition, endPosition);
+    unsigned textFragmentsSize = fragmentInfoList.size();
     for (unsigned i = 0; i < textFragmentsSize; ++i) {

[fs, 2015/12/17 10:10:55]

Using a range-based loop here would be an improvement:

for (const SVGTextFragmentWithRange& fragmentWithRange : fragmentInfoList)

[ramya.v, 2015/12/18 03:43:32]

Done.

-        const SVGTextFragment& fragment = m_svgInlineTextBox.textFragments().at(i);
-
-        fragmentStartPosition = startPosition;
-        fragmentEndPosition = endPosition;
-        if (!m_svgInlineTextBox.mapStartEndPositionsIntoFragmentCoordinates(fragment, fragmentStartPosition, fragmentEndPosition))
-            continue;
+        SVGTextFragmentWithRange& fragmentInfo = fragmentInfoList.at(i);
+        const SVGTextFragment& fragment = *(fragmentInfo.fragment);
 
         GraphicsContextStateSaver stateSaver(paintInfo.context);
         fragment.buildFragmentTransform(fragmentTransform);
         if (!fragmentTransform.isIdentity())
             paintInfo.context.concatCTM(fragmentTransform);
 
         paintInfo.context.setFillColor(backgroundColor);
-        paintInfo.context.fillRect(m_svgInlineTextBox.selectionRectForTextFragment(fragment, fragmentStartPosition, fragmentEndPosition, style), backgroundColor);
+        paintInfo.context.fillRect(m_svgInlineTextBox.selectionRectForTextFragment(fragment, fragmentInfo.fragmentStartPosition, fragmentInfo.fragmentEndPosition, style), backgroundColor);
     }
 }
 
 static inline LayoutObject* findLayoutObjectDefininingTextDecoration(InlineFlowBox* parentBox)
 {
     // Lookup first layout object in parent hierarchy which has text-decoration set.
     LayoutObject* layoutObject = 0;
     while (parentBox) {
         layoutObject = LineLayoutPaintShim::layoutObjectFrom(parentBox->lineLayoutItem());
 
@@ skipping 193 matching lines @@
         StyleDifference diff;
         diff.setNeedsPaintInvalidationObject();
         SVGResourcesCache::clientStyleChanged(LineLayoutPaintShim::layoutObjectFrom(m_svgInlineTextBox.parent()->lineLayoutItem()), diff, style);
     }
 
     // Eventually draw text using regular style from the end position of the selection to the end of the current chunk part
     if (endPosition < static_cast<int>(fragment.length) && !paintSelectedTextOnly)
         paintTextWithShadows(paintInfo, style, textRun, fragment, endPosition, fragment.length, resourceMode);
 }
 
-void SVGInlineTextBoxPainter::paintTextMatchMarker(GraphicsContext& context, const LayoutPoint&, DocumentMarker* marker, const ComputedStyle& style, const Font& font)
+Vector<SVGTextFragmentWithRange> SVGInlineTextBoxPainter::collectTextMatches(const PaintInfo& paintInfo, DocumentMarker* marker)
 {
-    // SVG is only interested in the TextMatch markers.
+    Vector<SVGTextFragmentWithRange> textMatchInfoList;
+
+    // SVG does not support grammar or spellcheck markers, so skip anything but TextMatch.
     if (marker->type() != DocumentMarker::TextMatch)
-        return;
+        return textMatchInfoList;
+
+    if (!LineLayoutPaintShim::layoutObjectFrom(m_svgInlineTextBox.lineLayoutItem())->frame()->editor().markedTextMatchesAreHighlighted())
+        return textMatchInfoList;
 
     LayoutSVGInlineText& textLayoutObject = toLayoutSVGInlineText(*LineLayoutPaintShim::layoutObjectFrom(m_svgInlineTextBox.lineLayoutItem()));
 
-    AffineTransform fragmentTransform;
     for (InlineTextBox* box = textLayoutObject.firstTextBox(); box; box = box->nextTextBox()) {

[fs, 2015/12/17 10:10:54]

I still don't think this additional nesting is needed.

Actually, the way the code is written right now it essentially depends on not
really doing anything for any other inline-box (collectFragments operate on
m_svgInlineTextBox and not the box that's iterated here.)

[ramya.v, 2015/12/18 03:43:32]

Done.

         if (!box->isSVGInlineTextBox())
             continue;
 
         SVGInlineTextBox* textBox = toSVGInlineTextBox(box);
 
         int markerStartPosition = std::max<int>(marker->startOffset() - textBox->start(), 0);
         int markerEndPosition = std::min<int>(marker->endOffset() - textBox->start(), textBox->len());
 
         if (markerStartPosition >= markerEndPosition)
             continue;
 
-        const Vector<SVGTextFragment>& fragments = textBox->textFragments();
-        unsigned textFragmentsSize = fragments.size();
+        Vector<SVGTextFragmentWithRange> fragmentList = collectFragments(markerStartPosition, markerEndPosition);
+        unsigned textFragmentsSize = fragmentList.size();
         for (unsigned i = 0; i < textFragmentsSize; ++i) {
-            const SVGTextFragment& fragment = fragments.at(i);
+            textMatchInfoList.append(fragmentList.at(i));

[fs, 2015/12/17 10:10:55]

Use appendVector or appendRange instead. Per above it shouldn't be needed
though.

[ramya.v, 2015/12/18 03:43:32]

Done.

+        }
+    }
+    return textMatchInfoList;
+}
 
-            int fragmentStartPosition = markerStartPosition;
-            int fragmentEndPosition = markerEndPosition;
-            if (!textBox->mapStartEndPositionsIntoFragmentCoordinates(fragment, fragmentStartPosition, fragmentEndPosition))
-                continue;
+Vector<SVGTextFragmentWithRange> SVGInlineTextBoxPainter::collectFragments(int startPosition, int endPosition)

[fs, 2015/12/17 10:10:54]

collectFragmentsInRange or something more narrow like that seems more
appropriate.

That being said though, I'm a bit irked by the potential for allocator churn
here (especially for the <textPath> case where there'll potentially be lots of
fragments.) I guess we can wish for few fragments for now though...

Could probably use a "functional" iteration style, but let's go with this for
now.

[ramya.v, 2015/12/18 03:43:33]

Done.

+{
+    Vector<SVGTextFragmentWithRange> fragmentInfoList;
+    const Vector<SVGTextFragment>& fragments = m_svgInlineTextBox.textFragments();
+    unsigned textFragmentsSize = fragments.size();
+    for (unsigned i = 0; i < textFragmentsSize; ++i) {

[fs, 2015/12/17 10:10:55]

Use a range-based for-loop.

[ramya.v, 2015/12/18 03:43:32]

Done.

+        const SVGTextFragment& fragment = fragments.at(i);
 
-            FloatRect fragmentRect = textBox->selectionRectForTextFragment(fragment, fragmentStartPosition, fragmentEndPosition, style);
-            fragment.buildFragmentTransform(fragmentTransform);
+        int fragmentStartPosition = startPosition;
+        int fragmentEndPosition = endPosition;
+        if (!m_svgInlineTextBox.mapStartEndPositionsIntoFragmentCoordinates(fragment, fragmentStartPosition, fragmentEndPosition))
+            continue;
 
-            // Draw the marker highlight.
-            if (LineLayoutPaintShim::layoutObjectFrom(m_svgInlineTextBox.lineLayoutItem())->frame()->editor().markedTextMatchesAreHighlighted()) {
-                Color color = marker->activeMatch() ?
-                    LayoutTheme::theme().platformActiveTextSearchHighlightColor() :
-                    LayoutTheme::theme().platformInactiveTextSearchHighlightColor();
-                GraphicsContextStateSaver stateSaver(context);
-                if (!fragmentTransform.isIdentity())
-                    context.concatCTM(fragmentTransform);
-                context.setFillColor(color);
-                context.fillRect(fragmentRect, color);
-            }
-        }
+        SVGTextFragmentWithRange fragmentInfo;
+        fragmentInfo.fragment = &fragment;
+        fragmentInfo.fragmentStartPosition = fragmentStartPosition;
+        fragmentInfo.fragmentEndPosition = fragmentEndPosition;
+
+        fragmentInfoList.append(fragmentInfo);

[fs, 2015/12/17 10:10:55]

With a constructor for SVGTextFragmentWithRange, this would turn into:

fragmentInfoList.append(SVGTextFragmentWithRange(fragment, ..., ...));

[ramya.v, 2015/12/18 03:43:33]

Done.

+    }
+    return fragmentInfoList;
+}
+
+void SVGInlineTextBoxPainter::paintTextMatchMarkerForeground(const PaintInfo& paintInfo, const LayoutPoint& point, DocumentMarker* marker, const ComputedStyle& style, const Font& font)
+{
+    Vector<SVGTextFragmentWithRange> textMatchInfoList = collectTextMatches(paintInfo, marker);
+    unsigned textFragmentsSize = textMatchInfoList.size();
+    if (textFragmentsSize == 0)
+        return;
+
+    Color textColor = marker->activeMatch() ?

[fs, 2015/12/17 10:10:55]

Same comment as for other instances of this.

[ramya.v, 2015/12/18 03:43:33]

Done.

+        LayoutTheme::theme().platformTextSearchColor(true) :
+        LayoutTheme::theme().platformTextSearchColor(false);
+
+    AffineTransform fragmentTransform;
+    for (unsigned i = 0; i < textFragmentsSize; ++i) {

[fs, 2015/12/17 10:10:55]

Use range-based for-loop.

[ramya.v, 2015/12/18 03:43:33]

Done.

+        SVGTextFragmentWithRange& textMatchInfo = textMatchInfoList.at(i);
+        const SVGTextFragment& fragment = *(textMatchInfo.fragment);
+        GraphicsContextStateSaver stateSaver(paintInfo.context);
+        fragment.buildFragmentTransform(fragmentTransform);
+        if (!fragmentTransform.isIdentity())
+            paintInfo.context.concatCTM(fragmentTransform);
+        RefPtr<ComputedStyle> markerStyle = ComputedStyle::clone(style);
+        markerStyle->setFillPaintColor(textColor);
+        TextRun textRun = m_svgInlineTextBox.constructTextRun(*markerStyle, fragment);
+        paintTextWithShadows(paintInfo, *markerStyle, textRun, fragment, textMatchInfo.fragmentStartPosition, textMatchInfo.fragmentEndPosition, ApplyToFillMode);
+    }
+}
+
+void SVGInlineTextBoxPainter::paintTextMatchMarkerBackground(const PaintInfo& paintInfo, const LayoutPoint& point, DocumentMarker* marker, const ComputedStyle& style, const Font& font)
+{
+    Vector<SVGTextFragmentWithRange> textMatchInfoList = collectTextMatches(paintInfo, marker);
+    unsigned textFragmentsSize = textMatchInfoList.size();
+    if (textFragmentsSize == 0)
+        return;
+
+    Color color = marker->activeMatch() ?

[fs, 2015/12/17 10:10:55]

Again.

[ramya.v, 2015/12/18 03:43:32]

Done.

+        LayoutTheme::theme().platformTextSearchHighlightColor(true) :
+        LayoutTheme::theme().platformTextSearchHighlightColor(false);
+
+    AffineTransform fragmentTransform;
+    for (unsigned i = 0; i < textFragmentsSize; ++i) {

[fs, 2015/12/17 10:10:54]

Use range-based for-loop.

[ramya.v, 2015/12/18 03:43:33]

Done.

+        SVGTextFragmentWithRange& textMatchInfo = textMatchInfoList.at(i);
+        const SVGTextFragment& fragment = *(textMatchInfo.fragment);
+        GraphicsContextStateSaver stateSaver(paintInfo.context);
+        fragment.buildFragmentTransform(fragmentTransform);
+        if (!fragmentTransform.isIdentity())
+            paintInfo.context.concatCTM(fragmentTransform);
+        FloatRect fragmentRect = m_svgInlineTextBox.selectionRectForTextFragment(fragment, textMatchInfo.fragmentStartPosition, textMatchInfo.fragmentEndPosition, style);
+        paintInfo.context.setFillColor(color);
+        paintInfo.context.fillRect(fragmentRect, color);
     }
 }
 
 } // namespace blink