Hi Team, While going through Blink's commit, I came across another failing test: NOTE - I took updated copy of test from source.chromium.org rather than from commit. Test Case - https://jsfiddle.net/tq84a3pv/show ^ Chrome Canary 111 and Firefox Nightly 110 passes all tests while Safari Technology 161 fails following: Testing LABEL FAIL formOwner should be not defined. Was defined. Testing INPUT FAIL formOwner should be not defined. Was defined. Testing LABEL FAIL formOwner should be not defined. Was defined. Testing INPUT FAIL formOwner should be not defined. Was defined. Testing DIV FAIL formOwner should be not defined. Was defined. Testing A FAIL formOwner should be not defined. Was defined. Testing P FAIL formOwner should be not defined. Was defined. ______ Blink Commit - https://chromium.googlesource.com/chromium/blink/+/ca6002a6e198c84792fed0eb5251a027d7611e9d Just wanted to raise so we can fix it. I haven't checked whether 1-1 merge is possible or not. Thanks!
That's a great find, thank you Ahmad! It's especially important wrt shipping form-associated custom elements. Blinks diff is useful yet 1-1 merge doesn't seem possible. Adding this to my backlog of form-associated follow-ups.
<rdar://problem/104582632>
(In reply to Alexey Shvayka from comment #1) > That's a great find, thank you Ahmad! It's especially important wrt shipping > form-associated custom elements. > > Blinks diff is useful yet 1-1 merge doesn't seem possible. Adding this to my > backlog of form-associated follow-ups. Alexey, I checked on WebKit Turnk and it seems to pass all tests and also STP162, do we need anything else or we can close this or just import test case into our tree? Thanks!
(In reply to Ahmad Saleem from comment #3) > (In reply to Alexey Shvayka from comment #1) > > That's a great find, thank you Ahmad! It's especially important wrt shipping > > form-associated custom elements. > > > > Blinks diff is useful yet 1-1 merge doesn't seem possible. Adding this to my > > backlog of form-associated follow-ups. > > Alexey, I checked on WebKit Turnk and it seems to pass all tests and also > STP162, do we need anything else or we can close this or just import test > case into our tree? > > Thanks! Thank you Ahmad for following up on this! I've also checked the code and it looks like all isConnected() checks are in place on Trunk. This was probably accidentally fixed along with form-associated custom elements implementation. It would be extremely valuable to import this test so we won't accidentally regress when landing follow-ups.
(In reply to Alexey Shvayka from comment #4) > (In reply to Ahmad Saleem from comment #3) > > (In reply to Alexey Shvayka from comment #1) > > > That's a great find, thank you Ahmad! It's especially important wrt shipping > > > form-associated custom elements. > > > > > > Blinks diff is useful yet 1-1 merge doesn't seem possible. Adding this to my > > > backlog of form-associated follow-ups. > > > > Alexey, I checked on WebKit Turnk and it seems to pass all tests and also > > STP162, do we need anything else or we can close this or just import test > > case into our tree? > > > > Thanks! > > Thank you Ahmad for following up on this! I've also checked the code and it > looks like all isConnected() checks are in place on Trunk. > > This was probably accidentally fixed along with form-associated custom > elements implementation. > > It would be extremely valuable to import this test so we won't accidentally > regress when landing follow-ups. Will do PR to import test. Don't worry, I got it. Thanks!
PR - https://github.com/WebKit/WebKit/pull/9823
Committed 260053@main (52f00031f862): <https://commits.webkit.org/260053@main> Reviewed commits have been landed. Closing PR #9823 and removing active labels.