[webkit-changes] [WebKit/WebKit] 21a5d7: [Cocoa] Make WebContextMenuProxy::copySubjectResul...

Wenson Hsieh noreply at github.com
Mon Oct 23 20:54:43 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 21a5d7f6e5b2c3d4fc502f6f1969461856260a43
      https://github.com/WebKit/WebKit/commit/21a5d7f6e5b2c3d4fc502f6f1969461856260a43
  Author: Wenson Hsieh <wenson_hsieh at apple.com>
  Date:   2023-10-23 (Mon, 23 Oct 2023)

  Changed paths:
    M Source/WebKit/UIProcess/WebContextMenuProxy.h
    M Source/WebKit/UIProcess/mac/WebContextMenuProxyMac.h
    M Source/WebKit/UIProcess/mac/WebPageProxyMac.mm

  Log Message:
  -----------
  [Cocoa] Make WebContextMenuProxy::copySubjectResult return a `RetainPtr`
https://bugs.webkit.org/show_bug.cgi?id=263567

Reviewed by Tim Horton.

Clarify the ownership model around this method by returning a `RetainPtr`, and additionally rename
the method so that it doesn't read like it copies the result.

* Source/WebKit/UIProcess/WebContextMenuProxy.h:
(WebKit::WebContextMenuProxy::imageForCopySubject const):
(WebKit::WebContextMenuProxy::copySubjectResult const): Deleted.
* Source/WebKit/UIProcess/mac/WebContextMenuProxyMac.h:
* Source/WebKit/UIProcess/mac/WebPageProxyMac.mm:
(WebKit::WebPageProxy::handleContextMenuCopySubject):

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




More information about the webkit-changes mailing list