Closed Bug 910696 Opened 10 years ago Closed 4 years ago

Further expand the variables view when appropriate


(DevTools :: Debugger, defect, P5)



(Not tracked)



(Reporter: fitzgen, Unassigned)


(Blocks 1 open bug)


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.
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.
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.
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
Assignee: nobody → nfitzgerald
Priority: -- → P2
I'm unassigning myself because I won't be working on this anytime soon.
Assignee: nfitzgerald → nobody
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 :)
Summary: expand the variables view by default → Further expand the variables view when appropriate
Product: Firefox → DevTools
Priority: P2 → P5

closing as I believe it is addressed by bug 1448166

Closed: 4 years ago
Resolution: --- → DUPLICATE
Blocks: 1565711
Blocks: 1565713
No longer blocks: 1565711
No longer blocks: 1565713
You need to log in before you can comment on or make changes to this bug.