Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 4221343a3e61e96082cef53d6bfd5b33003d3287
      
https://github.com/WebKit/WebKit/commit/4221343a3e61e96082cef53d6bfd5b33003d3287
  Author: Zak Ridouh <[email protected]>
  Date:   2026-05-15 (Fri, 15 May 2026)

  Changed paths:
    A 
LayoutTests/http/tests/site-isolation/resources/validation-message-iframe.html
    A 
LayoutTests/http/tests/site-isolation/validation-message-cross-origin-iframe-expected.txt
    A 
LayoutTests/http/tests/site-isolation/validation-message-cross-origin-iframe.html
    M Source/WebCore/page/LocalFrameView.h
    M Source/WebKit/UIProcess/WebPageProxy.cpp
    M Source/WebKit/UIProcess/WebPageProxy.h
    M Source/WebKit/UIProcess/WebPageProxy.messages.in
    M Source/WebKit/UIProcess/gtk/WebPageProxyGtk.cpp
    M Source/WebKit/UIProcess/ios/WebPageProxyIOS.mm
    M Source/WebKit/UIProcess/mac/WebPageProxyMac.mm
    M Source/WebKit/WebProcess/WebCoreSupport/WebValidationMessageClient.cpp

  Log Message:
  -----------
  [Site Isolation] Validation message bubble is positioned incorrectly in 
cross-origin iframes
https://bugs.webkit.org/show_bug.cgi?id=312399
rdar://174855372

Reviewed by Aditya Keerthi.

Add rootFrameID to the ShowValidationMessage IPC message and call
convertRectToMainFrameCoordinates() on the UIProcess side before displaying
the validation bubble. This converts the anchor rect from subframe coordinates
to main-frame coordinates when the element is inside a cross-origin iframe.

Updated the Mac, iOS, and GTK handlers to use the async coordinate conversion
pattern already established by the color picker, datalist, and datetime picker
fixes.

Test: http/tests/site-isolation/validation-message-cross-origin-iframe.html

* 
LayoutTests/http/tests/site-isolation/resources/validation-message-iframe.html: 
Added.
* 
LayoutTests/http/tests/site-isolation/validation-message-cross-origin-iframe-expected.txt:
 Added.
* 
LayoutTests/http/tests/site-isolation/validation-message-cross-origin-iframe.html:
 Added.
* Source/WebCore/page/LocalFrameView.h:
* Source/WebKit/UIProcess/WebPageProxy.cpp:
(WebKit::WebPageProxy::showValidationMessage):
* Source/WebKit/UIProcess/WebPageProxy.h:
* Source/WebKit/UIProcess/WebPageProxy.messages.in:
* Source/WebKit/UIProcess/gtk/WebPageProxyGtk.cpp:
(WebKit::WebPageProxy::showValidationMessageWithMainFrameRect):
(WebKit::WebPageProxy::showValidationMessage):
* Source/WebKit/UIProcess/ios/WebPageProxyIOS.mm:
(WebKit::WebPageProxy::showValidationMessageWithMainFrameRect):
(WebKit::WebPageProxy::showValidationMessage):
* Source/WebKit/UIProcess/mac/WebPageProxyMac.mm:
(WebKit::WebPageProxy::showValidationMessageWithMainFrameRect):
(WebKit::WebPageProxy::showValidationMessage):
* Source/WebKit/WebProcess/WebCoreSupport/WebValidationMessageClient.cpp:
(WebKit::WebValidationMessageClient::showValidationMessage):

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



To unsubscribe from these emails, change your notification settings at 
https://github.com/WebKit/WebKit/settings/notifications

Reply via email to