[webkit-changes] [WebKit/WebKit] a8801e: NEW TEST(265708 at main): [ wk2 ] fast/mediastream/vi...

youennf noreply at github.com
Thu Oct 5 02:03:12 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: a8801edb72066c945f69d34863752f34cb56f2a9
      https://github.com/WebKit/WebKit/commit/a8801edb72066c945f69d34863752f34cb56f2a9
  Author: Youenn Fablet <youennf at gmail.com>
  Date:   2023-10-05 (Thu, 05 Oct 2023)

  Changed paths:
    R LayoutTests/fast/mediastream/video-srcObject-set-twice-expected.html
    A LayoutTests/fast/mediastream/video-srcObject-set-twice-expected.txt
    M LayoutTests/fast/mediastream/video-srcObject-set-twice.html
    M LayoutTests/platform/wk2/TestExpectations

  Log Message:
  -----------
  NEW TEST(265708 at main): [ wk2 ] fast/mediastream/video-srcObject-set-twice.html is a flaky ImageOnlyFailure.
https://bugs.webkit.org/show_bug.cgi?id=261920
rdar://115868374

Reviewed by Jean-Yves Avenard.

The flakiness comes from the fact that the canvas capture video frame is not always properly rendered.
While we should fix this, we can make the test no longer flaky by relying on the existing take snapshot test.
This keeps the current coverage of the test which is to check that setting twice srcObject will still allow rendering video.

* LayoutTests/fast/mediastream/video-srcObject-set-twice-expected.html: Removed.
* LayoutTests/fast/mediastream/video-srcObject-set-twice-expected.txt: Added.
* LayoutTests/fast/mediastream/video-srcObject-set-twice.html:
* LayoutTests/platform/wk2/TestExpectations:

Canonical link: https://commits.webkit.org/268908@main




More information about the webkit-changes mailing list