New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Many evaluated expression are listed in the script panel #7954

Open
fflorent opened this Issue Oct 13, 2015 · 2 comments

Comments

Projects
None yet
2 participants
@fflorent
Member

fflorent commented Oct 13, 2015

STR :

  1. Open this page : http://getfirebug.com/
  2. Open Firebug and switch to the Script panel
  3. Place a BP line 2 of jquery.js
  4. Open the Command Line Popup, enter this expression : jQuery()
  5. Hover some variables / expressions in the Source Editor ("a" for example)
    => As you hover expressions, it opens a tooltip with the result of the expression in it.

Actual Result :
The evaluation of the expression adds a new entry in the location menu of the Script panel. It also slows down Firebug.

Expected Result :
No additional entry.

Initially reported here: #7917 (comment)

Florent

@fflorent

This comment has been minimized.

Show comment
Hide comment
@fflorent

fflorent Oct 13, 2015

Member

Try-build here: https://drive.google.com/file/d/0B5NqEPwxS4QwRG1BTUZJWVU1YXM/view?usp=sharing

In order to install it, download the file and open it in Firefox.

Florent

Member

fflorent commented Oct 13, 2015

Try-build here: https://drive.google.com/file/d/0B5NqEPwxS4QwRG1BTUZJWVU1YXM/view?usp=sharing

In order to install it, download the file and open it in Firefox.

Florent

@giovannibgr

This comment has been minimized.

Show comment
Hide comment
@giovannibgr

giovannibgr Oct 14, 2015

After debugging for a while with the new build, I can confirm the entries are gone, thank you.
FB still struggles when I try to select an expression (e.g. i == 0 && next == 1) and hover-evaluate it, but it's around line 21000 of the file, big JS files were always a soft spot for the debugger.

giovannibgr commented Oct 14, 2015

After debugging for a while with the new build, I can confirm the entries are gone, thank you.
FB still struggles when I try to select an expression (e.g. i == 0 && next == 1) and hover-evaluate it, but it's around line 21000 of the file, big JS files were always a soft spot for the debugger.

janodvarko added a commit that referenced this issue Oct 14, 2015

Merge pull request #7955 from firebug/issue7954
#7954 Many evaluated expression are listed in the script panel
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment