Closed Bug 855971 Opened 7 years ago Closed 7 years ago
Switch HTMLDocument to Web
The bug we used to have on this got repurposed.
Assignee: nobody → peterv
Status: NEW → ASSIGNED
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.
Attachment #743209 - Flags: review?(bzbarsky) → review+
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
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla23
You need to log in before you can comment on or make changes to this bug.