Title: [278116] branches/safari-612.1.15.4-branch/Source/WebKit/UIProcess/Cocoa/WebPageProxyCocoa.mm
Revision
278116
Author
repst...@apple.com
Date
2021-05-26 11:48:53 -0700 (Wed, 26 May 2021)

Log Message

Cherry-pick r278009. rdar://problem/78463104

    Ensure app highlight visibility observer is initialized before adding a new highlight
    https://bugs.webkit.org/show_bug.cgi?id=226214
    rdar://78173370

    Patch by Matt Mokary <mmok...@apple.com> on 2021-05-25
    Reviewed by Wenson Hsieh.

    The app highlight visibility observer is initialized before restoring highlights, but was not initialized
    before adding a new one. It must be initialized before adding a highlight so a change in visibility resulting
    from a new highlight is handled correct.

    * UIProcess/Cocoa/WebPageProxyCocoa.mm:
    (WebKit::WebPageProxy::createAppHighlightInSelectedRange):

    git-svn-id: https://svn.webkit.org/repository/webkit/trunk@278009 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Modified Paths

Diff

Modified: branches/safari-612.1.15.4-branch/Source/WebKit/UIProcess/Cocoa/WebPageProxyCocoa.mm (278115 => 278116)


--- branches/safari-612.1.15.4-branch/Source/WebKit/UIProcess/Cocoa/WebPageProxyCocoa.mm	2021-05-26 18:48:51 UTC (rev 278115)
+++ branches/safari-612.1.15.4-branch/Source/WebKit/UIProcess/Cocoa/WebPageProxyCocoa.mm	2021-05-26 18:48:53 UTC (rev 278116)
@@ -556,6 +556,8 @@
     if (!hasRunningProcess())
         return;
 
+    setUpHighlightsObserver();
+
     send(Messages::WebPage::CreateAppHighlightInSelectedRange(createNewGroup, requestOriginatedInApp));
 }
 
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to