DocumentMarkerList refactor as an interface

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
new file mode 100644
index 0000000000000000000000000000000000000000..73140cf3384f8c9c9cdbd06f58365691c5400958
--- /dev/null
+++ b/third_party/WebKit/Source/core/editing/markers/TextMatchMarker.h
@@ -0,0 +1,64 @@
+// Copyright 2017 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 TextMatchMarker_h
+#define TextMatchMarker_h
+
+#include <utility>
+#include "core/editing/markers/DocumentMarker.h"
+#include "platform/geometry/LayoutRect.h"
+
+namespace blink {
+
+class CORE_EXPORT TextMatchMarker final : public DocumentMarker {
+ public:
+  TextMatchMarker(unsigned startOffset,
+                  unsigned endOffset,
+                  DocumentMarker::MatchStatus);
+
+  bool isRendered() const { return m_state == State::ValidNotNull; }
+
+  void setRenderedRect(const LayoutRect& rect) {
+    if (m_state == State::ValidNotNull && rect == m_renderedRect)
+      return;
+    m_state = State::ValidNotNull;
+    m_renderedRect = rect;
+  }
+
+  const LayoutRect& renderedRect() const {
+    DCHECK_EQ(m_state, State::ValidNotNull);
+    return m_renderedRect;
+  }
+
+  void updateRenderedRectIfNeeded(const Node&);
+  void updateRenderedRect(const Node&);
+
+  void nullifyRenderedRect() {
+    m_state = State::ValidNull;
+    // Now |m_renderedRect| can not be accessed until |setRenderedRect| is
+    // called.
+  }
+
+  void invalidate() { m_state = State::Invalid; }
+  bool isValid() const { return m_state != State::Invalid; }
+
+ protected:
+  TextMatchMarker(const TextMatchMarker&) = default;
+
+ private:
+  enum class State { Invalid, ValidNull, ValidNotNull };

[yosin_UTC9, 2017/04/07 03:16:22]

nit: Please add prefix "k" to enum members, e.g. kInvalid, kValidNull, ...

+
+  LayoutRect m_renderedRect;
+  State m_state;
+};
+
+DEFINE_TYPE_CASTS(TextMatchMarker,
+                  DocumentMarker,
+                  marker,
+                  marker->type() == DocumentMarker::TextMatch,
+                  marker.type() == DocumentMarker::TextMatch);
+
+}  // namespace blink
+
+#endif  // TextMatchMarker_h