Revert "Make a css-contain test less flaky." #52477
Conversation
This reverts commit 4b41123.
|
@Psychpsyo @tabatkins PTAL |
|
Ah, yes. Didn't notice that that one uses the same reference file, sorry! Swapping <!DOCTYPE HTML>
<title>'contain: layout' establishes stacking context.</title>
<link rel="author" title="Psychpsyo" href="mailto:[email protected]">
<link rel="help" href="https://drafts.csswg.org/css2/visuren.html#x43">
<link rel="help" href="https://drafts.csswg.org/css-contain/#containment-layout">
<link rel="match" href="contain-paint-stacking-context-001-ref.html">
<style>
div {
width: 100px;
height: 100px;
}
#front {
background-color: green;
/* makes a stacking context and puts this on top */
position: absolute;
z-index: 10;
}
#back {
contain: layout;
}
#notOnTop {
background-color: red;
/* z-index is higher than on #front, but this should still be covered up because it is inside #back, which has 'contain: layout' */
position: absolute;
z-index: 1000;
}
</style>
<div id="front"></div>
<div id="back">
<div id="notOnTop"></div>
</div>
Test succeeds if there is no red.(This is the new If this sounds like a sensible resolution, would you want to amend this PR, should I open one that does this, or should we revert and then re-apply with this test also changed? Duplicating the reference file or giving it a better name might also be good since right now it's called paint but is also the reference for a layout containment test. |
|
@Psychpsyo - up to you - if you'd like to fix separately that's ok. |
I've opened #52625 now, which should take care of this. |
Reverts #52321
This breaks - contain-layout-stacking-context-001.html