[webkit-changes] [WebKit/WebKit] 003107: Disabled inputs should not get Interaction Regions

Etienne Segonzac noreply at github.com
Sat Oct 21 09:15:00 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 0031075b36df98b33c4fe43e8188a64f1f94965d
      https://github.com/WebKit/WebKit/commit/0031075b36df98b33c4fe43e8188a64f1f94965d
  Author: Etienne Segonzac <sgz at apple.com>
  Date:   2023-10-21 (Sat, 21 Oct 2023)

  Changed paths:
    A LayoutTests/interaction-region/disabled-controls-expected.txt
    A LayoutTests/interaction-region/disabled-controls.html
    M LayoutTests/interaction-region/interaction-layers-culling-expected.txt
    M LayoutTests/interaction-region/labels-expected.txt
    M LayoutTests/interaction-region/labels.html
    M LayoutTests/interaction-region/layer-tree-expected.txt
    M Source/WebCore/page/InteractionRegion.cpp

  Log Message:
  -----------
  Disabled inputs should not get Interaction Regions
https://bugs.webkit.org/show_bug.cgi?id=263379
<rdar://116931433>

Reviewed by Wenson Hsieh and Aditya Keerthi.

Before generating an Interaction Region for an input, or a label
associated with an input, make sure the label is enabled.

* Source/WebCore/page/InteractionRegion.cpp:
(WebCore::shouldAllowElement):
Disallow disabled inputs.
(WebCore::interactionRegionForRenderedRegion):
Check if the label's associated element is enabled.

* LayoutTests/interaction-region/labels.html:
* LayoutTests/interaction-region/labels-expected.txt:
Update the labels test to cover this case. No new region in the
expectation since we're not generating extra regions.
* LayoutTests/interaction-region/disabled-controls-expected.txt: Added.
* LayoutTests/interaction-region/disabled-controls.html: Added.
Add a new test covering disabled controls.

* LayoutTests/interaction-region/interaction-layers-culling-expected.txt:
* LayoutTests/interaction-region/layer-tree-expected.txt:
Re-baseline of layer based tests.

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




More information about the webkit-changes mailing list