Chromium Code Reviews Chromium Code Reviews
Issue 18732004:
Add fast path for querySelector(All) (Closed)
Base URL: svn://svn.chromium.org/blink/trunk| OLD | NEW |
|---|---|
| (Empty) | |
| 1 <!doctype html> | |
| 2 <html> | |
| 3 <head> | |
| 4 <script src="../../js/resources/js-test-pre.js"></script> | |
| 5 </head> | |
| 6 <body> | |
| 7 <div id="test"> | |
| 8 <div></div> | |
| 9 <div> | |
| 10 <div class="target"></div> | |
| 11 </div> | |
| 12 <div> | |
| 13 <div class="target"></div> | |
| 14 <div></div> | |
| 15 </div> | |
| 16 <div id="test2" class="target">line 4</div> | |
| 17 <div id="test3"></div> | |
| 18 <p>line 5</p> | |
| 19 </div> | |
| 20 <pre id="console"></pre> | |
| 21 <script> | |
| 22 Element.prototype.dump = function() | |
| 23 { | |
| 24 return this; | |
|
esprehn
2013/07/25 23:11:27
You don't call this anywhere.
tasak
2013/07/26 04:38:52
Done.
| |
| 25 } | |
| 26 | |
| 27 NodeList.prototype.dump = function() | |
| 28 { | |
| 29 var result = ""; | |
| 30 var length = this.length; | |
| 31 for (var i = 0; i < length; i++) | |
| 32 result += this[i] + ", "; | |
| 33 result += "length: " + length; | |
| 34 return result; | |
| 35 } | |
| 36 | |
| 37 function testQuerySelectorAll(node, selectorString) | |
| 38 { | |
| 39 try { | |
| 40 return node.querySelectorAll(selectorString).dump(); | |
| 41 } catch (e) { | |
| 42 return "" + e; | |
| 43 } | |
| 44 } | |
| 45 | |
| 46 shouldBeEqualToString('testQuerySelectorAll(document, ".target + div")', "[objec t HTMLDivElement], [object HTMLDivElement], length: 2"); | |
| 47 shouldBeEqualToString('testQuerySelectorAll(document.getElementById("test"), ".t arget + div")', "[object HTMLDivElement], [object HTMLDivElement], length: 2"); | |
| 48 shouldBeEqualToString('testQuerySelectorAll(document.getElementById("test2"), ". target + div")', "length: 0"); | |
| 49 shouldBeEqualToString('testQuerySelectorAll(document.getElementById("test3"), ". target + div")', "length: 0"); | |
| 50 </script> | |
| 51 </body> | |
| 52 <script src="../../js/resources/js-test-post.js"></script> | |
| 53 </html> | |
| OLD | NEW |
