[DMC #25] Make DocumentMarker::GetType() virtual

third_party/WebKit/Source/core/editing/markers/TextMatchMarker.h

Index: third_party/WebKit/Source/core/editing/markers/TextMatchMarker.h
diff --git a/third_party/WebKit/Source/core/editing/markers/TextMatchMarker.h b/third_party/WebKit/Source/core/editing/markers/TextMatchMarker.h
index 4422679bb954e8e7770a519bf38d24474cb7428b..d950f30bd7eca01b357bc462a141c5d4d24b8482 100644
--- a/third_party/WebKit/Source/core/editing/markers/TextMatchMarker.h
+++ b/third_party/WebKit/Source/core/editing/markers/TextMatchMarker.h
@@ -36,7 +36,7 @@ namespace blink {
 // markers. We store whether or not the match is active, a LayoutRect used for
 // rendering the marker, and whether or not the LayoutRect is currently
 // up-to-date.
-class TextMatchMarker final : public DocumentMarker {
+class CORE_EXPORT TextMatchMarker final : public DocumentMarker {
  private:
   enum class State { kInvalid, kValidNull, kValidNotNull };
 
@@ -46,11 +46,14 @@ class TextMatchMarker final : public DocumentMarker {
   TextMatchMarker(unsigned start_offset,

[yosin_UTC9, 2017/06/01 05:48:59]

In following patch, could you move inline functions in |TextMatchMarker| class
to "TextMatchMaker.cpp"?
Because of they are not performance sensitive, and we want to have small header
file for fast compilation.

                   unsigned end_offset,
                   MatchStatus status)
-      : DocumentMarker(DocumentMarker::kTextMatch, start_offset, end_offset),
-        match_status_(status) {
+      : DocumentMarker(start_offset, end_offset), match_status_(status) {
     layout_state_ = State::kInvalid;
   }
 
+  // DocumentMarker implementations
+  MarkerType GetType() const final;
+
+  // TextMatchMarker-specific
   bool IsActiveMatch() const { return match_status_ == MatchStatus::kActive; }
   void SetIsActiveMatch(bool active) {
     match_status_ = active ? MatchStatus::kActive : MatchStatus::kInactive;