Oilpan: Introduce TempRangeScope to avoid needless Range attaches on Document.

Source/core/dom/Range.cpp

 /*
  * (C) 1999 Lars Knoll (knoll@kde.org)
  * (C) 2000 Gunnstein Lye (gunnstein@netcom.no)
  * (C) 2000 Frederik Holljen (frederik.holljen@hig.no)
  * (C) 2001 Peter Kelly (pmk@post.com)
  * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2010, 2011 Apple Inc. All rights reserved.
  * Copyright (C) 2011 Motorola Mobility. All rights reserved.
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
@@ skipping 39 matching lines @@
 #include "wtf/text/CString.h"
 #include "wtf/text/StringBuilder.h"
 #ifndef NDEBUG
 #include <stdio.h>
 #endif
 
 namespace WebCore {
 
 DEFINE_DEBUG_ONLY_GLOBAL(WTF::RefCountedLeakCounter, rangeCounter, ("Range"));
 
+#if ENABLE(OILPAN)
+unsigned TemporaryRangeScope::s_nesting = 0;
+#endif
+
 inline Range::Range(Document& ownerDocument)
     : m_ownerDocument(&ownerDocument)
     , m_start(m_ownerDocument)
     , m_end(m_ownerDocument)
 {
 #ifndef NDEBUG
     rangeCounter.increment();
 #endif
-
     ScriptWrappable::init(this);
-    m_ownerDocument->attachRange(this);
+    if (!TemporaryRangeScope::active())

[haraken, 2014/06/24 12:24:22]

You can move this check into Document::attachRange, since we're verifying
!TemporaryRangeScope::active() in Document's methods.

+        m_ownerDocument->attachRange(this);
 }
 
 PassRefPtrWillBeRawPtr<Range> Range::create(Document& ownerDocument)
 {
     return adoptRefWillBeNoop(new Range(ownerDocument));
 }
 
 inline Range::Range(Document& ownerDocument, Node* startContainer, int startOffset, Node* endContainer, int endOffset)
     : m_ownerDocument(&ownerDocument)
     , m_start(m_ownerDocument)
     , m_end(m_ownerDocument)
 {
 #ifndef NDEBUG
     rangeCounter.increment();
 #endif
-
     ScriptWrappable::init(this);
-    m_ownerDocument->attachRange(this);
+    if (!TemporaryRangeScope::active())

[haraken, 2014/06/24 12:24:22]

Ditto.

+        m_ownerDocument->attachRange(this);
 
     // Simply setting the containers and offsets directly would not do any of the checking
     // that setStart and setEnd do, so we call those functions.
     setStart(startContainer, startOffset);
     setEnd(endContainer, endOffset);
 }
 
 PassRefPtrWillBeRawPtr<Range> Range::create(Document& ownerDocument, Node* startContainer, int startOffset, Node* endContainer, int endOffset)
 {
     return adoptRefWillBeNoop(new Range(ownerDocument, startContainer, startOffset, endContainer, endOffset));
@@ skipping 1614 matching lines @@
 
 void showTree(const WebCore::Range* range)
 {
     if (range && range->boundaryPointsValid()) {
         range->startContainer()->showTreeAndMark(range->startContainer(), "S", range->endContainer(), "E");
         fprintf(stderr, "start offset: %d, end offset: %d\n", range->startOffset(), range->endOffset());
     }
 }
 
 #endif