[webkit-changes] [WebKit/WebKit] ba02cc: Displayed datalist dropdown is out of sync with da...
Rob Buis
noreply at github.com
Tue Feb 13 22:44:26 PST 2024
Branch: refs/heads/main
Home: https://github.com/WebKit/WebKit
Commit: ba02cc6aade5caa1ddecb92778f6bfe82d5bef25
https://github.com/WebKit/WebKit/commit/ba02cc6aade5caa1ddecb92778f6bfe82d5bef25
Author: Rob Buis <rbuis at igalia.com>
Date: 2024-02-13 (Tue, 13 Feb 2024)
Changed paths:
A LayoutTests/fast/forms/datalist/datalist-textinput-dynamically-add-options-expected.txt
A LayoutTests/fast/forms/datalist/datalist-textinput-dynamically-add-options-on-keydown-expected.txt
A LayoutTests/fast/forms/datalist/datalist-textinput-dynamically-add-options-on-keydown.html
A LayoutTests/fast/forms/datalist/datalist-textinput-dynamically-add-options.html
M Source/WebCore/html/DataListSuggestionInformation.h
M Source/WebCore/html/HTMLDataListElement.cpp
M Source/WebCore/html/HTMLDataListElement.h
M Source/WebCore/html/TextFieldInputType.cpp
M Source/WebKit/Shared/WebCoreArgumentCoders.serialization.in
M Source/WebKit/UIProcess/ios/WebDataListSuggestionsDropdownIOS.mm
Log Message:
-----------
Displayed datalist dropdown is out of sync with datalist options elements after DOM update
https://bugs.webkit.org/show_bug.cgi?id=201121
Reviewed by Aditya Keerthi.
Adding datalist dropdown items by dynamically inserting options to the datalist on keydown/input change
was not taking effect right away. To fix this, detect datalist DOM changes through HTMLDataListElement::childrenChanged
and notify the corresponding input. This will eventually land in TextFieldInputType::dataListMayHaveChanged which
now ends with trying to display the suggestions if the input is focused.
Note that here a new DataListSuggestionActivationType value DataListMayHaveChanged is added to distinguish from
current usages (text changed, indicator/control clicked).
* LayoutTests/fast/forms/datalist/datalist-textinput-dynamically-add-options-expected.txt: Added.
* LayoutTests/fast/forms/datalist/datalist-textinput-dynamically-add-options-on-keydown-expected.txt: Added.
* LayoutTests/fast/forms/datalist/datalist-textinput-dynamically-add-options-on-keydown.html: Added.
* LayoutTests/fast/forms/datalist/datalist-textinput-dynamically-add-options.html: Added.
* Source/WebCore/html/DataListSuggestionInformation.h:
* Source/WebCore/html/HTMLDataListElement.cpp:
(WebCore::HTMLDataListElement::childrenChanged):
* Source/WebCore/html/HTMLDataListElement.h:
* Source/WebCore/html/TextFieldInputType.cpp:
(WebCore::TextFieldInputType::dataListMayHaveChanged):
(WebCore::TextFieldInputType::displaySuggestions):
* Source/WebKit/Shared/WebCoreArgumentCoders.serialization.in:
* Source/WebKit/UIProcess/ios/WebDataListSuggestionsDropdownIOS.mm:
(-[WKDataListSuggestionsDropdown _displayWithActivationType:]):
Canonical link: https://commits.webkit.org/274608@main
More information about the webkit-changes
mailing list