Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 1ba1f5ef79b5665f85d7b57acd3384cd64ae8179
      
https://github.com/WebKit/WebKit/commit/1ba1f5ef79b5665f85d7b57acd3384cd64ae8179
  Author: Abrar Rahman Protyasha <a_protya...@apple.com>
  Date:   2024-02-05 (Mon, 05 Feb 2024)

  Changed paths:
    M Source/WebKit/WebProcess/Plugins/PDF/UnifiedPDF/UnifiedPDFPlugin.mm
    M Source/WebKit/WebProcess/Plugins/PluginView.cpp

  Log Message:
  -----------
  [UnifiedPDF] Pinch-to-zoom starts zooming from original scale factor after 
zooming through the context menu
https://bugs.webkit.org/show_bug.cgi?id=268764
rdar://122327672

Reviewed by Simon Fraser.

We fail to inform the UIP about a change in the page scale factor
through context menu actions, so when we try a gesture that sends the
PDF plugin a scale factor from the UIP, we receive out-dated
information.

This patch addresses the bug by calling PluginView::setPageScaleFactor,
which sends the PluginScaleFactorDidChange and PluginZoomFactorDidChange
IPC messages to the UIP, and then also calls into
UnifiedPDFPlugin::setPageScaleFactor.

Finally, we move the IPC calls in PluginView::setPageScaleFactor to
after the plugin view initialized check. This ensures that we don't
communicate a page scale factor change when one hasn't happened.

* Source/WebKit/WebProcess/Plugins/PDF/UnifiedPDF/UnifiedPDFPlugin.mm:
(WebKit::UnifiedPDFPlugin::performContextMenuAction):
(WebKit::UnifiedPDFPlugin::zoomIn):
(WebKit::UnifiedPDFPlugin::zoomOut):
* Source/WebKit/WebProcess/Plugins/PluginView.cpp:
(WebKit::PluginView::setPageScaleFactor):

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


_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to