Closed Bug 1212099 Opened 9 years ago Closed 5 years ago

Debugger fails to stop for breakpoints in migration.js

Categories

(DevTools :: Debugger, defect, P2)

defect

Tracking

(firefox44 affected)

RESOLVED FIXED
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: 5 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.