 Chromium Code Reviews
 Chromium Code Reviews Issue 22875013:
  Make several DOMImplementation API arguments mandatory  (Closed) 
  Base URL: svn://svn.chromium.org/blink/trunk
    
  
    Issue 22875013:
  Make several DOMImplementation API arguments mandatory  (Closed) 
  Base URL: svn://svn.chromium.org/blink/trunk| OLD | NEW | 
|---|---|
| 1 <script> | 1 <script> | 
| 2 if (window.testRunner) | 2 if (window.testRunner) | 
| 3 testRunner.dumpAsText(); | 3 testRunner.dumpAsText(); | 
| 4 | 4 | 
| 5 root = document.createElement("body"); | 5 root = document.createElement("body"); | 
| 6 d = document.implementation.createDocument(); | 6 d = document.implementation.createDocument("", ""); | 
| 
arv (Not doing code reviews)
2013/08/16 13:33:27
FYI, (null, "") is a better default
 
do-not-use
2013/08/16 13:47:03
Right, my bad, I misread the spec.
 | |
| 7 d.adoptNode(root); | 7 d.adoptNode(root); | 
| 8 node = document.createElement("title"); | 8 node = document.createElement("title"); | 
| 9 root.appendChild(node); | 9 root.appendChild(node); | 
| 10 node.setAttribute("style", "zoom:document;") | 10 node.setAttribute("style", "zoom:document;") | 
| 11 node.appendChild(node.cloneNode()); | 11 node.appendChild(node.cloneNode()); | 
| 12 </script> | 12 </script> | 
| 13 <body> | 13 <body> | 
| 14 This test passes if it does not crash. | 14 This test passes if it does not crash. | 
| 15 </body> | 15 </body> | 
| OLD | NEW |