Open
Bug 982370
Opened 11 years ago
Updated 2 years ago
Sometimes breakpoints completely hang firefox
Categories
(Core :: Widget: Cocoa, defect)
Tracking
()
NEW
People
(Reporter: irakli, Unassigned)
References
Details
Attachments
(2 files)
So I have run into cases where breakpoints completely freeze firefox UI and only
way to recover is to force quit firefox.
To reproduce bug unzip attached file, open index.html and set a breakpoint at base.js:36. Try refreshing firefox.
Reporter | ||
Comment 1•11 years ago
|
||
This are cases where I just give up and open chrome to debug :(
Comment 2•11 years ago
|
||
It looks like Firefox becomes very slow when we hit that breakpoint, but doesn't actually hang. I was able to focus on a different line in the debugger editor even though it would often take 10'' or more for the click to register. Anecdotally, it seems to me that debug builds were more responsive then optimized builds.
I wonder if there is some weird interaction between MutationObserver, the JIT and Debugger. Also the breakpoint is triggered in a forEach loop which involves self-hosted JS IIANM.
Here is a debug session from lldb that shows what is going on a few seconds after the breakpoint was hit in an unoptimized debug build. In this particular case it was in the middle of GC, but other times I ended up in layout code or other places. I don't know what to make of this.
Comment 3•11 years ago
|
||
CCing some folks who may be able to dig further.
Comment 4•11 years ago
|
||
So the issue is spinning a nested event loop from inside AfterProcessNextEvent(), right? Pretty sure Olli and Steven have been looking into issues like that recently: the Mac event loop gets confused if you do that and event processing stops working right.
Comment 5•11 years ago
|
||
> So the issue is spinning a nested event loop from inside AfterProcessNextEvent(), right?
One way to test this hypothesis is to see if my "radical simplification" patch for bug 959281 (attachment 8366280 [details] [diff] [review]) clears up the problem.
Comment 6•11 years ago
|
||
Yes, that patch makes this bug go away as far as I can tell.
Depends on: 959281
Updated•11 years ago
|
Component: Developer Tools: Debugger → Widget: Cocoa
Product: Firefox → Core
Comment 7•11 years ago
|
||
I wonder whether a web page could do this via alert() in a mutation observer...
Comment 8•11 years ago
|
||
As far as I know, it can.
Updated•2 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•