Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 292927b976641ccbaa30c5fd67d256589f7fc306
      
https://github.com/WebKit/WebKit/commit/292927b976641ccbaa30c5fd67d256589f7fc306
  Author: Charlie Wolfe <[email protected]>
  Date:   2024-04-14 (Sun, 14 Apr 2024)

  Changed paths:
    M Source/WebKit/UIProcess/BrowsingContextGroup.cpp
    M Source/WebKit/UIProcess/BrowsingContextGroup.h
    M Source/WebKit/UIProcess/WebPageProxy.cpp

  Log Message:
  -----------
  FindString should not be sent twice if a webpage has no iframe processes
https://bugs.webkit.org/show_bug.cgi?id=271029
rdar://124652108

Reviewed by Alex Christensen.

With site isolation, we need to search all web processes initially without 
setting selection because we
do not know which frame should receive selection until we know which frames 
have matches. If a webpage
has no iframe processes, or site isolation is disabled, we don’t need to do 
this because we can
synchronously search all frames in the web process. So we should just send one 
FindString message in
this situation.

* Source/WebKit/UIProcess/BrowsingContextGroup.cpp:
(WebKit::BrowsingContextGroup::hasRemotePages):
* Source/WebKit/UIProcess/BrowsingContextGroup.h:
* Source/WebKit/UIProcess/WebPageProxy.cpp:
(WebKit::WebPageProxy::findString):

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



To unsubscribe from these emails, change your notification settings at 
https://github.com/WebKit/WebKit/settings/notifications
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to