DevTools: Inspect element mode does not select element in ShadowDOM

LayoutTests/http/tests/inspector/elements-test.js

 var initialize_ElementTest = function() {
 
 InspectorTest.findNode = function(matchFunction, callback)
 {
+    var originalCallback = callback;

[vsevik, 2013/06/07 08:41:00]

Unused.

[pfeldman, 2013/06/07 08:53:49]

Done.

     callback = InspectorTest.safeWrap(callback);
     var result = null;
-    var topLevelChildrenRequested = false;
     var pendingRequests = 0;
-    function processChildren(topLevel, children)
+    function processChildren(children)
     {
-        pendingRequests--;
-        if (result)
-            return;
+        try {
+            if (result)
+                return;
 
-        for (var i = 0; children && i < children.length; ++i) {
-            var childNode = children[i];
-            if (matchFunction(childNode)) {
-                result = childNode;
-                callback(result);
-                return;
+            for (var i = 0; children && i < children.length; ++i) {
+                var childNode = children[i];
+                if (matchFunction(childNode)) {
+                    result = childNode;
+                    callback(result);
+                    pendingRequests--;

[vsevik, 2013/06/07 08:41:00]

remove

[pfeldman, 2013/06/07 08:53:49]

Done.

+                    return;
+                }
+                pendingRequests++;
+                childNode.getChildNodes(processChildren);
             }
-            pendingRequests++;
-            childNode.getChildNodes(processChildren.bind(null, false));
+        } finally {

[apavlov, 2013/06/06 16:18:59]

Curlies not necessary here? The new styleguide is a bit ambiguous.

+            pendingRequests--;
         }
 
-        if (topLevel)
-            topLevelChildrenRequested = true;
-        if (topLevelChildrenRequested && !result && !pendingRequests)
+        if (!result && !pendingRequests) {

[apavlov, 2013/06/06 16:18:59]

ditto

[pfeldman, 2013/06/07 08:53:49]

Done.

             callback(null);
+        }
     }
-    pendingRequests++;
 
     WebInspector.domAgent.requestDocument(documentRequested.bind(this));
     function documentRequested(doc)
     {
-        doc.getChildNodes(processChildren.bind(this, true));
+        pendingRequests++;
+        doc.getChildNodes(processChildren);
     }
 };
 
 InspectorTest.nodeWithId = function(idValue, callback)
 {
     function nodeIdMatches(node)
     {
         return node.getAttribute("id") === idValue;
     }
     InspectorTest.findNode(nodeIdMatches, callback);
@@ skipping 563 matching lines @@
 {
     var rectNames = ["margin", "border", "padding", "content"];
     var rects = window.internals.inspectorHighlightRects(document);
     for (var i = 0; i < rects.length; i++)
     {
         var rectName = (i < rectNames.length ? rectNames[i] : "untitled");
         var rect = rects.item(i);
         output(rectName + " rect is " + rect.width + " x " + rect.height + " at (" + rect.left + ", " + rect.top + ")");
     }
 }