[DMC #27] Move TextMatchMarker method implementations to TextMatchMarker.cpp

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 e4c9fc2f738e39c636aea2e529b5789dff70087d..750c5155e37006d2ecc1768c659947977f4b6058 100644
--- a/third_party/WebKit/Source/core/editing/markers/TextMatchMarker.h
+++ b/third_party/WebKit/Source/core/editing/markers/TextMatchMarker.h
@@ -43,47 +43,25 @@ class CORE_EXPORT TextMatchMarker final : public DocumentMarker {
  public:
   enum class MatchStatus { kInactive, kActive };
 
-  TextMatchMarker(unsigned start_offset,
-                  unsigned end_offset,
-                  MatchStatus status)
-      : DocumentMarker(start_offset, end_offset), match_status_(status) {
-    layout_state_ = State::kInvalid;
-  }
+  TextMatchMarker(unsigned start_offset, unsigned end_offset, MatchStatus);
 
   // 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;
-  }
-
-  bool IsRendered() const { return layout_state_ == State::kValidNotNull; }
-  bool Contains(const LayoutPoint& point) const {
-    DCHECK_EQ(layout_state_, State::kValidNotNull);
-    return rendered_rect_.Contains(point);
-  }
-  void SetRenderedRect(const LayoutRect& rect) {
-    if (layout_state_ == State::kValidNotNull && rect == rendered_rect_)
-      return;
-    layout_state_ = State::kValidNotNull;
-    rendered_rect_ = rect;
-  }
-
-  const LayoutRect& RenderedRect() const {
-    DCHECK_EQ(layout_state_, State::kValidNotNull);
-    return rendered_rect_;
-  }
-
-  void NullifyRenderedRect() {
-    layout_state_ = State::kValidNull;
-    // Now |m_renderedRect| can not be accessed until |setRenderedRect| is
-    // called.
-  }
-
-  void Invalidate() { layout_state_ = State::kInvalid; }
-  bool IsValid() const { return layout_state_ != State::kInvalid; }
+  bool IsActiveMatch() const;
+  void SetIsActiveMatch(bool active);
+
+  bool IsRendered() const;
+  bool Contains(const LayoutPoint&) const;
+  void SetRenderedRect(const LayoutRect&);
+

[Xiaocheng, 2017/06/01 21:42:50]

nit: Could you remove the empty lines at L58, L60 and L62?

+  const LayoutRect& RenderedRect() const;
+
+  void NullifyRenderedRect();
+
+  void Invalidate();
+  bool IsValid() const;
 
  private:
   MatchStatus match_status_;