[Webkit-unassigned] [Bug 182222] REGRESSION(r227223): http/tests/resourceLoadStatistics/clear-in-memory-and-persistent-store-one-hour.html, http/tests/resourceLoadStatistics/grandfathering.html timing out on GTK, WPE

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Sun Jan 28 15:14:12 PST 2018


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

Michael Catanzaro <mcatanzaro at igalia.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |mcatanzaro at igalia.com,
                   |                            |wilander at apple.com

--- Comment #2 from Michael Catanzaro <mcatanzaro at igalia.com> ---
(In reply to Zan Dobersek from comment #1)
> Also failing on WPE. For both ports is likely due to missing port-specific
> resource-load statistics functionality.

That commit added completion handler parameters (required for the WebsiteDataManager removal API to work properly), but it only updated the Cocoa API, not the cross-platform C API used by the test runner. I'm not sure whether we should roll it out or not. It doesn't break anything except the test runner (well, it breaks the C API, but that's used only by the test runner), and the tests must have already been flaky before this change, and it's probably simpler to just add the callback parameters (haven't checked). But this is not the first time this has happened recently (see bug #181231), and we need to be stricter about breaking the test runner like this.

The root of the problem is that it's hard to update the cross-platform test runner code from Mac, because it is using Cocoa-specific API instead of the C API, to make it easier to delete the C API in the future. But this makes it much harder to make changes that affect the test runner, and it's not clear what the endgame is for removing the C API without breaking the test runner. This is worth discussion on webkit-dev.

-- 
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/20180128/a1105488/attachment.html>


More information about the webkit-unassigned mailing list