Bug 153941 - [GTK] visible-control-characters tests should not be passing
Summary: [GTK] visible-control-characters tests should not be passing
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKitGTK (show other bugs)
Version: Other
Hardware: PC Linux
: P2 Normal
Assignee: Nobody
Depends on:
Reported: 2016-02-06 08:55 PST by Michael Catanzaro
Modified: 2024-03-28 18:05 PDT (History)
2 users (show)

See Also:


Note You need to log in before you can comment on or make changes to this bug.
Description Michael Catanzaro 2016-02-06 08:55:09 PST
The following tests are sometimes passing for the last couple of months:


It's bad because these are expected mismatch tests, they should never pass.

The first pass (known bad commit) is r192730, but that commit is probably not to blame. Then there are many flaky runs. Then r192779 is the last known bad commit, and then test consistently fails (good) for a while, so it was fixed for a range of commits. The next known pass (bad) is r193641, and it's flaky from then on.

So we know:

* Tests broken shortly before r192730, probably exclusive
* Tests fixed shortly after r192779, exclusive
* Tests broken again sometimes before r193642, exclusive

Unfortunately there was some problem with the bot, and the last results we have before r193641 are for r193617, which is a rather big range of commits.

Since the tests were broken, fixed, then broken again, I at first suspected r192727, shortly before r192730. However that was rolled out quickly in r192731, but the tests continued to pass (bad) for some time after that, which doesn't match up. Still, it seems quite possible I broke it with theme changes.

(visible-control-characters-1 seems to have a few additional rare, flaky failures outside of those ranges.)
Comment 2 Vitaly Dyackhov 2023-02-17 08:54:00 PST
Pull request: https://github.com/WebKit/WebKit/pull/10284
Comment 3 EWS 2023-03-10 02:13:33 PST
Committed 261493@main (4ae8edc354b7): <https://commits.webkit.org/261493@main>

Reviewed commits have been landed. Closing PR #10284 and removing active labels.