WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
20042
UI frozen when the debugger is paused
https://bugs.webkit.org/show_bug.cgi?id=20042
Summary
UI frozen when the debugger is paused
Anthony Ricaud
Reported
2008-07-15 10:22:32 PDT
In certain cases, the inspector UI can be frozen while the debugger is paused. I've found one way to reproduce this, there might be others. The test uses /loaded|complete/.test(document.readyState) to simulate DOMContentLoaded. It's a test widely used for earlier versions of WebKit. Steps to reproduce: 1) Open the testcase 2) Open the inspector 3) Try to show/hide the console. Actual behaviour: The console panel doesn't open Expected behaviour: The console panel should open
Attachments
Testcase
(845 bytes, text/html)
2008-07-15 10:27 PDT
,
Anthony Ricaud
no flags
Details
Real testcase
(828 bytes, text/html)
2008-07-15 10:29 PDT
,
Anthony Ricaud
no flags
Details
Proposed patch
(4.08 KB, patch)
2008-08-22 14:05 PDT
,
Timothy Hatcher
darin
: review+
Details
Formatted Diff
Diff
Show Obsolete
(1)
View All
Add attachment
proposed patch, testcase, etc.
Anthony Ricaud
Comment 1
2008-07-15 10:27:09 PDT
Created
attachment 22280
[details]
Testcase
Anthony Ricaud
Comment 2
2008-07-15 10:29:20 PDT
Created
attachment 22281
[details]
Real testcase
Anthony Ricaud
Comment 3
2008-07-15 10:34:00 PDT
(In reply to
comment #0
)
> In certain cases, the inspector UI can be frozen while the debugger is paused. > I've found one way to reproduce this, there might be others. > > The test uses /loaded|complete/.test(document.readyState) to simulate > DOMContentLoaded. It's a test widely used for earlier versions of WebKit. > > Steps to reproduce: > 1) Open the testcase > 2) Open the inspector > 3) Try to show/hide the console. > > Actual behaviour: > The console panel doesn't open > > Expected behaviour: > The console panel should open >
Sorry for the wrong steps. New ones : 1) Open the testcase 2) Open the inspector 3) Enable the script debugger (the page reloads) 4) Try to show/hide the console.
Timothy Hatcher
Comment 4
2008-08-22 14:05:54 PDT
Created
attachment 22945
[details]
Proposed patch
Darin Adler
Comment 5
2008-08-24 15:08:24 PDT
Comment on
attachment 22945
[details]
Proposed patch r=me This seems like the right fix unless we can change the design so fireTimersInNestedEventLoop is no longer needed. I'm still hoping we can do that some day.
Timothy Hatcher
Comment 6
2008-08-24 15:29:20 PDT
Landed in
r35908
.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug