Make removeChild, parserRemoveChild and removeChildren more consistent.

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 4076 matching lines @@
   DCHECK_NE(root.document(), this);
   if (m_ranges.isEmpty())
     return;
 
   AttachedRangeSet ranges = m_ranges;
   for (Range* range : ranges)
     range->updateOwnerDocumentIfNeeded();
 }
 
 void Document::nodeChildrenWillBeRemoved(ContainerNode& container) {
+  ScriptForbiddenScope scriptForbiddenScope;
   EventDispatchForbiddenScope assertNoEventDispatch;
+
   for (Range* range : m_ranges)
     range->nodeChildrenWillBeRemoved(container);
 
   for (NodeIterator* ni : m_nodeIterators) {
     for (Node& n : NodeTraversal::childrenOf(container))
       ni->nodeWillBeRemoved(n);
   }
 
   if (LocalFrame* frame = this->frame()) {
     frame->selection().nodeChildrenWillBeRemoved(container);
     for (Node& n : NodeTraversal::childrenOf(container)) {
       frame->eventHandler().nodeWillBeRemoved(n);
       frame->page()->dragCaretController().nodeWillBeRemoved(n);
     }
   }
 
   if (containsV1ShadowTree()) {
     for (Node& n : NodeTraversal::childrenOf(container))
       n.checkSlotChangeBeforeRemoved();
   }
 }
 
 void Document::nodeWillBeRemoved(Node& n) {
+  ScriptForbiddenScope scriptForbiddenScope;
+  EventDispatchForbiddenScope assertNoEventDispatch;
+
   for (NodeIterator* ni : m_nodeIterators)
     ni->nodeWillBeRemoved(n);
 
   for (Range* range : m_ranges)
     range->nodeWillBeRemoved(n);
 
   if (LocalFrame* frame = this->frame()) {
     frame->eventHandler().nodeWillBeRemoved(n);
     frame->selection().nodeWillBeRemoved(n);
     frame->page()->dragCaretController().nodeWillBeRemoved(n);
@@ skipping 2333 matching lines @@
 }
 
 void showLiveDocumentInstances() {
   WeakDocumentSet& set = liveDocumentSet();
   fprintf(stderr, "There are %u documents currently alive:\n", set.size());
   for (Document* document : set)
     fprintf(stderr, "- Document %p URL: %s\n", document,
             document->url().getString().utf8().data());
 }
 #endif