[webkit-changes] [WebKit/WebKit] 7728d2: Unified PDF: HUDs for embedded PDFs don't re-posit...

Tim Horton noreply at github.com
Tue Feb 6 15:37:16 PST 2024


  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 7728d242f2cb3fc400c5aecee69088e237dc5678
      https://github.com/WebKit/WebKit/commit/7728d242f2cb3fc400c5aecee69088e237dc5678
  Author: Tim Horton <thorton at apple.com>
  Date:   2024-02-06 (Tue, 06 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/PDFPluginBase.mm
    M Source/WebKit/WebProcess/Plugins/PDF/UnifiedPDF/UnifiedPDFPlugin.h
    M Source/WebKit/WebProcess/Plugins/PDF/UnifiedPDF/UnifiedPDFPlugin.mm

  Log Message:
  -----------
  Unified PDF: HUDs for embedded PDFs don't re-position after page zoom
https://bugs.webkit.org/show_bug.cgi?id=268855
rdar://122038277

Reviewed by Simon Fraser.

* Source/WebKit/WebProcess/Plugins/PDF/PDFPlugin.h:
* Source/WebKit/WebProcess/Plugins/PDF/PDFPlugin.mm:
(WebKit::PDFPlugin::geometryDidChange):
* Source/WebKit/WebProcess/Plugins/PDF/PDFPluginBase.h:
* Source/WebKit/WebProcess/Plugins/PDF/PDFPluginBase.mm:
(WebKit::PDFPluginBase::geometryDidChange):
* Source/WebKit/WebProcess/Plugins/PDF/UnifiedPDF/UnifiedPDFPlugin.h:
* Source/WebKit/WebProcess/Plugins/PDF/UnifiedPDF/UnifiedPDFPlugin.mm:
(WebKit::UnifiedPDFPlugin::geometryDidChange):
Make PDFPluginBase::geometryDidChange report back whether there was any change,
and use that to bail, instead of implementing the bailing in each subclass.

Don't bail if the transform changes, since we need to update the HUD (at least)
in that case, and that matches historical behavior.

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




More information about the webkit-changes mailing list