Closed Bug 1212099 Opened 9 years ago Closed 6 years ago

Debugger fails to stop for breakpoints in migration.js

Categories

(DevTools :: Debugger, defect, P2)

Product:
DevTools
Component:
Debugger
Type:
defect

Tracking

(firefox44 affected)

Status:
RESOLVED FIXED
Tracking Flags:
Tracking Status
firefox44 --- affected

People

(Reporter: Gijs, Unassigned)

References

(Blocks 1 open bug)

Details

STR: 1. open firefox 2. open browser toolbox 3. open bookmarks manager (accel-shift-b) 4. click the last icon in the toolbar, then click "Import Data from another browser" 5. close the window 6. in the browser toolbox, filter for migration.js 7. set a breakpoint in the "init" method 8. reopen the import wizard (step 4) ER: stop on breakpoint AR: doesn't stop on breakpoint. This seems like a regression of bug 883342. Do you know what's up here, Eddy?
Flags: needinfo?(ejpbruel)
I've tried to reproduce the issue, but got stuck on step 6: the file migration.js doesn't seem to exist (I typed migration.js into the search bar, but there are no matching search results). Any idea what I could be doing wrong?
Flags: needinfo?(ejpbruel) → needinfo?(gijskruitbosch+bugs)
(In reply to Eddy Bruel [:ejpbruel] from comment #1) > I've tried to reproduce the issue, but got stuck on step 6: the file > migration.js doesn't seem to exist (I typed migration.js into the search > bar, but there are no matching search results). > > Any idea what I could be doing wrong? No. You opened the importer first to ensure it loaded and showed up in the debugger? Does chrome://browser/content/migration/migration.js result in a file in the address bar?
Flags: needinfo?(gijskruitbosch+bugs) → needinfo?(ejpbruel)
I'm sorry Gijs. The number of P1 bugs for the debugger is piling up quickly atm, so I don't think I will be able to look into this any time soon. Marking this as P2 so it at least stays on our radar.
Flags: needinfo?(ejpbruel)
Priority: -- → P2
Product: Firefox → DevTools

:Gijs would you mind re-testing this? A lot has changed over the past 3 years :) Fortunately, one thing we're doing now is going through the bugzilla backlog and re-prioritizing stale items.

Flags: needinfo?(gijskruitbosch+bugs)

(In reply to Jason Laster [:jlast] from comment #4)

:Gijs would you mind re-testing this? A lot has changed over the past 3 years :) Fortunately, one thing we're doing now is going through the bugzilla backlog and re-prioritizing stale items.

It's still broken, following the steps from comment #0.

Flags: needinfo?(gijskruitbosch+bugs) → needinfo?(jlaster)

Thanks for looking Gijs.

Flags: needinfo?(jlaster)

I can no longer reproduce this issue.

Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.