[webkit-changes] [WebKit/WebKit] 2ee8d3: Highlight test has a too long a defined wait.

megangardner noreply at github.com
Mon Oct 9 15:08:50 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 2ee8d3fa3d705cd2feff3719e4442454f7cdcf99
      https://github.com/WebKit/WebKit/commit/2ee8d3fa3d705cd2feff3719e4442454f7cdcf99
  Author: Megan Gardner <megan_gardner at apple.com>
  Date:   2023-10-09 (Mon, 09 Oct 2023)

  Changed paths:
    M LayoutTests/fast/repaint/highlight-paint-after-range-change-expected.txt
    M LayoutTests/fast/repaint/highlight-paint-after-range-change.html

  Log Message:
  -----------
  Highlight test has a too long a defined wait.
https://bugs.webkit.org/show_bug.cgi?id=262398
rdar://116254079

Reviewed by Aditya Keerthi and Simon Fraser.

Responding to feedback that having a 100ms timeout is too long, rearchitecting the test to
instead await UIHelper.renderingUpdate() to wait the minimum amount of time needed.

* LayoutTests/fast/repaint/highlight-paint-after-range-change-expected.txt:
* LayoutTests/fast/repaint/highlight-paint-after-range-change.html:

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




More information about the webkit-changes mailing list