[webkit-changes] [WebKit/WebKit] 4e58e6: [UnifiedPDF] [iOS] Unable to type in plugin passwo...

Abrar Rahman Protyasha noreply at github.com
Thu Dec 19 22:50:22 PST 2024


  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 4e58e62d6fcf2f7932167c3c6d12bbf987131ed1
      https://github.com/WebKit/WebKit/commit/4e58e62d6fcf2f7932167c3c6d12bbf987131ed1
  Author: Abrar Rahman Protyasha <a_protyasha at apple.com>
  Date:   2024-12-19 (Thu, 19 Dec 2024)

  Changed paths:
    M Source/WebKit/WebProcess/WebPage/WebPage.cpp
    M Source/WebKit/WebProcess/WebPage/ios/WebPageIOS.mm

  Log Message:
  -----------
  [UnifiedPDF] [iOS] Unable to type in plugin password field after 287724 at main
https://bugs.webkit.org/show_bug.cgi?id=284977
rdar://141619610

Reviewed by Wenson Hsieh.

In 287724 at main, we undid an assumption in various WebPage methods that
delegated to the plugin for gathering selection information or handling
synthetic clicks, where we moved from calling into a _focused_ plugin in
the frame to doing so for any plugin -- dropping the focus requirement.
This meant that an unfocused plugin element, instead of the focused
password field above it, would be used to populate editor state (among
other things) and would result in inputs made on the password field not
being reflected in the editor state, or cursor updates being correct
when hovering over the password field.

This patch revives said assumption of only consulting the frame's plugin
view if it is focused. For WebPage::completeSyntheticClick() and
WebPage::positionInformation(), we instead decide to consult the plugin
view depending on whether or not the node in question is a plugin
element.

* Source/WebKit/WebProcess/WebPage/WebPage.cpp:
(WebKit::WebPage::editorState const):
* Source/WebKit/WebProcess/WebPage/ios/WebPageIOS.mm:
(WebKit::WebPage::completeSyntheticClick):
(WebKit::WebPage::updateSelectionWithTouches):
(WebKit::WebPage::setSelectionRange):
(WebKit::WebPage::updateSelectionWithExtentPointAndBoundary):
(WebKit::WebPage::positionInformation):

Canonical link: https://commits.webkit.org/288149@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