Make default for deep parameter in importNode false

Source/core/dom/Document.cpp

Index: Source/core/dom/Document.cpp
diff --git a/Source/core/dom/Document.cpp b/Source/core/dom/Document.cpp
index 56203732bc1f5132cde33a1d48691e506bdb9391..93a0144edc62c2b37ff2428af01c4b14176aa135 100644
--- a/Source/core/dom/Document.cpp
+++ b/Source/core/dom/Document.cpp
@@ -914,6 +914,12 @@ bool Document::importContainerNodeChildren(ContainerNode* oldContainerNode, Pass
     return true;
 }
 
+PassRefPtr<Node> Document::importNode(Node* importedNode, ExceptionState& ec)
+{
+    addConsoleMessage(JSMessageSource, WarningMessageLevel, "The behavior of importNode() with no boolean argument is about to change from doing a deep clone to doing a shallow clone.  Make sure to pass an explicit boolean argument to keep your current behavior.");

[Inactive, 2014/04/30 14:58:28]

Don't we want to use a UseCounter so that we are sure not too many websites are
calling importNode without the deep argument?

+    return importNode(importedNode, false, ec);

[Inactive, 2014/04/30 14:58:28]

Well, it looks like you are still changing the behavior? :D

+}
+
 PassRefPtr<Node> Document::importNode(Node* importedNode, bool deep, ExceptionState& exceptionState)
 {
     if (!importedNode) {