tree 410438b3dbefa6574a2fdc4f612ba0754a19909d
parent e982628c9793d69af9e621cfa3f632a2e58be911
author Lei Zhang <thestig@chromium.org> 1623167702 +0000
committer Pdfium LUCI CQ <pdfium-scoped@luci-project-accounts.iam.gserviceaccount.com> 1623167702 +0000

Remove dependency on fuzzer impls from pdfium target. (try 2)

Add the dependency to the fuzzers target instead for PDFium's compile
checking purposes.

To make the fuzzers actually build properly:
- Add missing dependencies that show up when linking the fuzzer
  executables in Chromium component builds, now that the pdfium target
  no longer depends on the set of all dependencies from all fuzzers.
  All of these are missing because the //fxjs target has circular
  dependencies with the missing targets and cannot depend on them.
- Change the Chromium-side fuzzer build rules to have direct
  dependencies on what it needs, rather than indirectly go through the
  pdfium target. [1]

Then update the pdfium_fuzzer GN template comment to explain the new
approach.

[1] https://crrev.com/c/2946810

Bug: chromium:1217515
Change-Id: Id17f43c2a329d2e7836f0e25c0d2a2ee407c90a5
Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/81634
Reviewed-by: Tom Sepez <tsepez@chromium.org>
Reviewed-by: Thomas Anderson <thomasanderson@chromium.org>
Commit-Queue: Lei Zhang <thestig@chromium.org>