Josh Matthews

Panic in custom-elements/throw-on-dynamic-markup-insertion-counter-construct-xml-parser.xhtml

1

To reproduce: `./mach test-wpt tests/wpt/tests/custom-elements/throw-on-dynamic-markup-insertion-counter-construct-xml-parser.xhtml` Backtrace: ``` 0:03.43 pid:15762 called `Option::unwrap()` on a `None` value (thread Script(1,1), at components/script/dom/bindings/settings_stack.rs:97) 0:05.83 pid:15762 14: core::option::Option::unwrap 0:05.83 pid:15762 at /rustc/9b00956e56009bab2aa15d7bff10916599e3d6d6/library/core/src/option.rs:933:21 0:05.83 pid:15762 script::dom::bindings::settings_stack::entry_global...

To reproduce: `./mach test-wpt tests/wpt/tests/html/semantics/embedded-content/the-img-element/srcset/parse-a-srcset-attribute.html` The code in https://github.com/servo/servo/blob/main/components/script/dom/htmlimageelement.rs#L1894-L1897 is misusing the chars iterator to obtain byte indexes from the string. We should use the [char_indices](https://doc.rust-lang.org/stable/std/primitive.str.html#method.char_indices) iterator instead. Backtrace: ```...

Panic in html/dom/render-blocking/remove-pending-async-render-blocking-script.html

1

To reproduce: `./mach test-wpt tests/wpt/tests/html/dom/render-blocking/remove-pending-async-render-blocking-script.html` The test has an async script element present in the test source, then a subsequent synchronous script moves the script element to another document. It...

To reproduce: `./mach test-wpt --headless tests/wpt/tests/css/css-fonts/crash-large-grapheme-cluster.html --debugger` The backtrace is full of frames like: ``` frame #15071: 0x0000000103ded688 servo`fonts::glyph::GlyphIterator::next_complex_glyph::hdcd1aef33361a89c(self=0x0000000175908ca8, entry=0x00000001758f7164, i=(__0 = 142)) at glyph.rs:749:9 frame #15072: 0x0000000103ded548 servo`fonts::glyph::GlyphIterator::next_glyph_range::h10cedb4e4af03b65 at...

IDL callback returning promises should return rejected promises if they would throw an exception

4

https://heycam.github.io/webidl/#dfn-attribute-getter (`if an exception E was thrown`) https://heycam.github.io/webidl/#dfn-create-operation-function (`if an exception E was thrown`) https://heycam.github.io/webidl/#invoke-a-callback-function (step 14.5) An example of a test that fails without this is the idlharness tests...

Borrow hazard when calling Tokenizer::end from ServoParser::do_parse_sync

2

With zealous GC, we expose a borrow failure when ending tokenizing: ``` 4:16.33 pid:93228 DomRefCell already mutably borrowed: BorrowError (thread ScriptThread PipelineId { namespace_id: PipelineNamespaceId(1), index: PipelineIndex(1) }, at src/libcore/result.rs:1084)...

Tracking issues that affect twitter.com in particular: * [x] https://github.com/servo/servo/issues/9869 * [ ] https://github.com/servo/servo/issues/9870 * [ ] https://github.com/servo/servo/issues/14577 * [x] https://github.com/servo/servo/issues/18298 * [ ] https://github.com/servo/servo/issues/18299 * [ ] https://github.com/servo/servo/issues/18811 *...

Release lumin runtime crashes at startup

5

When I do a release build of servo and package it with `-t lumin_release` the app does not start correctly. When I use `-t lumin_debug` instead, it works.

Transparent rects demo detroys magic leap responsiveness

1

The perf is really bad in a release build, and it's quite janky when moving the laser pointer around, and almost impossible to position the 2d cursor correctly because it...