Rewrite DocumentMarkerController to use SynchronousMutationObserver

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

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 1999 Antti Koivisto (koivisto@kde.org)
  *           (C) 2001 Dirk Mueller (mueller@kde.org)
  *           (C) 2006 Alexey Proskuryakov (ap@webkit.org)
  * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2011, 2012 Apple Inc. All
  * rights reserved.
  * Copyright (C) 2008, 2009 Torch Mobile Inc. All rights reserved.
  * (http://www.torchmobile.com/)
  * Copyright (C) 2008, 2009, 2011, 2012 Google Inc. All rights reserved.
@@ skipping 4236 matching lines @@
   notifyNodeWillBeRemoved(n);
 
   if (containsV1ShadowTree())
     n.checkSlotChangeBeforeRemoved();
 
   if (n.inActiveDocument() && n.isElementNode())
     styleEngine().elementWillBeRemoved(toElement(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,
                                  const Text& nodeToBeRemoved,
                                  unsigned oldLength) {
   NodeWithIndex nodeToBeRemovedWithIndex(const_cast<Text&>(nodeToBeRemoved));
   if (!m_ranges.isEmpty()) {
     for (Range* range : m_ranges)
       range->didMergeTextNodes(nodeToBeRemovedWithIndex, oldLength);
   }
@@ skipping 2301 matching lines @@
 }
 
 void showLiveDocumentInstances() {
   WeakDocumentSet& set = liveDocumentSet();
   fprintf(stderr, "There are %u documents currently alive:\n", set.size());
   for (blink::Document* document : set)
     fprintf(stderr, "- Document %p URL: %s\n", document,
             document->url().getString().utf8().data());
 }
 #endif