Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 4aee10c2202aa5e2012f612ea1b626f4e02b3ca4
      
https://github.com/WebKit/WebKit/commit/4aee10c2202aa5e2012f612ea1b626f4e02b3ca4
  Author: Anne van Kesteren <ann...@annevk.nl>
  Date:   2025-03-14 (Fri, 14 Mar 2025)

  Changed paths:
    M 
LayoutTests/fast/events/popup-blocked-from-unique-frame-via-window-open-named-sibling-frame-expected.txt
    M LayoutTests/http/tests/security/no-popup-from-sandbox-top-expected.txt
    M 
LayoutTests/imported/w3c/web-platform-tests/url/a-element-origin-expected.txt
    M 
LayoutTests/imported/w3c/web-platform-tests/url/a-element-origin-xhtml-expected.txt
    M 
LayoutTests/imported/w3c/web-platform-tests/url/a-element-xhtml_exclude=(file_javascript_mailto)-expected.txt
    M 
LayoutTests/imported/w3c/web-platform-tests/url/a-element_exclude=(file_javascript_mailto)-expected.txt
    M 
LayoutTests/imported/w3c/web-platform-tests/url/resources/setters_tests.json
    M LayoutTests/imported/w3c/web-platform-tests/url/resources/urltestdata.json
    M 
LayoutTests/imported/w3c/web-platform-tests/url/url-constructor.any.worker_exclude=(file_javascript_mailto)-expected.txt
    M 
LayoutTests/imported/w3c/web-platform-tests/url/url-constructor.any_exclude=(file_javascript_mailto)-expected.txt
    M 
LayoutTests/imported/w3c/web-platform-tests/url/url-origin.any-expected.txt
    M 
LayoutTests/imported/w3c/web-platform-tests/url/url-origin.any.worker-expected.txt
    M 
LayoutTests/imported/w3c/web-platform-tests/url/url-setters-a-area.window-expected.txt
    M 
LayoutTests/imported/w3c/web-platform-tests/url/url-setters.any.worker_exclude=(file_javascript_mailto)-expected.txt
    M 
LayoutTests/imported/w3c/web-platform-tests/url/url-setters.any_exclude=(file_javascript_mailto)-expected.txt
    M 
LayoutTests/imported/w3c/web-platform-tests/url/urlsearchparams-delete.any-expected.txt
    M 
LayoutTests/imported/w3c/web-platform-tests/url/urlsearchparams-delete.any.js
    M 
LayoutTests/imported/w3c/web-platform-tests/url/urlsearchparams-delete.any.worker-expected.txt
    M Source/WTF/wtf/URL.cpp
    M Source/WTF/wtf/URL.h
    M Source/WTF/wtf/URLParser.cpp

  Log Message:
  -----------
  Ensure opaque URL paths always roundtrip
https://bugs.webkit.org/show_bug.cgi?id=289160
rdar://146848690

Reviewed by Alex Christensen.

Instead of attempting to account for opaque paths sometimes ending in
spaces in the API implementation and failing because that did not
account for serialization, this new approach always percent-encodes the
final space of the path. This is thought to be the least invasive
change that is hopefully web-compatible. This is standardized here:
https://github.com/whatwg/url/pull/844

Test changes are upstreamed here:
https://github.com/web-platform-tests/wpt/pull/51129

* 
LayoutTests/fast/events/popup-blocked-from-unique-frame-via-window-open-named-sibling-frame-expected.txt:
* LayoutTests/http/tests/security/no-popup-from-sandbox-top-expected.txt:
* LayoutTests/imported/w3c/web-platform-tests/url/a-element-origin-expected.txt:
* 
LayoutTests/imported/w3c/web-platform-tests/url/a-element-origin-xhtml-expected.txt:
* 
LayoutTests/imported/w3c/web-platform-tests/url/a-element-xhtml_exclude=(file_javascript_mailto)-expected.txt:
* 
LayoutTests/imported/w3c/web-platform-tests/url/a-element_exclude=(file_javascript_mailto)-expected.txt:
* LayoutTests/imported/w3c/web-platform-tests/url/resources/setters_tests.json:
* LayoutTests/imported/w3c/web-platform-tests/url/resources/urltestdata.json:
* 
LayoutTests/imported/w3c/web-platform-tests/url/url-constructor.any.worker_exclude=(file_javascript_mailto)-expected.txt:
* 
LayoutTests/imported/w3c/web-platform-tests/url/url-constructor.any_exclude=(file_javascript_mailto)-expected.txt:
* LayoutTests/imported/w3c/web-platform-tests/url/url-origin.any-expected.txt:
* 
LayoutTests/imported/w3c/web-platform-tests/url/url-origin.any.worker-expected.txt:
* 
LayoutTests/imported/w3c/web-platform-tests/url/url-setters-a-area.window-expected.txt:
* 
LayoutTests/imported/w3c/web-platform-tests/url/url-setters.any.worker_exclude=(file_javascript_mailto)-expected.txt:
* 
LayoutTests/imported/w3c/web-platform-tests/url/url-setters.any_exclude=(file_javascript_mailto)-expected.txt:
* 
LayoutTests/imported/w3c/web-platform-tests/url/urlsearchparams-delete.any-expected.txt:
* LayoutTests/imported/w3c/web-platform-tests/url/urlsearchparams-delete.any.js:
(test):
* 
LayoutTests/imported/w3c/web-platform-tests/url/urlsearchparams-delete.any.worker-expected.txt:
* Source/WTF/wtf/URL.cpp:
(WTF::URL::removeFragmentIdentifier):
(WTF::URL::removeQueryAndFragmentIdentifier):
(WTF::URL::setQuery):
(WTF::URL::maybeTrimTrailingSpacesFromOpaquePath): Deleted.
* Source/WTF/wtf/URL.h:
* Source/WTF/wtf/URLParser.cpp:
(WTF::URLParser::parse):

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



To unsubscribe from these emails, change your notification settings at 
https://github.com/WebKit/WebKit/settings/notifications
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to