Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 934f8ed073f26981d56de40e1848240732b7500c
      
https://github.com/WebKit/WebKit/commit/934f8ed073f26981d56de40e1848240732b7500c
  Author: Antoine Quint <grao...@webkit.org>
  Date:   2023-03-22 (Wed, 22 Mar 2023)

  Changed paths:
    M 
LayoutTests/media/modern-media-controls/macos-inline-media-controls/macos-inline-media-controls-top-right-controls-bar-hidden-when-mute-button-disabled-expected.txt
    M 
LayoutTests/media/modern-media-controls/macos-inline-media-controls/macos-inline-media-controls-top-right-controls-bar-hidden-when-mute-button-disabled.html
    M LayoutTests/platform/mac/TestExpectations

  Log Message:
  -----------
  REGRESSION(261912@main): [ macOS ] 
media/modern-media-controls/macos-inline-media-controls/macos-inline-media-controls-top-right-controls-bar-hidden-when-mute-button-disabled.html
 is a constant text failure
https://bugs.webkit.org/show_bug.cgi?id=254245
rdar://107028528

Reviewed by Dean Jackson.

This test regressed because it assumed the mute button would be located in the 
bottoms bar
with controls sized with a width of 250px. However, in 261912@main, we enforced 
correctly
removing the container to the right of the time control if there isn't enough 
room to show
the ellipsis button at all.

We fix this test by making the controls wider and also make it more robust by 
not making
assumption on timing of the first test condition being true and using 
shouldBecomeEqual()
instead, wrapper in a promise for code clarity.

* 
LayoutTests/media/modern-media-controls/macos-inline-media-controls/macos-inline-media-controls-top-right-controls-bar-hidden-when-mute-button-disabled-expected.txt:
* 
LayoutTests/media/modern-media-controls/macos-inline-media-controls/macos-inline-media-controls-top-right-controls-bar-hidden-when-mute-button-disabled.html:
* LayoutTests/platform/mac/TestExpectations:

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


_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to