[webkit-changes] [WebKit/WebKit] 7a5f73: [WebKitTestRunner] Succeeds to load "file://C:/......

Fujii Hironori noreply at github.com
Thu Aug 8 13:37:37 PDT 2024


  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 7a5f73f050ffd81659d945e9b2806473059118d5
      https://github.com/WebKit/WebKit/commit/7a5f73f050ffd81659d945e9b2806473059118d5
  Author: Fujii Hironori <Hironori.Fujii at sony.com>
  Date:   2024-08-08 (Thu, 08 Aug 2024)

  Changed paths:
    M Tools/WebKitTestRunner/TestController.cpp

  Log Message:
  -----------
  [WebKitTestRunner] Succeeds to load "file://C:/...", but fails to load "file:///C:/..." on Windows
https://bugs.webkit.org/show_bug.cgi?id=277788

Reviewed by Don Olmstead.

Windows WebKitTestRunner didn't accept some styles of URL and file
path as an command line argument.

Does Not Accept:
  "file:///C:/temp/a.html"
  "C:/temp/a.html"

Accepts:
  "file://C:/temp/a.html"
  "C:\temp\a.html"

This was annoying. I have to remove slash when I copy a URL.

There is a function `testPath` to properly retrieve a file path from a
URL. Use the function for arguments. Use PathIsRelativeA API too.

* Tools/WebKitTestRunner/TestController.cpp:
(WTR::testPath):
(WTR::TestController::createTestURL):

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



To unsubscribe from these emails, change your notification settings at https://github.com/WebKit/WebKit/settings/notifications


More information about the webkit-changes mailing list