Bug 249114 - Remove non-standard Range.compareNode()
Summary: Remove non-standard Range.compareNode()
Status: NEW
Alias: None
Product: WebKit
Classification: Unclassified
Component: DOM (show other bugs)
Version: Safari Technology Preview
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords: BrowserCompat, InRadar
Depends on:
Blocks:
 
Reported: 2022-12-11 15:37 PST by Ahmad Saleem
Modified: 2022-12-18 15:38 PST (History)
3 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Ahmad Saleem 2022-12-11 15:37:20 PST
Hi Team,

While going through Blink commit's, I came across another non-standard, which is now only present in Webkit from quick Github search:

e.g., https://github.com/WebKit/WebKit/blob/bf6ec0b71883ada1ee6d8d0c66a8ea7e421c5e27/Source/WebCore/dom/Range.idl#L68 & https://github.com/WebKit/WebKit/blob/bf6ec0b71883ada1ee6d8d0c66a8ea7e421c5e27/Source/WebCore/dom/Range.cpp#L187

Blink Commit - https://chromium.googlesource.com/chromium/blink/+/9cec195ebb459d3ebdc7ef91a6336047c07ae5b7

Intent to Remove Thread - https://groups.google.com/a/chromium.org/g/blink-dev/c/5dNJaHFNFGM

Some background: Added in 2006 - https://trac.webkit.org/changeset/16302/webkit

MDN - https://developer.mozilla.org/en-US/docs/Web/API/range/compareNode (Removed from Firefox in 1.9)

Web-Spec - https://dom.spec.whatwg.org/#interface-range

Just wanted to raise bug to track it for discussion or for future tracking.

Thanks!
Comment 1 Ryosuke Niwa 2022-12-11 22:37:28 PST
A quick GitHub search finds quite a few entries:
https://github.com/search?q=Range.compareNode+language%3AJavaScript&type=code&l=JavaScript

I don't think it's safe to remove this API.
Comment 2 Radar WebKit Bug Importer 2022-12-18 15:38:16 PST
<rdar://problem/103499504>