
In https://crrev.com/c/6339009, the entire structure tree from the PDF will be attached to the AccessibilityDocInfo struct. Since the AccessibilityDocInfo struct will own, via an std::unique_ptr, the structure tree's root, it should not have any copy constructors and so they are deleted, and the struct is returned via a unique_ptr to clearly indicate the ownership flow. Design doc at: https://docs.google.com/document/d/1ScD93clMA7AtViWINnaQgRTAK3vbpMshQZQXmWkfCyQ/edit?usp=sharing AX-Relnotes: n/a. Bug: 40707542 Change-Id: Ifc5934f5ab4a70a10777040eeb8057dfe1080ce6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6369795 Commit-Queue: Nektarios Paisios <nektar@chromium.org> Reviewed-by: Lei Zhang <thestig@chromium.org> Cr-Commit-Position: refs/heads/main@{#1436092}
//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.