NEW268137
Change document.documentURI and document.URL to match web specification 
https://bugs.webkit.org/show_bug.cgi?id=268137
Summary Change document.documentURI and document.URL to match web specification
Ahmad Saleem
Reported 2024-01-25 19:17:34 PST
hi Team, While going through Blink's commit, I came across potential clean-up and at the same time failing test case: Test Case: https://jsfiddle.net/jyxnf2rd/show ^ Chrome Canary 123 passes all while Firefox Nightly 124 fails few. Blink Commit: https://src.chromium.org/viewvc/blink?view=revision&revision=193066 WebKit Source: https://searchfox.org/wubkat/rev/66bf68c9f4593eecb8dc370eb0761cb7a0e1ae77/Source/WebCore/dom/Document.cpp#637 Minibrowser fails following: FAIL testWindow.location.host = 'c' should be c. Threw exception SyntaxError: Invalid URL FAIL testWindow.location.hostname = 'd' should be d. Threw exception SyntaxError: Invalid URL FAIL testWindow.location.port = 'e' should be e. Threw exception SyntaxError: Invalid URL FAIL testWindow.location.pathname = 'f' should be f. Threw exception SyntaxError: Invalid URL FAIL testWindow.location.search = 'g' should be g. Threw exception SyntaxError: Invalid URL __ Just wanted to raise to align with other browser as needed or fix as needed.
Attachments
Add attachment proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1 2024-02-01 19:18:13 PST
<rdar://problem/122154366>
Anne van Kesteren
Comment 2 2024-02-02 04:07:53 PST
This test case is quite confusing and not really related to the title of this bug. It seems to be mainly testing what window.location returns on a newly created popup. I suspect we already have a bug on the lifetime of the Location object.
Note You need to log in before you can comment on or make changes to this bug.