Pretty-print button takes effect in a wrong script if I switch to another script

RESOLVED FIXED

Status

DevTools
Debugger
RESOLVED FIXED
2 years ago
5 days ago

People

(Reporter: arni2033, Unassigned)

Tracking

({regression})

Trunk
regression

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

2 years ago
>>>   My Info:   Win7_64, Nightly 49, 32bit, ID 20160526082509
STR_1:
1. Open attached "testcase 1" in Firefox
2. Open debugger, select script "onclick=()=>{}", pretty print that script. Click on "analytics.js"
3.A) Click pretty-print button at the bottom, and the first moment you notice a green progress bar
     press Down key to switch to script "onclick=()=>{}"
3.B) Click pretty-print button at the bottom, and immediately
     press Down key to switch to script "onclick=()=>{}"
4. Wait 5 seconds (just in case), then switch to "analytics.js"

AR:  A) Script "analytics.js" isn't prettyfied, but the button's state says it is.
     B) Script "analytics.js" isn't prettyfied, script "onclick=()=>{}" isn't prettyfied
ER:  Both scripts should be prettyfied

This is regression from bug 1233927. Regression range:
> https://hg.mozilla.org/integration/fx-team/pushloghtml?fromchange=f40c52c4b8005e98065133bf4baaa33b5ca62084&tochange=2572bf0929df26a694f58cc8765da73abea0f655
(Reporter)

Updated

2 years ago
No longer blocks: 1277113
(Reporter)

Updated

2 years ago
Component: Untriaged → Developer Tools: Canvas Debugger
(Reporter)

Updated

2 years ago
Component: Developer Tools: Canvas Debugger → Developer Tools: Debugger
Comment hidden (spam)
arni2033 - i believe this is fixed in the new frontend in nightly. Could you confirm?
Closing as I believe the new frontend has addressed this issue. If that's not the case, we can re-open it.
Status: NEW → RESOLVED
Last Resolved: a year ago
Resolution: --- → FIXED

Updated

5 days ago
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.