[Webkit-unassigned] [Bug 234987] New: css/css-transitions/pseudo-elements-002.html WPT is a failure

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=234987

            Bug ID: 234987
           Summary: css/css-transitions/pseudo-elements-002.html WPT is a
                    failure
           Product: WebKit
           Version: WebKit Nightly Build
          Hardware: Unspecified
                OS: Unspecified
            Status: NEW
          Severity: Normal
          Priority: P2
         Component: Animations
          Assignee: webkit-unassigned at lists.webkit.org
          Reporter: graouts at webkit.org
                CC: dino at apple.com, graouts at apple.com

The WPT at css/css-transitions/pseudo-elements-002.html is a failure. It's a simple test, which looks like this:

    <style>
      #inner::before {
        content: "This text should transition from red to green.";
        height: 100px;
        transition: height steps(2, start) 1s;
      }
      .flex #inner::before {
        height: 300px;
      }
      .flex { display: flex }
    </style>

    <div id="outer">
      <div id="inner"></div>
    </div>

    <script>

      test(() => {
          assert_equals(getComputedStyle(inner, "::before").height, "100px");
          outer.className = "flex";
          assert_equals(getComputedStyle(inner, "::before").height, "200px");
      }, "Check that transitions run on a pseudo element whose ancestor changes display type.");
    </script>

>From what I can see via logging, we:

1. create a PseudoElement as we query the ::before computed style the first time
2. set the CSS "flex" class
3. return the ::before computed style for the second failed assertion
4. create the "height" transition from 100px to 300px (too late, the previous style check did not account for it)
4. destroy the PseudoElement, terminating the transition
5. create a new PseudoElement
6. eventually consider running a transition again but the before and after styles for height are both 300px so do nothing

Something is not being invalidated correctly between setting the "flex" CSS class and querying the computed style. We should have created the transition while the computed style was queried, but we didn't.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-unassigned/attachments/20220107/62dd2e82/attachment-0001.htm>