Range: surroundContents() should not check detached CharacterData node beforehand.

third_party/WebKit/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
@@ skipping 1343 matching lines @@
           "The node provided is of type '" + newParent->nodeName() + "'.");
       return;
     case Node::kCdataSectionNode:
     case Node::kCommentNode:
     case Node::kElementNode:
     case Node::kProcessingInstructionNode:
     case Node::kTextNode:
       break;
   }
 
-  // Raise a HierarchyRequestError if m_start.container() doesn't accept
-  // children like newParent.
-  Node* parentOfNewParent = m_start.container();
-
-  // If m_start.container() is a character data node, it will be split and it
-  // will be its parent that will need to accept newParent (or in the case of a
-  // comment, it logically "would" be inserted into the parent, although this
-  // will fail below for another reason).
-  if (parentOfNewParent->isCharacterDataNode())
-    parentOfNewParent = parentOfNewParent->parentNode();
-
-  if (!parentOfNewParent) {
-    exceptionState.throwDOMException(HierarchyRequestError,
-                                     "The container node is a detached "
-                                     "character data node; no parent node is "
-                                     "available for insertion.");
-    return;
-  }
-
   EventQueueScope scope;
 
   // 3. Let fragment be the result of extracting context object.
   DocumentFragment* fragment = extractContents(exceptionState);
   if (exceptionState.hadException())
     return;
 
   // 4. If newParent has children, replace all with null within newParent.
   while (Node* n = newParent->firstChild()) {
     toContainerNode(newParent)->removeChild(n, exceptionState);
@@ skipping 427 matching lines @@
                      .data()
               << "start offset: " << range->startOffset()
               << ", end offset: " << range->endOffset();
   } else {
     LOG(INFO) << "Cannot show tree if range is null, or if boundary points are "
                  "invalid.";
   }
 }
 
 #endif