[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