1214219 - Object inspector sidebar doesn't display "__proto__" property after clearing filter field

Status: NEW

(DevTools :: Storage Inspector, defect, P3)

Description

[arni2033]

> Win7_64, Nightly 44, 32bit, ID 20151012030612, new profile
STR_1:
1. Open Scratchpad (Shift+F4)
2. Type there   "var A = {a:1,b:2,p:3}; A"   w/o quotes, place cursor at the end of the line, press Ctrl+I to inspect object A
3. Type "p" w/o quotes in the filter field in the sidebar
4. Press BackSpace to delete letter typed in Step 3

STR_2:
1.   Open this "data:" url or click URL in the form above
>   data:text/html,<form name="F" onsubmit="e=arguments[0];e.preventDefault();console.log(e)"><input/><input type="submit" name="S"/><script>setTimeout(()=>{document.F.S.click()},1000)</script>
2.   Open console (Ctrl+Shift+K), click "submit" event   [Object inspector sidebar will open]
3.   Focus the filter field in the sidebar
4.A) Press Escape
4.B) Type "p" w/o quotes (or any other letter), then delete it with BackSpace

Result:       "__proto__" disappears from list of object's properties
Expectations: "__proto__" should be displayed when user cancels filtering

It was regressed between 2015-05-27 and 2015-05-28 (as well as some other things):
> pushlog_url:    https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=29a9acde728e4ce55535c4e7695fbb4d076230d6&tochange=fb9275dab645dead337f919548a789c068741c5b
> Last good build url:    https://queue.taskcluster.net/v1/task/Afp-ReWpRuaBcQFPT4AMWA/artifacts/public/build/firefox-41.0a1.en-US.win32.zip
> First bad build url:    https://archive.mozilla.org/pub/mozilla.org/firefox/nightly/2015/05/2015-05-28-03-02-06-mozilla-central/firefox-41.0a1.en-US.win32.zip

Also, please tell if sidebar in console should hide after pressing ESC in the empty filter field (like in Scratchpad)

Comment 1

[Nicolas Chevobbe [:nchevobbe]]

Moving to the DOM panel as it's VariableView related and only the DOM panel uses the VariableView now

Comment 2

[Nicolas Chevobbe [:nchevobbe]]

erratum: this should have been moved to the storage inspector, which is the last consumer of the variable view