
Store a PdfInkBrush for every brush type in PdfInkModule. This is to prepare for storing all brush data in PdfInkModule instead of in the frontend. All brush data of all types must persist, even when changing brushes. Change the PdfInkBrush fields from unique ptrs to direct objects and use setter methods to change the brush values. This reduces the amount of object creation. Bug: 373672165 Change-Id: I45af45186775fc0da7678ff6ea4aa3b2ad074c4a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5932590 Code-Coverage: findit-for-me@appspot.gserviceaccount.com <findit-for-me@appspot.gserviceaccount.com> Commit-Queue: Andy Phan <andyphan@chromium.org> Reviewed-by: Lei Zhang <thestig@chromium.org> Cr-Commit-Position: refs/heads/main@{#1370362}
//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.