[webkit-changes] [WebKit/WebKit] a27ac6: Restore 100, 000 limit in HTMLOptionsCollection.len...

Ahmad Saleem noreply at github.com
Mon Feb 27 12:00:07 PST 2023


  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: a27ac64f683e8d0a798123758cb4bf5a0bc75cd0
      https://github.com/WebKit/WebKit/commit/a27ac64f683e8d0a798123758cb4bf5a0bc75cd0
  Author: Ahmad Saleem <ahmad.saleem792+github at gmail.com>
  Date:   2023-02-27 (Mon, 27 Feb 2023)

  Changed paths:
    M LayoutTests/fast/dom/HTMLSelectElement/select-selectedIndex-expected.txt
    M LayoutTests/fast/dom/HTMLSelectElement/select-selectedIndex-multiple-expected.txt
    M LayoutTests/fast/forms/select-max-length-expected.txt
    A LayoutTests/imported/w3c/web-platform-tests/html/select/options-length-too-large-expected.txt
    A LayoutTests/imported/w3c/web-platform-tests/html/select/options-length-too-large.html
    M Source/WebCore/html/HTMLSelectElement.cpp

  Log Message:
  -----------
  Restore 100,000 limit in HTMLOptionsCollection.length setter

https://bugs.webkit.org/show_bug.cgi?id=252983

Reviewed by Chris Dumez.

This patch is to align WebKit with Blink / Chromium and Gecko / Firefox.

Merge - https://chromium.googlesource.com/chromium/src/+/f27e6ea87ecf211c8b8644813422a9e7f19cd1cc

This patch updates 'maxSelectItems' to new value of 100,000
to reflect update in spec.
Further, it is updated to be only used when new length is
greater than current length.
Additionally, add comments to reflect the details as needed.

Web-Spec: https://html.spec.whatwg.org/#dom-htmloptionscollection-length
Issue: https://github.com/whatwg/html/issues/8337

* Source/WebCore/html/HTMLSelectElement.cpp:
(maxSelectItems): Update constant value
(HTMLSelectElement::setItem): Remove '=' and update comments and console message
(HTMLSelectElement::setLength): Add comment and update console message
* LayoutTests/imported/w3c/web-platform-tests/html/select/options-length-too-large.html: Add Test Case
* LayoutTests/imported/w3c/web-platform-tests/html/select/options-length-too-large-expected.txt: Add Test Case Expectation
* LayoutTests/fast/forms/select-max-length-expected.txt: Rebaselined
* LayoutTests/fast/dom/HTMLSelectElement/select-selectedIndex-multiple-expected.txt: Rebaselined
* LayoutTests/fast/dom/HTMLSelectElement/select-selectedIndex-expected.txt: Rebaselined

Canonical link: https://commits.webkit.org/260896@main




More information about the webkit-changes mailing list