Summary: | Rename activeWindow to incumbentWindow in the Location class | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Chris Dumez <cdumez> | ||||||
Component: | DOM | Assignee: | Chris Dumez <cdumez> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | darin, ggaren, sam, webkit-bug-importer | ||||||
Priority: | P2 | Keywords: | InRadar | ||||||
Version: | WebKit Nightly Build | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Unspecified | ||||||||
See Also: | https://bugs.webkit.org/show_bug.cgi?id=228287 | ||||||||
Attachments: |
|
Description
Chris Dumez
2021-07-26 13:10:33 PDT
Created attachment 434229 [details]
Patch
Comment on attachment 434229 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=434229&action=review > Source/WebCore/page/Location.cpp:284 > URL completedURL = firstFrame->document()->completeURL(urlString); > + > + > // FIXME: The specification says to throw a SyntaxError if the URL is not valid. Not sure why there are two blank lines here. Created attachment 434230 [details]
Patch
Committed r280313 (239963@main): <https://commits.webkit.org/239963@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 434230 [details]. |