Bug 247647

Summary:	Sync Document.readyState with the HTML Specification
Product:	WebKit	Reporter:	Ahmad Saleem <ahmad.saleem792>
Component:	DOM	Assignee:	Nobody <webkit-unassigned>
Status:	RESOLVED WORKSFORME
Severity:	Normal	CC:	akeerthi, cdumez, karlcow, rniwa, webkit-bug-importer
Priority:	P2	Keywords:	InRadar
Version:	WebKit Nightly Build
Hardware:	Unspecified
OS:	Unspecified

Description Ahmad Saleem 2022-11-08 14:50:16 PST

Hi Team,

While going through the WebDIL, I noticed that we are missing following:

'''
readonly attribute DocumentReadyState readyState;
'''

Web-Spec: https://html.spec.whatwg.org/#the-document-object

Checked with "readyState" and we only have "enum" but not above "read-only" bit.

https://github.com/WebKit/WebKit/blob/main/Source/WebCore/dom/Document.idl

___

I am just creating this bug to align with specs.

Thanks!

Comment 1 Ahmad Saleem 2022-11-08 14:54:33 PST

Note - Chrome do have it - https://source.chromium.org/chromium/chromium/src/+/main:third_party/blink/renderer/core/dom/document.idl;l=90?q=document.idl

Comment 2 Karl Dubost 2022-11-09 15:47:59 PST

What about 
https://github.com/WebKit/WebKit/blob/eece793cfe01232ecbbf6a69457b83fcbfac896a/LayoutTests/imported/w3c/web-platform-tests/interfaces/html.idl#L57

Comment 3 Karl Dubost 2022-11-09 15:48:43 PST

Same for Gecko
https://searchfox.org/mozilla-central/rev/99f83e78415867726a5076a525e4a60840f3be33/testing/web-platform/tests/interfaces/html.idl#57

Comment 4 Radar WebKit Bug Importer 2022-11-15 14:51:29 PST

<rdar://problem/102388385>

Comment 5 Karl Dubost 2022-11-15 15:45:57 PST

I must not have been fully awake when I posted these two comments. Sigh.


It's not in https://searchfox.org/wubkat/source/Source/WebCore/dom/Document.idl#102
but I see it here
https://searchfox.org/wubkat/source/Source/WebCore/dom/Document+HTML.idl#38

Comment 6 Karl Dubost 2022-11-15 15:49:42 PST

I confirmed internally, this is working as intended.