[DMC #18] Eliminate DocumentMarkerTextMatch (subclass of DocumentMarkerDetails)

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

 /*
  * Copyright (C) 2011 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Neither the name of Google Inc. nor the names of its
@@ skipping 22 matching lines @@
 namespace blink {
 
 class TextMatchMarker final : public DocumentMarker {
  private:
   enum class State { kInvalid, kValidNull, kValidNotNull };
 
  public:
   TextMatchMarker(unsigned start_offset,
                   unsigned end_offset,
                   MatchStatus status)
-      : DocumentMarker(start_offset, end_offset, status),
+      : DocumentMarker(DocumentMarker::kTextMatch, start_offset, end_offset),
+        status_(status),
         state_(State::kInvalid) {}
 
+  bool IsActiveMatch() const { return status_ == MatchStatus::kActive; }
+  void SetIsActiveMatch(bool active) {
+    status_ = active ? MatchStatus::kActive : MatchStatus::kInactive;
+  }
+
   bool IsRendered() const { return state_ == State::kValidNotNull; }
   bool Contains(const LayoutPoint& point) const {
     DCHECK_EQ(state_, State::kValidNotNull);
     return rendered_rect_.Contains(point);
   }
   void SetRenderedRect(const LayoutRect& rect) {
     if (state_ == State::kValidNotNull && rect == rendered_rect_)
       return;
     state_ = State::kValidNotNull;
     rendered_rect_ = rect;
   }
 
   const LayoutRect& RenderedRect() const {
     DCHECK_EQ(state_, State::kValidNotNull);
     return rendered_rect_;
   }
 
   void NullifyRenderedRect() {
     state_ = State::kValidNull;
     // Now |m_renderedRect| can not be accessed until |setRenderedRect| is
     // called.
   }
 
   void Invalidate() { state_ = State::kInvalid; }
   bool IsValid() const { return state_ != State::kInvalid; }
 
  private:
-  explicit TextMatchMarker(const DocumentMarker& marker)
-      : DocumentMarker(marker), state_(State::kInvalid) {}
+  MatchStatus status_;

[yosin_UTC9, 2017/05/26 04:39:47]

nit: s/status_/match_status_/

It is confusing to have both |state_| and |status_|.

We may also want to rename |state_| to |layout_state_|.

 
   LayoutRect rendered_rect_;
   State state_;
 };
 
 DEFINE_TYPE_CASTS(TextMatchMarker, DocumentMarker, marker, true, true);
 
 }  // namespace blink
 
 WTF_ALLOW_MOVE_INIT_AND_COMPARE_WITH_MEM_FUNCTIONS(blink::TextMatchMarker);
 
 #endif