Bug 235530

Summary: [Forms] make sure :active is removed on keyup for radios
Product: WebKit Reporter: zsun
Component: FormsAssignee: zsun
Status: RESOLVED FIXED    
Severity: Normal CC: cdumez, changseok, esprehn+autocc, ews-watchlist, gyuyoung.kim, koivisto, mifenton, ntim, webkit-bug-importer, wenson_hsieh
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Patch
none
Patch
ews-feeder: commit-queue-
Patch ews-feeder: commit-queue-

Description zsun 2022-01-24 11:20:05 PST
WPT test-

imported/w3c/web-platform-tests/html/semantics/forms/the-input-element/radio-double-activate-pseudo.html
Comment 1 zsun 2022-01-24 11:24:45 PST
Created attachment 449834 [details]
Patch
Comment 2 Radar WebKit Bug Importer 2022-01-31 11:21:46 PST
<rdar://problem/88283930>
Comment 3 zsun 2022-02-03 01:26:40 PST
Created attachment 450745 [details]
Patch
Comment 4 Chris Dumez 2022-03-28 11:39:14 PDT
Comment on attachment 450745 [details]
Patch

Seems we need a layout test.
Comment 5 Tim Nguyen (:ntim) 2022-05-16 15:37:22 PDT
Why does html/semantics/forms/the-input-element/radio-double-activate-pseudo.html not start passing?
Comment 6 zsun 2022-05-17 01:49:44 PDT
Created attachment 459490 [details]
Patch
Comment 7 zsun 2022-05-17 01:53:32 PDT
(In reply to Tim Nguyen (:ntim) from comment #5)
> Why does
> html/semantics/forms/the-input-element/radio-double-activate-pseudo.html not
> start passing?

bug 237589 corrected the test expectation file for test html/semantics/forms/the-input-element/radio-double-activate-pseudo.html.

Patch has been updated and the test should pass now.
Comment 8 Chris Dumez 2022-05-17 07:56:24 PDT
Comment on attachment 459490 [details]
Patch

r=me
Comment 9 Tim Nguyen (:ntim) 2022-05-17 10:39:09 PDT
For this bug and bug 235522, note that changelogs are now deprecated (see the webkit-dev message), you'll need to use the Github workflow with a complete commit message.
Comment 10 zsun 2022-05-19 02:12:51 PDT
Created attachment 459573 [details]
Patch
Comment 11 EWS 2022-05-19 04:37:47 PDT
Committed r294475 (250734@main): <https://commits.webkit.org/250734@main>

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