The bug we used to have on this got repurposed.

Comment on attachment 743209 [details] [diff] [review] v2 >+ is(globalScope.querySelectorAll(".variables-view-variable:not([non-match])").length, 1, > "There should be 2 variables displayed in the global scope"); Fix the test message? >+ is(innerScope.querySelectorAll(".variables-view-property:not([non-match])").length, 7, > "There should be 8 properties displayed in the inner scope"); And here. Do we understand why these were needed? r=me with that.

Backed out because of browser-chrome test failures: https://tbpl.mozilla.org/?tree=Mozilla-Inbound&rev=a2245b038bcc https://hg.mozilla.org/integration/mozilla-inbound/rev/2b63238518a3

Looks like the debugger tests no longer need some of the changes from this bug (the [object Proxy] bit because jimb fixed that bustage, and not sure about the rest). I took out those parts of the patch, verified that the tests that failed when this was pushed now pass for me locally, and pushed https://hg.mozilla.org/integration/mozilla-inbound/rev/adaaf6641785