-
Notifications
You must be signed in to change notification settings - Fork 1.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[UnifiedPDF] PDFPluginBase should notify the web page of selection changes #23755
Conversation
EWS run on previous version of this PR (hash 3b74ce7)
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM but I'll let a proper PDFPlugin engineer do the formal review.
EWS run on previous version of this PR (hash a3dd9df) |
EWS run on current version of this PR (hash d46739f)
|
…anges https://bugs.webkit.org/show_bug.cgi?id=268618 rdar://122179410 Reviewed by Simon Fraser. In anticipation of selection tracking support in UnifiedPDFPlugin, this patch factors out PDFPlugin::notifySelectionChanged into a common space. We also take the liberty to drop the PDFSelection argument to notifySelectionChanged, given that it was not being read or passed to the web page by the PDF plugin. Lastly, we add a WebFrame::protectedCoreLocalFrame getter to better abide by our smart pointer rules, where we protect both the objects on which we call non-trivial member functions, and parameters we pass to such functions. * Source/WebKit/WebProcess/Plugins/PDF/PDFPlugin.h: * Source/WebKit/WebProcess/Plugins/PDF/PDFPlugin.mm: (-[WKPDFLayerControllerDelegate pdfLayerController:didChangeSelection:]): (WebKit::PDFPlugin::notifySelectionChanged): Deleted. * Source/WebKit/WebProcess/Plugins/PDF/PDFPluginBase.h: * Source/WebKit/WebProcess/Plugins/PDF/PDFPluginBase.mm: (WebKit::PDFPluginBase::notifySelectionChanged): * Source/WebKit/WebProcess/WebPage/WebFrame.cpp: (WebKit::WebFrame::protectedCoreLocalFrame const): * Source/WebKit/WebProcess/WebPage/WebFrame.h: Canonical link: https://commits.webkit.org/274005@main
d46739f
to
749aa44
Compare
Committed 274005@main (749aa44): https://commits.webkit.org/274005@main Reviewed commits have been landed. Closing PR #23755 and removing active labels. |
749aa44
d46739f
🛠 wincairo🧪 wpe-wk2🧪 ios-wk2🧪 api-mac🧪 ios-wk2-wpt🧪 api-ios🧪 mac-wk2🧪 gtk-wk2🧪 mac-AS-debug-wk2🧪 api-gtk🛠 tv-sim🛠 watch-sim