Rewrite DocumentMarkerController to use SynchronousMutationObserver

third_party/WebKit/Source/core/dom/Document.cpp

Index: third_party/WebKit/Source/core/dom/Document.cpp
diff --git a/third_party/WebKit/Source/core/dom/Document.cpp b/third_party/WebKit/Source/core/dom/Document.cpp
index 77c1366b3133990800bad1df47978b3df3500aa2..3eda52c83139010f0ca7f0a236c090da5510663d 100644
--- a/third_party/WebKit/Source/core/dom/Document.cpp
+++ b/third_party/WebKit/Source/core/dom/Document.cpp
@@ -4256,16 +4256,11 @@ void Document::nodeWillBeRemoved(Node& n) {
 void Document::didInsertText(Node* text, unsigned offset, unsigned length) {
   for (Range* range : m_ranges)

[yosin_UTC9, 2017/02/15 02:23:25]

Note: We'll change Range to use didUpdateCharacterData() pattern as same as SMO.
Note: Relocating Range should be done before other observers. I'm still thinking
how to utilize SMO for Range list but I don't have cleaner way to do so yet.

     range->didInsertText(text, offset, length);
-
-  m_markers->shiftMarkers(text, offset, length);
 }
 
 void Document::didRemoveText(Node* text, unsigned offset, unsigned length) {
   for (Range* range : m_ranges)
     range->didRemoveText(text, offset, length);
-
-  m_markers->removeMarkers(text, offset, length);
-  m_markers->shiftMarkers(text, offset + length, 0 - length);
 }
 
 void Document::didMergeTextNodes(const Text& mergedNode,