[webkit-changes] [WebKit/WebKit] c646f2: [GLib] In new API, update hsts directories for Tes...

Amanda Falke noreply at github.com
Wed Jul 26 13:07:46 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: c646f2370d230da0bf274514b07f0b6f8d624309
      https://github.com/WebKit/WebKit/commit/c646f2370d230da0bf274514b07f0b6f8d624309
  Author: Amanda Falke <afalke at igalia.com>
  Date:   2023-07-26 (Wed, 26 Jul 2023)

  Changed paths:
    M Tools/TestWebKitAPI/Tests/WebKitGLib/TestWebsiteData.cpp
    M Tools/TestWebKitAPI/glib/TestExpectations.json

  Log Message:
  -----------
  [GLib] In new API, update hsts directories for TestWebsiteData.
https://bugs.webkit.org/show_bug.cgi?id=259515

Reviewed by Michael Catanzaro.

As mentioned in 260511 at main and 261018 at main, A GLib-specific bug in the
defaultHSTSStorageDirectory is fixed, but only for the new API versions.
The HSTS database goes under the base cache directory if provided, but
if not provided it goes under the base data directory; leave it broken in
the original API versions to avoid leaking the HSTS database on disk.
Also, move it to a subdirectory in the new API version.

Updating the directories (to baseCacheDirectory for old API and
under subdirectory for new API, using code that we previously had before
259132 at main) in prepopulateHstsData fixes the hsts test in
TestWebsiteData, which were marked as failing under a meta-gardening bug
in webkit.org/b/256557.

* Tools/TestWebKitAPI/Tests/WebKitGLib/TestWebsiteData.cpp:
(prepopulateHstsData): add HSTS subdir for new API for hsts tests,
this was code we previously used prior to 259132 at main.
* Tools/TestWebKitAPI/glib/TestExpectations.json: Remove WPE failure
for hsts test.

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




More information about the webkit-changes mailing list