Join GitHub today
GitHub is home to over 50 million developers working together to host and review code, manage projects, and build software together.
Sign upIntermittent crash in /workers/WorkerLocation-origin.sub.window.html #22652
Comments
jdm
commented
Jan 8, 2019
|
@gterzian I remember seeing some comments from you about what it means when the layout thread is disconnected, but I can't recall where those comments were. This looks like another case where that would happen, but I'm surprised by the stack given the testcase source (https://github.com/web-platform-tests/wpt/blob/master/workers/WorkerLocation-origin.sub.window.js). |
|
The issue is opened at #22507, I guess the topic will expand as we discover similar cases... It looks similar due to the crash on reflow inside a It might have something to do with the iframe being cross-origin(I think), because this occured before at #22559 |
…es-panic, r=jdm Return Option for Window's layout channel <!-- Please describe your changes on the following line: --> `Window::layout_chan()` now returns an `Option<Sender<Msg>>`, returning `None` if the window is dead. FIX #26969 FIX #26429 FIX #21208 FIX #19092 FIX #22559 FIX #22584 FIX #22652 --- <!-- Thank you for contributing to Servo! Please replace each `[ ]` by `[X]` when the step is complete, and replace `___` with appropriate data: --> - [X] `./mach build -d` does not report any errors - [X] `./mach test-tidy` does not report any errors - [X] These changes fix #23053 <!-- Either: --> - [x] There are tests for these changes <!-- Also, please make sure that "Allow edits from maintainers" checkbox is checked, so that we can help you if you get stuck somewhere along the way.--> <!-- Pull requests that do not address these steps are welcome, but they will require additional verification as part of the review process. --> This is my first contribution, I'm trying to figure things out! This fix passes the test case shown in #23053, however I don't know what the behavior should be in `Document` and `ScriptThread` if `Window::is_alive()` is false : simply ignore it, don't do anything ? Or is this something that should not happen now that we return false in `Window::force_reflow()` ? I'm not sure about the directory where the test case should go, any advice?