[webkit-changes] [WebKit/WebKit] 208d70: Cherry-pick 276895 at main (b52f2b8abac2). https://bu...
Aditya Keerthi
noreply at github.com
Mon May 13 00:30:49 PDT 2024
Branch: refs/heads/webkitglib/2.44
Home: https://github.com/WebKit/WebKit
Commit: 208d709bf7604891c1b3f8a082cd29b95f83fef9
https://github.com/WebKit/WebKit/commit/208d709bf7604891c1b3f8a082cd29b95f83fef9
Author: Aditya Keerthi <akeerthi at apple.com>
Date: 2024-05-13 (Mon, 13 May 2024)
Changed paths:
A LayoutTests/fast/forms/email-input-type-value-and-multiple-attributes-set-expected.html
A LayoutTests/fast/forms/email-input-type-value-and-multiple-attributes-set.html
M Source/WebCore/html/EmailInputType.cpp
Log Message:
-----------
Cherry-pick 276895 at main (b52f2b8abac2). https://bugs.webkit.org/show_bug.cgi?id=271043
REGRESSION (273461 at main): The value attribute is not displayed in an input field with type="email" and multiple attributes
https://bugs.webkit.org/show_bug.cgi?id=271043
rdar://125221858
Reviewed by Ryosuke Niwa.
273461 at main deferred input element shadow tree construction outside of attribute
parsing. However, when combined with an earlier change to sanitize the value of
email inputs when the multiple attribute is set (263555 at main), the deferral of
shadow tree construction results in the value not being displayed at all.
The root cause of the issue is the use of `setValueFromRenderer` introduced by
263555 at main. The use of that method in `EmailInputType::attributeChanged` has
always been inappropriate, as it is specifically meant for user modifications.
However, in this case, the problem arises from the fact that `setValueFromRenderer`
calls `setFormControlValueMatchesRenderer(true)`. This results in the elision
of shadow tree construction in `TextFieldInputType::updateInnerTextValue`, as
`HTMLTextFormControlElement::setInnerTextValue` is only called if
`formControlValueMatchesRenderer()` returns false. Before 273461 at main, this elision
was benign, as the shadow tree was constructed at attribute parse time.
Fix by using a more appropriate method to update the email input's value when
the multiple attribute is changed.
A new regression test is added. The coverage for the issue fixed by 263555 at main
continues to be provided by the WPT: html/semantics/forms/the-input-element/email.html.
* LayoutTests/fast/forms/email-input-type-value-and-multiple-attributes-set-expected.html: Added.
* LayoutTests/fast/forms/email-input-type-value-and-multiple-attributes-set.html: Added.
* Source/WebCore/html/EmailInputType.cpp:
(WebCore::EmailInputType::attributeChanged):
Canonical link: https://commits.webkit.org/276895@main
To unsubscribe from these emails, change your notification settings at https://github.com/WebKit/WebKit/settings/notifications
More information about the webkit-changes
mailing list