Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 74c184818206247ddf6be2dc4fb14dc6be50d1e0
      
https://github.com/WebKit/WebKit/commit/74c184818206247ddf6be2dc4fb14dc6be50d1e0
  Author: Anne van Kesteren <[email protected]>
  Date:   2023-09-02 (Sat, 02 Sep 2023)

  Changed paths:
    M 
LayoutTests/imported/w3c/web-platform-tests/html/infrastructure/urls/resolving-urls/query-encoding/windows-1251_include=base-href-expected.txt
    M 
LayoutTests/imported/w3c/web-platform-tests/html/infrastructure/urls/resolving-urls/query-encoding/windows-1252_include=base-href-expected.txt
    M Source/WebCore/dom/Document.cpp
    M Source/WebCore/html/HTMLBaseElement.cpp
    M Source/WebCore/xml/parser/XMLDocumentParserLibxml2.cpp

  Log Message:
  -----------
  Correct URL encoding of <base> elements
https://bugs.webkit.org/show_bug.cgi?id=261060
rdar://114861187

Reviewed by Chris Dumez.

HTMLBaseElement::href and Document::processBaseElement can both invoke
Document::completeURL safely when they use an explicit base URL. That
in turn ensures Document::completeURL can take care of the encoding
requirements.

Also, there is no need to trim ASCII whitespace as the URL parser
already takes care of that.

This aligns us with the HTML standard and Gecko.

* 
LayoutTests/imported/w3c/web-platform-tests/html/infrastructure/urls/resolving-urls/query-encoding/windows-1251_include=base-href-expected.txt:
* 
LayoutTests/imported/w3c/web-platform-tests/html/infrastructure/urls/resolving-urls/query-encoding/windows-1252_include=base-href-expected.txt:
* Source/WebCore/dom/Document.cpp:
(WebCore::Document::processBaseElement):
(WebCore::Document::completeURL const):
* Source/WebCore/html/HTMLBaseElement.cpp:
(WebCore::HTMLBaseElement::href const):
* Source/WebCore/xml/parser/XMLDocumentParserLibxml2.cpp:
(WebCore::openFunc):

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


_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to