mirror of
https://github.com/mozilla/gecko-dev.git
synced 2024-10-22 17:55:50 +00:00
e80b8667d5
We were spuriously reframing the <shadow> because it initially shared style with the <br>, which ended up being display: none, while the <shadow> should've been display: contents from the beginning. lookup_by_rules seems pretty prone to obscure bugs, and also it's pretty complex... Probably we should try to get rid of it, I'm unconvinced that it's worth it. Even with that, in a normal restyle the <details> wouldn't have ended up with a style. It of course never had it before the reframe because the <shadow> was display: none, but that doesn't mean it shouldn't have gotten one, since we detected we needed to go through kids in: https://searchfox.org/mozilla-central/rev/6eea08365e7386a2b81c044e7cc8a3daa51d8754/servo/components/style/matching.rs#500 That code did happen, but since it's an animation-only restyle, we don't look at unstyled stuff. That looks somewhat fishy, but I guess for now it's fine as long as display isn't animatable. MozReview-Commit-ID: B6NMSTNOKgK
13 lines
191 B
HTML
13 lines
191 B
HTML
<style>
|
|
* { display: contents }
|
|
</style>
|
|
<script>
|
|
function go() {
|
|
a.parentElement.animate({"ping": [0, 1]}, 0.281207776578);
|
|
}
|
|
</script>
|
|
<body onload=go()>
|
|
></br>
|
|
<shadow>
|
|
<details id="a">
|