script: Fix handling changes to relevant attributes for stylesheets#42273
Conversation
|
🔨 Triggering try run (#21564819094) for Linux (WPT) |
|
🤖 Opened new upstream WPT pull request (web-platform-tests/wpt#57469) with upstreamable changes. |
|
|
761857d to
d4ea74f
Compare
|
🔨 Triggering try run (#21566873079) for Linux (WPT) |
|
📝 Transplanted new upstreamable changes to existing upstream WPT pull request (web-platform-tests/wpt#57469). |
|
Test results for linux-wpt from try job (#21566873079): Flaky unexpected result (37)
Stable unexpected results that are known to be intermittent (27)
Stable unexpected results (2)
|
|
|
d4ea74f to
b60f230
Compare
|
📝 Transplanted new upstreamable changes to existing upstream WPT pull request (web-platform-tests/wpt#57469). |
We weren't removing stylesheets when we should be doing so. This most notably happens when the stylesheet no longer has a href or its "rel" changes. The crash is also fixed, since the issue was that we were passing the value of the attribute that was changed as if it were an href. However, if we set the type attribute, then that's not the href. To fix that, we now retrieve the href inside the method so we always have the correct value. Fixes servo#42259 Signed-off-by: Tim van der Lippe <[email protected]>
b60f230 to
f294b4a
Compare
|
📝 Transplanted new upstreamable changes to existing upstream WPT pull request (web-platform-tests/wpt#57469). |
Signed-off-by: Tim van der Lippe <[email protected]>
|
📝 Transplanted new upstreamable changes to existing upstream WPT pull request (web-platform-tests/wpt#57469). |
Signed-off-by: Tim van der Lippe <[email protected]>
|
📝 Transplanted new upstreamable changes to existing upstream WPT pull request (web-platform-tests/wpt#57469). |
Signed-off-by: Tim van der Lippe <[email protected]>
|
📝 Transplanted new upstreamable changes to existing upstream WPT pull request (web-platform-tests/wpt#57469). |
We weren't removing stylesheets when we should be doing so. This most notably happens when the stylesheet no longer has a href or its "rel" changes.
The crash is also fixed, since the issue was that we were passing the value of the attribute that was changed as if it were an href. However, if we set the type attribute, then that's not the href.
To fix that, we now retrieve the href inside the method so we always have the correct value.
Fixes #42259