Bug 271094
| Summary: | Remove non-standard `initHashChangeEvent` | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Ahmad Saleem <ahmad.saleem792> |
| Component: | DOM | Assignee: | Nobody <webkit-unassigned> |
| Status: | RESOLVED FIXED | ||
| Severity: | Normal | CC: | annevk, webkit-bug-importer |
| Priority: | P2 | Keywords: | InRadar |
| Version: | WebKit Nightly Build | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
Ahmad Saleem
Hi Team,
While looking into HTML Spec and comparing with our implementation, I noticed that the web-spec don't have `initHashChangeEvent`.
https://html.spec.whatwg.org/multipage/web-messaging.html#message-ports
and then I tried to find any issue on GitHub to indicate if it was removed but noticed that it does not exist in Blink but does exist in WebKit & Gecko.
Blink removed it in 2015 with following commit - https://src.chromium.org/viewvc/blink?view=revision&revision=202347
Just wanted to raise so either we can add it to spec if needed (since there are two implementations) or remove it as well.
Blink stated there was zero usage.
CCing @Anne - for his input.
Thanks!
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Ahmad Saleem
I hate my clipboard app messing up my copy paste. [Time to remove it] :-(
Web-Spec link - https://html.spec.whatwg.org/multipage/nav-history-apis.html#the-hashchangeevent-interface
EWS
Committed 276232@main (cf053c07242e): <https://commits.webkit.org/276232@main>
Reviewed commits have been landed. Closing PR #25995 and removing active labels.
Radar WebKit Bug Importer
<rdar://problem/124736521>