Changeset 29849 in webkit

Timestamp:

Jan 29, 2008, 1:35:07 PM (17 years ago)

Author:

Adam Roben

Message:

Fix Bug 16234: Inspector should support searching for elements by CSS selectors

<​http://bugs.webkit.org/show_bugs.cgi?id=16234>
<rdar://5712862>

Reviewed by Tim.

• page/inspector/inspector.js: Use Document.querySelectorAll to search for elements by CSS selector. Also store a custom property on nodes being added to the search results to avoid showing the same node more than once.

Location:

trunk/WebCore

Files:

• ChangeLog (modified) (1 diff)
• page/inspector/inspector.js (modified) (1 diff)

trunk/WebCore/ChangeLog

@@ +1 @@
+2008-01-29  Adam Roben  <aroben@apple.com>
+
+        Fix Bug 16234: Inspector should support searching for elements by CSS selectors
+
+        <http://bugs.webkit.org/show_bugs.cgi?id=16234>
+        <rdar://5712862>
+
+        Reviewed by Tim.
+
+        * page/inspector/inspector.js: Use Document.querySelectorAll to search
+        for elements by CSS selector. Also store a custom property on nodes
+        being added to the search results to avoid showing the same node more
+        than once.
+
 2008-01-29  Adam Roben  <aroben@apple.com>
 

trunk/WebCore/page/inspector/inspector.js

@@ -870 +870 @@
 
         var domResults = [];
+        const searchResultsProperty = "__includedInInspectorSearchResults";
+        function addNodesToDOMResults(nodes, length, getItem)
+        {
+            for (var i = 0; i < length; ++i) {
+                var node = getItem(nodes, i);
+                if (searchResultsProperty in node)
+                    continue;
+                node[searchResultsProperty] = true;
+                domResults.push(node);
+            }
+        }
+
+        function cleanUpDOMResultsNodes()
+        {
+            for (var i = 0; i < domResults.length; ++i)
+                delete domResults[i][searchResultsProperty];
+        }
+
         if (resource.category === this.resourceCategories.documents) {
+            var doc = resource.documentNode;
             try {
-                var doc = resource.documentNode;
-                var nodeList = doc.evaluate(xpathQuery, doc, null, XPathResult.ORDERED_NODE_SNAPSHOT_TYPE);
-                for (var j = 0; j < nodeList.snapshotLength; ++j)
-                    domResults.push(nodeList.snapshotItem(i));
+                var result = doc.evaluate(xpathQuery, doc, null, XPathResult.ORDERED_NODE_SNAPSHOT_TYPE);
+                addNodesToDOMResults(result, result.snapshotLength, function(l, i) { return l.snapshotItem(i); });
             } catch(err) {
                 // ignore any exceptions. the query might be malformed, but we allow that.
             }
+
+            var result = doc.querySelectorAll(query);
+            addNodesToDOMResults(result, result.length, function(l, i) { return l.item(i); });
+
+            cleanUpDOMResultsNodes();
         }