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 | ||
Ahmad Saleem
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!
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Ahmad Saleem
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
Karl Dubost
What about
https://github.com/WebKit/WebKit/blob/eece793cfe01232ecbbf6a69457b83fcbfac896a/LayoutTests/imported/w3c/web-platform-tests/interfaces/html.idl#L57
Karl Dubost
Same for Gecko
https://searchfox.org/mozilla-central/rev/99f83e78415867726a5076a525e4a60840f3be33/testing/web-platform/tests/interfaces/html.idl#57
Radar WebKit Bug Importer
<rdar://problem/102388385>
Karl Dubost
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
Karl Dubost
I confirmed internally, this is working as intended.