
Modernize ReadLoadedBytes() to only take 1 parameter. Along the way: - Document ReadLoadedBytes() behavior. - Remove the DCHECK() in the production implementation, since DocumentLoader will handle the cases when the buffer is too big. - Use base::span::copy_from() in the test implementation. Remove the DCHECK() here as well, since span APIs do even more rigorious checks. Bug: 390223051 Change-Id: If958538615396ff9fd71e6122085a0aa559c1ccd Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6350871 Reviewed-by: Alan Screen <awscreen@chromium.org> Commit-Queue: Lei Zhang <thestig@chromium.org> Cr-Commit-Position: refs/heads/main@{#1432238}
//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.