Closed
Bug 910696
Opened 11 years ago
Closed 6 years ago
Further expand the variables view when appropriate
Categories
(DevTools :: Debugger, defect, P5)
Tracking
(Not tracked)
RESOLVED
DUPLICATE
of bug 1448166
People
(Reporter: fitzgen, Unassigned)
References
(Blocks 1 open bug)
Details
It is annoying that when I go into a new frame, only the youngest environment is expanded in the variables view. Often it is a block scope with only one or two bindings. We should expand all the environments, so that you can actually see the bindings.
Comment 1•11 years ago
|
||
Note that doing that would make things slower, obviously. But maybe if we just omit the global scope it will be OK. Some experimentation is warranted.
Reporter | ||
Comment 2•11 years ago
|
||
Victor and I talked about continuing to expand frames while the total number of variables shown was less than N (excluding the global scope, of course). I think this is a pretty good way to control (and experiment with) the performance trade off.
Comment 3•11 years ago
|
||
From what I talked to Nick in Paris, a better plan would be to use some heuristics, not just blindingly expand all environments all the time.
We should keep the current behavior, expect the following cases, in which it'd be helpful if we expanded everything (maybe not the global scope though?):
* selecting a different stackframe
* having less than n enumerable variables in the local scope
* (?) if the local scope is a |with| environment
Reporter | ||
Updated•11 years ago
|
Assignee: nobody → nfitzgerald
Priority: -- → P2
Reporter | ||
Comment 4•11 years ago
|
||
I'm unassigning myself because I won't be working on this anytime soon.
Assignee: nfitzgerald → nobody
Comment 5•11 years ago
|
||
I fixed this in bug 758157 (but entirely accidentally, so without taking into consideration my own fears, described from comment 3), and quickly realized that it can be a significant performance regression in some cases, which I fixed in bug 952767.
All I can say is, let's be very careful when actually fixing this bug :)
Updated•10 years ago
|
Summary: expand the variables view by default → Further expand the variables view when appropriate
Updated•10 years ago
|
Blocks: dbg-frontend
Updated•6 years ago
|
Product: Firefox → DevTools
Updated•6 years ago
|
Priority: P2 → P5
Comment 6•6 years ago
|
||
closing as I believe it is addressed by bug 1448166
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → DUPLICATE
You need to log in
before you can comment on or make changes to this bug.
Description
•