Bug 266743 - Remove 'DOMPointInit' constructor from DOMPoint.idl and DOMPointReadOnly.idl
Summary: Remove 'DOMPointInit' constructor from DOMPoint.idl and DOMPointReadOnly.idl
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: CSS (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords: BrowserCompat, InRadar, WPTImpact
Depends on:
Blocks:
 
Reported: 2023-12-20 16:40 PST by Ahmad Saleem
Modified: 2024-07-22 07:47 PDT (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 2023-12-20 16:40:20 PST
Hi Team,

While trying to understand test failures in 'CSS' especially below:

https://wpt.fyi/results/css/geometry/DOMPoint-001.html?label=master&label=experimental&aligned&q=safari%3Afail

and

https://wpt.fyi/results/css/geometry/DOMPoint-002.html?label=master&label=experimental&aligned&q=safari%3Afail

and noticed this following comment as well.

https://github.com/WebKit/WebKit/blob/926054f254028ee2cc29b0e96a50cca42592ce66/Source/WebCore/dom/DOMPoint.idl#L31

"The DOMPointInit constructor exists in https://www.w3.org/TR/geometry-1/ but is removed in https://drafts.fxtf.org/geometry/"

and if I remove following line:

https://github.com/WebKit/WebKit/blob/926054f254028ee2cc29b0e96a50cca42592ce66/Source/WebCore/dom/DOMPoint.idl#L36

and also from here:

https://github.com/WebKit/WebKit/blob/926054f254028ee2cc29b0e96a50cca42592ce66/Source/WebCore/dom/DOMPointReadOnly.idl#L37

We start progressing above failing test case. I don't know whether we have C++ code for this constructor or not but just raising so we can fix it.

CCing - Matt & Simon for their input.

Thanks!
Comment 1 Ahmad Saleem 2023-12-20 16:41:58 PST
NOTE - I also have another change and it is to make both IDL files have 'Serializable' as required by web-spec in my local build.
Comment 2 Ahmad Saleem 2023-12-20 17:49:07 PST
This PR added following constructor:

https://commits.webkit.org/183333@main
Comment 3 Ahmad Saleem 2023-12-20 18:12:18 PST
(In reply to Ahmad Saleem from comment #2)
> This PR added following constructor:
> 
> https://commits.webkit.org/183333@main

Simple revert of this leads to compile failures in DOMQuad.cpp:

https://github.com/WebKit/WebKit/blob/5ce41d80d50760401a9399bc7cb3d3be7b97b58e/Source/WebCore/dom/DOMQuad.cpp#L40
Comment 4 Radar WebKit Bug Importer 2023-12-27 16:41:11 PST
<rdar://problem/120222447>
Comment 5 EWS 2024-07-22 07:47:57 PDT
Committed 281192@main (98dc100a1cd7): <https://commits.webkit.org/281192@main>

Reviewed commits have been landed. Closing PR #31006 and removing active labels.