Bug 100837 – Web Inspector: page crash when pausing in dedicated worker

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

[Yury Semikhatsky]

Created attachment 171607 [details]
Example page

Steps:

1. Navigate to a page with dedicated worker and open inspector for the worker.
2. Click pause in the worker inspector.

Result:
Inspected page crash

[Yury Semikhatsky]

Created attachment 171608 [details]
Example worker script

[Yury Semikhatsky]

Created attachment 171609 [details]
Patch

[Yury Semikhatsky]

Stack trace
#0  base::debug::BreakDebugger () at ../../base/debug/debugger_posix.cc:243
#1  0x00007f95bc1d22ca in base::(anonymous namespace)::StackDumpSignalHandler (signal=11, info=0x7f95aa8924b0, context=0x7f95aa892380)
    at ../../base/process_util_posix.cc:142
#2  <signal handler called>
#3  0x00007f95bb7dac04 in WebCore::InjectedScriptHost::scriptDebugServer (this=0x7f95ab3f5900)
    at ../../third_party/WebKit/Source/WebCore/inspector/InjectedScriptHost.cpp:169
#4  0x00007f95bbb3bb29 in WebCore::V8InjectedScriptHost::getInternalPropertiesCallback (args=...)
    at ../../third_party/WebKit/Source/WebCore/bindings/v8/custom/V8InjectedScriptHostCustom.cpp:206
#5  0x00007f95bd31b645 in v8::internal::HandleApiCallHelper<false> (args=..., isolate=0x7f95aaaf6000) at ../../v8/src/builtins.cc:1146
#6  0x00007f95bd31b223 in v8::internal::Builtin_Impl_HandleApiCall (args=..., isolate=0x7f95aaaf6000) at ../../v8/src/builtins.cc:1164
#7  0x00007f95bd314dec in v8::internal::Builtin_HandleApiCall (args=..., isolate=0x7f95aaaf6000) at ../../v8/src/builtins.cc:1163

[Yury Semikhatsky]

Created attachment 171632 [details]
Patch

[Alexander Pavlov (apavlov)]

Comment on attachment 171632 [details]
Patch

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

> LayoutTests/ChangeLog:9
> +        on all platforms that use JSC as workder debugging is not supported there

workder -> worker

[Yury Semikhatsky]

(In reply to comment #5)
> (From update of attachment 171632 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=171632&action=review
> 
> > LayoutTests/ChangeLog:9
> > +        on all platforms that use JSC as workder debugging is not supported there
> 
> workder -> worker

Done.

[Yury Semikhatsky]

Created attachment 171635 [details]
Patch

[Yury Semikhatsky]

Chromium bugs:
http://code.google.com/p/chromium/issues/detail?id=153913
http://code.google.com/p/chromium/issues/detail?id=158191

[WebKit Review Bot]

Comment on attachment 171635 [details]
Patch

Rejecting attachment 171635 [details] from commit-queue.

Failed to run "['/mnt/git/webkit-commit-queue/Tools/Scripts/webkit-patch', '--status-host=queues.webkit.org', '-..." exit_code: 2

Last 500 characters of output:
t/git/webkit-commit-queue/Source/WebKit/chromium/v8 --revision 12787 --non-interactive --force --accept theirs-conflict --ignore-externals' in '/mnt/git/webkit-commit-queue/Source/WebKit/chromium'
50>At revision 12787.

________ running '/usr/bin/python tools/clang/scripts/update.py --mac-only' in '/mnt/git/webkit-commit-queue/Source/WebKit/chromium'

________ running '/usr/bin/python gyp_webkit' in '/mnt/git/webkit-commit-queue/Source/WebKit/chromium'
Updating webkit projects from gyp files...

Full output: http://queues.webkit.org/results/14656167

[Yury Semikhatsky]

Committed r133030: <http://trac.webkit.org/changeset/133030>