Bug 101065 – Web Inspector: JavaScript web workers debugging crashes

• Component: Web Inspector (Deprecated)
• Status: RESOLVED FIXED

[Yury Semikhatsky]

Original Chromium bug report: http://code.google.com/p/chromium/issues/detail?id=153913

Repro steps:

1. Open attached page, enable debugger tools with {Workers.Pause on start}
2. Continue from the top debugging screen (press start, or just close the window)
3. Crash

[Yury Semikhatsky]

Created attachment 172076 [details]
Patch

[Yury Semikhatsky]

Created attachment 172488 [details]
Patch

[Yury Semikhatsky]

(In reply to comment #2)
> Created an attachment (id=172488) [details]
> Patch

Added layout test.

[Alexander Pavlov (apavlov)]

Comment on attachment 172488 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=172488&action=review

> LayoutTests/inspector-protocol/debugger-terminate-dedicated-worker-while-paused.html:31
> +                             "id": workerRequestId++ }

closing brace could be aligned with its opening counterpart, like anywhere else

[Yury Semikhatsky]

(In reply to comment #4)
> (From update of attachment 172488 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=172488&action=review
> 
> > LayoutTests/inspector-protocol/debugger-terminate-dedicated-worker-while-paused.html:31
> > +                             "id": workerRequestId++ }
> 
> closing brace could be aligned with its opening counterpart, like anywhere else

Done.

[Yury Semikhatsky]

Created attachment 172505 [details]
Patch

[Yury Semikhatsky]

Committed r133569: <http://trac.webkit.org/changeset/133569>