Skip to content

/ servo

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Intermittent timeout in /workers/semantics/xhr/001.html (Panic: "expected a Window scope") #8931

Closed
jdm opened this issue Dec 11, 2015 · 2 comments
Closed

Intermittent timeout in /workers/semantics/xhr/001.html (Panic: "expected a Window scope") #8931

jdm opened this issue Dec 11, 2015 · 2 comments
Labels
A-content/dom I-intermittent

Comments

@jdm
Copy link
Member

@jdm jdm commented Dec 11, 2015 

▶ TIMEOUT [expected CRASH] /workers/semantics/xhr/001.html
  │ 
  │ thread 'WebWorker for http://web-platform.test:8000/workers/semantics/xhr/001.html#' panicked at 'expected a Window scope', /home/servo/buildbot/slave/linux-rel/build/components/script/dom/bindings/global.rs:73
  │ stack backtrace:
  │    1:     0x5631fa8d1880 - sys::backtrace::tracing::imp::write::h92808652e552bec0MZt
  │    2:     0x5631fa8d46e5 - panicking::log_panic::_<closure>::closure.41184
  │    3:     0x5631fa8d4160 - panicking::log_panic::hc4038fe8a2a609d626x
  │    4:     0x5631fa8bdbe3 - sys_common::unwind::begin_unwind_inner::hd5a3cc6e3320d6b8lSs
  │    5:     0x5631f964eaef - sys_common::unwind::begin_unwind::begin_unwind::h10528527346985475295
  │    6:     0x5631f9a939f9 - dom::xmlhttprequest::_<impl>::new_doc::h3c7a40b41e9a9169mEY
  │    7:     0x5631f9a92a8c - dom::xmlhttprequest::_<impl>::handle_xml::h36b9539d653de1d2nDY
  │    8:     0x5631f98af957 - dom::xmlhttprequest::_<impl>::GetResponseXML::h99eaed5c42cbf28cfiY
  │    9:     0x5631f98af2db - dom::bindings::codegen::Bindings::XMLHttpRequestBinding::get_responseXML::h3d7d04f717088d5cGTd
  │   10:     0x5631f9fe82e6 - CallJitGetterOp
  │   11:     0x5631f968bfe7 - dom::bindings::utils::generic_call::hbcacf4900a536a849Zf
  │   12:     0x5631fa10403c - _ZN2js6InvokeEP9JSContextN2JS8CallArgsENS_14MaybeConstructE
  │                         at /home/servo/.cargo/git/checkouts/mozjs-06d7f04b6dbb8a8e/master/mozjs/js/src/jscntxtinlines.h:235
  │                         at /home/servo/.cargo/git/checkouts/mozjs-06d7f04b6dbb8a8e/master/mozjs/js/src/vm/Interpreter.cpp:502
  │   13:     0x5631fa10552e - _ZN2js6InvokeEP9JSContextRKN2JS5ValueES5_jPS4_NS2_13MutableHandleIS3_EE
  │                         at /home/servo/.cargo/git/checkouts/mozjs-06d7f04b6dbb8a8e/master/mozjs/js/src/vm/Interpreter.cpp:558
  │   14:     0x5631fa108c49 - _ZN2js20InvokeGetterOrSetterEP9JSContextP8JSObjectN2JS5ValueEjPS5_NS4_13MutableHandleIS5_EE
  │                         at /home/servo/.cargo/git/checkouts/mozjs-06d7f04b6dbb8a8e/master/mozjs/js/src/vm/Interpreter.cpp:628
  │   15:     0x5631fa12a0f2 - _ZL10CallGetterP9JSContextN2JS6HandleIP8JSObjectEENS2_IPN2js5ShapeEEENS1_13MutableHandleINS1_5ValueEEE
  │                         at /home/servo/.cargo/git/checkouts/mozjs-06d7f04b6dbb8a8e/master/mozjs/js/src/vm/NativeObject.cpp:1614
  │   16:     0x5631fa149e00 - _ZN2js17NativeGetPropertyEP9JSContextN2JS6HandleIPNS_12NativeObjectEEENS3_IP8JSObjectEENS3_I4jsidEENS2_13MutableHandleINS2_5ValueEEE
  │                         at /home/servo/.cargo/git/checkouts/mozjs-06d7f04b6dbb8a8e/master/mozjs/js/src/vm/NativeObject.cpp:1664
  │                         at /home/servo/.cargo/git/checkouts/mozjs-06d7f04b6dbb8a8e/master/mozjs/js/src/vm/NativeObject.cpp:1878
  │                         at /home/servo/.cargo/git/checkouts/mozjs-06d7f04b6dbb8a8e/master/mozjs/js/src/vm/NativeObject.cpp:1912
  │   17:     0x5631fa110908 - _ZN2js11GetPropertyEP9JSContextN2JS6HandleIP8JSObjectEES6_NS3_I4jsidEENS2_13MutableHandleINS2_5ValueEEE
  │                         at /home/servo/.cargo/git/checkouts/mozjs-06d7f04b6dbb8a8e/master/mozjs/js/src/vm/NativeObject.h:1434
  │   18:     0x5631fa0f851b - _ZL9InterpretP9JSContextRN2js8RunStateE
  │                         at /home/servo/.cargo/git/checkouts/mozjs-06d7f04b6dbb8a8e/master/mozjs/js/src/vm/Interpreter.cpp:260
  │                         at /home/servo/.cargo/git/checkouts/mozjs-06d7f04b6dbb8a8e/master/mozjs/js/src/vm/Interpreter.cpp:2426
  │   19:     0x5631fa103d1c - _ZN2js9RunScriptEP9JSContextRNS_8RunStateE
  │                         at /home/servo/.cargo/git/checkouts/mozjs-06d7f04b6dbb8a8e/master/mozjs/js/src/vm/Interpreter.cpp:452
  │   20:     0x5631fa103f9d - _ZN2js6InvokeEP9JSContextN2JS8CallArgsENS_14MaybeConstructE
  │                         at /home/servo/.cargo/git/checkouts/mozjs-06d7f04b6dbb8a8e/master/mozjs/js/src/vm/Interpreter.cpp:521
  │   21:     0x5631fa10552e - _ZN2js6InvokeEP9JSContextRKN2JS5ValueES5_jPS4_NS2_13MutableHandleIS3_EE
  │                         at /home/servo/.cargo/git/checkouts/mozjs-06d7f04b6dbb8a8e/master/mozjs/js/src/vm/Interpreter.cpp:558
  └   22:     0x5631fa3cda39 - _Z20JS_CallFunctionValueP9JSContextN2JS6HandleIP8JSObjectEENS2_INS1_5ValueEEERKNS1_16HandleValueArrayENS1_13MutableHandleIS6_EE

This is the worker thread not causing the original script thread to end up panicking I guess.
@jdm jdm added the I-intermittent label Dec 11, 2015
@jdm jdm mentioned this issue Dec 11, 2015
Merged
@Ms2ger
Copy link
Contributor

@Ms2ger Ms2ger commented Dec 11, 2015

We should probably use spawn_named_with_send_on_failure to tell the parent thread about it.
This was referenced Dec 11, 2015
Merged
Closed
@jdm jdm added the A-content/dom label Jan 31, 2016
@jdm jdm mentioned this issue Feb 4, 2016
Closed
@Ms2ger
Copy link
Contributor

@Ms2ger Ms2ger commented Feb 5, 2016

Note that the test uses != null, which is less strict than the check for undefined it should have.
@larsbergstrom larsbergstrom mentioned this issue Apr 6, 2016
Merged
@KiChjang KiChjang mentioned this issue Apr 7, 2016
Merged
@frewsxcv frewsxcv mentioned this issue Apr 7, 2016
Merged
@KiChjang KiChjang mentioned this issue Apr 9, 2016
Merged
@jdm jdm mentioned this issue Apr 9, 2016
Merged
@KiChjang KiChjang mentioned this issue Apr 11, 2016
Merged
@wafflespeanut wafflespeanut mentioned this issue Apr 11, 2016
Merged
@KiChjang KiChjang mentioned this issue Apr 12, 2016
Merged
@emilio emilio mentioned this issue Apr 12, 2016
Merged
@frewsxcv frewsxcv mentioned this issue Apr 12, 2016
Merged
@KiChjang KiChjang mentioned this issue Apr 12, 2016
Merged
This was referenced Apr 13, 2016
Merged
Merged
Merged
bors-servo added a commit that referenced this issue Apr 13, 2016
@bors-servo
Auto merge of #10573 - jdm:xhrintermittent, r=Manishearth
Loading status checks…
4f63319 
Do not support XHR.responseXML in worker globals. Fixes #8931.

<!-- Reviewable:start -->
This change is [<img src="https://reviewable.io/review_button.svg" height="35" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/10573)
<!-- Reviewable:end -->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A-content/dom I-intermittent
Projects
None yet
Milestone
No milestone
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
@jdm @Ms2ger
You can’t perform that action at this time.