Bug 20042

Summary:

UI frozen when the debugger is paused

Product:

WebKit

Reporter:

Anthony Ricaud <rik>

Component:

Web Inspector (Deprecated)

Assignee:

Timothy Hatcher <timothy>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

aroben, timothy

Priority:

Version:

528+ (Nightly build)

Hardware:

All

OS:

All

Attachments:

Description	Flags
Testcase	none
Real testcase	none
Proposed patch	darin: review+

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.