[webkit-changes] [WebKit/WebKit] c484fa: [UnifiedPDF] PDFPluginBase::getSelectionString() h...

Abrar Rahman Protyasha noreply at github.com
Tue Feb 13 03:31:42 PST 2024


  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: c484fafd02753998e26c1a04e6bdd39cf39cb991
      https://github.com/WebKit/WebKit/commit/c484fafd02753998e26c1a04e6bdd39cf39cb991
  Author: Abrar Rahman Protyasha <a_protyasha at apple.com>
  Date:   2024-02-13 (Tue, 13 Feb 2024)

  Changed paths:
    M Source/WebKit/WebProcess/Plugins/PDF/PDFPlugin.h
    M Source/WebKit/WebProcess/Plugins/PDF/PDFPlugin.mm
    M Source/WebKit/WebProcess/Plugins/PDF/PDFPluginBase.h
    M Source/WebKit/WebProcess/Plugins/PDF/UnifiedPDF/UnifiedPDFPlugin.h
    M Source/WebKit/WebProcess/Plugins/PDF/UnifiedPDF/UnifiedPDFPlugin.mm
    M Source/WebKit/WebProcess/Plugins/PluginView.cpp
    M Source/WebKit/WebProcess/Plugins/PluginView.h
    M Source/WebKit/WebProcess/WebPage/WebPage.cpp
    M Source/WebKit/WebProcess/WebPage/mac/WebPageMac.mm

  Log Message:
  -----------
  [UnifiedPDF] PDFPluginBase::getSelectionString() has an unnecessary "get" prefix
https://bugs.webkit.org/show_bug.cgi?id=269266
rdar://122850109

Reviewed by Chris Dumez.

This patch removes the unnecessary "get" prefix, since the method was
not taking any out-params.

Also, adds an explicit null check in selectionString() to improve
readability (avoids having to reason about messaging a nil object).

* Source/WebKit/WebProcess/Plugins/PDF/PDFPlugin.h:
* Source/WebKit/WebProcess/Plugins/PDF/PDFPlugin.mm:
(WebKit::PDFPlugin::focusPreviousAnnotation):
(WebKit::PDFPlugin::getSelectionString const): Deleted.
* Source/WebKit/WebProcess/Plugins/PDF/PDFPluginBase.h:
* Source/WebKit/WebProcess/Plugins/PDF/UnifiedPDF/UnifiedPDFPlugin.h:
* Source/WebKit/WebProcess/Plugins/PDF/UnifiedPDF/UnifiedPDFPlugin.mm:
(WebKit::UnifiedPDFPlugin::setCurrentSelection):
(WebKit::UnifiedPDFPlugin::getSelectionString const): Deleted.
* Source/WebKit/WebProcess/Plugins/PluginView.cpp:
(WebKit::PluginView::selectionString const):
(WebKit::PluginView::getSelectionString const): Deleted.
* Source/WebKit/WebProcess/Plugins/PluginView.h:
* Source/WebKit/WebProcess/WebPage/WebPage.cpp:
(WebKit::WebPage::editorState const):
* Source/WebKit/WebProcess/WebPage/mac/WebPageMac.mm:
(WebKit::WebPage::getStringSelectionForPasteboard):

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




More information about the webkit-changes mailing list