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

third_party/WebKit/Source/core/paint/SVGInlineTextBoxPainter.h

 // 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.
 
 #ifndef SVGInlineTextBoxPainter_h
 #define SVGInlineTextBoxPainter_h
 
 #include "core/style/ComputedStyleConstants.h"
 #include "core/layout/svg/LayoutSVGResourcePaintServer.h"
 #include "wtf/Allocator.h"
 
 namespace blink {
 
 class FloatPoint;
 class Font;
 class GraphicsContext;
 struct PaintInfo;
 class LayoutPoint;
 class ComputedStyle;
 class SVGInlineTextBox;
 struct SVGTextFragment;
 class TextRun;
 class DocumentMarker;
 
+struct SVGTextFragmentWithRange {
+    const SVGTextFragment* fragment;

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

Maybe add a constructor and make this a reference instead.

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

Done.

+    int fragmentStartPosition;

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

I don't think we need to be quite this verbose. Dropping the 'fragment' prefix
seems entirely reasonable to me (it should be possible to infer from the
context).

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

Done.

+    int fragmentEndPosition;
+};
+
 class SVGInlineTextBoxPainter {
     STACK_ALLOCATED();
 public:
     SVGInlineTextBoxPainter(const SVGInlineTextBox& svgInlineTextBox) : m_svgInlineTextBox(svgInlineTextBox) { }
     void paint(const PaintInfo&, const LayoutPoint&);
     void paintSelectionBackground(const PaintInfo&);
-    virtual void paintTextMatchMarker(GraphicsContext&, const LayoutPoint&, DocumentMarker*, const ComputedStyle&, const Font&);
+    virtual void paintTextMatchMarkerForeground(const PaintInfo&, const LayoutPoint&, DocumentMarker*, const ComputedStyle&, const Font&);
+    virtual void paintTextMatchMarkerBackground(const PaintInfo&, const LayoutPoint&, DocumentMarker*, const ComputedStyle&, const Font&);

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

Why are these virtual?

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

Done.

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

Since I've split paintTextMatchMarker took these in the same line. Since there
is no subclass to this class which needs to override this, removing for now.

 
 private:
     bool shouldPaintSelection(const PaintInfo&) const;
     void paintTextFragments(const PaintInfo&, LayoutObject&);
     void paintDecoration(const PaintInfo&, TextDecoration, const SVGTextFragment&);
     void paintTextWithShadows(const PaintInfo&, const ComputedStyle&, TextRun&, const SVGTextFragment&, int startPosition, int endPosition, LayoutSVGResourceMode);
     void paintText(const PaintInfo&, const ComputedStyle&, const ComputedStyle& selectionStyle, const SVGTextFragment&, LayoutSVGResourceMode, bool shouldPaintSelection);
+    Vector<SVGTextFragmentWithRange> collectTextMatches(const PaintInfo&, DocumentMarker*);
+    Vector<SVGTextFragmentWithRange> collectFragments(int startPosition, int endPosition);
 
     const SVGInlineTextBox& m_svgInlineTextBox;
 };
 
 } // namespace blink
 
 #endif // SVGInlineTextBoxPainter_h