[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