
Change PdfViewWebPlugin::HandleSetPresentationModeMessage() to reset the cursor type when entering presentation mode. This prevents presentation mode from being stuck with the most recently used cursor, which may be a cursor used when editing. Add a test case for HandleSetPresentationModeMessage() to check this behavior and backfill a missing test to check PDFiumEngine correctly enters read-only mode. Add some test-only accessors to PdfViewWebPlugin to support this test. Bug: 383029258 Change-Id: Ieef1a4b93ef1045c454d4826cf348ffb6b90d6f1 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6082220 Commit-Queue: Lei Zhang <thestig@chromium.org> Reviewed-by: Andy Phan <andyphan@chromium.org> Cr-Commit-Position: refs/heads/main@{#1395530}
//pdf
contains the PDF plugin, its Blink-based replacement, as well as PDF
utility functions that leverage PDFium. It can use low-level components that
live below the content layer, as well as other foundational code like
//printing
. It should not use //content
or anything in //components
that
lives above the content layer. Code that lives above the content layer should
live in //components/pdf
, or in the embedder. All the code here should run in
sandboxed child processes.
TODO(crbug.com/40186598): Remove existing //content
dependencies.