Bug 258073 - Implement 'dirtiness' of HTMLOptionElement
Summary: Implement 'dirtiness' of HTMLOptionElement
Status: NEW
Alias: None
Product: WebKit
Classification: Unclassified
Component: Forms (show other bugs)
Version: Safari Technology Preview
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL: http://wpt.live/html/semantics/forms/...
Keywords: BrowserCompat, InRadar, WPTImpact
Depends on:
Blocks:
 
Reported: 2023-06-14 05:05 PDT by Ahmad Saleem
Modified: 2024-07-15 07:48 PDT (History)
8 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-06-14 05:05:59 PDT
Hi Team,

We need to implement 'dirtiness' for HTMLOptionElement to pass following failing WPT tests:

Test: https://wpt.fyi/results/html/semantics/forms/the-option-element/option-selected.html?label=experimental&label=master&aligned

Blink Commit: https://chromium.googlesource.com/chromium/src.git/+/a17cc1eda58ad8270dc56ebafab215e67e6c1bfe

Just wanted to raise, so we can fix it.

Thanks!
Comment 1 Tim Nguyen (:ntim) 2023-06-14 20:07:22 PDT
Seems like an 1:1 port.
Comment 2 Ahmad Saleem 2023-06-18 16:27:47 PDT
Blink patch will cause regression, so this is follow-up: https://chromium.googlesource.com/chromium/src.git/+/f67251360e4b76d188bb1523d7a0952c2f449e68
Comment 3 Ahmad Saleem 2023-06-18 16:37:45 PDT
(In reply to Ahmad Saleem from comment #2)
> Blink patch will cause regression, so this is follow-up:
> https://chromium.googlesource.com/chromium/src.git/+/
> f67251360e4b76d188bb1523d7a0952c2f449e68

Also another regression: https://chromium.googlesource.com/chromium/src/+/6d095ec7a71eb72e2ad9992d1ca3080f5726f6bf
Comment 4 Radar WebKit Bug Importer 2023-06-21 05:06:17 PDT
<rdar://problem/111096541>
Comment 5 Ahmad Saleem 2023-07-07 08:40:20 PDT
I merged these patches and got them compiled but got crashes in three tests. So I might be doing something wrong. Happy to zip and attach screenshots of my local patches.
Comment 6 Ahmad Saleem 2024-07-14 12:53:28 PDT
I tried this again locally and now running via WIP to see what happens - last time it caused crashes locally but this time didn't.

https://github.com/WebKit/WebKit/pull/30802

In WIP / Draft - would be good to at least post what I tried locally.
Comment 7 Ahmad Saleem 2024-07-15 07:48:04 PDT
Just documenting progress on bug rather than PR - it leads to failing two tests:

accessibility/listbox-clear-selection.html 
accessibility/select-element-at-index.html

I haven't explored yet but good to maintain list (some time old EWS runs are not accessible - hence documenting it here).