0

Remove stray dependencies in pdf/BUILD.gn.

- out_of_process_instance target doesn't use //base:i18n.
- internal target doesn't use //pdf/pdfium/fuzzers, but once that
  dependency gets removed, //pdf/pdfium/fuzzers is no longer in GN's
  build graph. Fix that by adding it to the "gn_all" target.

Change-Id: I811f085ce5666fcf353d2cb429e52a84e1fbd1f6
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2967774
Reviewed-by: Daniel Hosseinian <dhoss@chromium.org>
Reviewed-by: Yaron Friedman <yfriedman@chromium.org>
Commit-Queue: Lei Zhang <thestig@chromium.org>
Cr-Commit-Position: refs/heads/master@{#894312}
This commit is contained in:
Lei Zhang
2021-06-21 18:23:13 +00:00
committed by Chromium LUCI CQ
parent d2ae731be5
commit 9d59491442
2 changed files with 4 additions and 3 deletions

@ -292,7 +292,10 @@ group("gn_all") {
}
if (enable_pdf) {
deps += [ "//third_party/pdfium/samples:pdfium_test" ]
deps += [
"//pdf/pdfium/fuzzers",
"//third_party/pdfium/samples:pdfium_test",
]
}
if (enable_remoting) {

@ -188,7 +188,6 @@ if (enable_pdf) {
"//gin",
"//net",
"//pdf:buildflags",
"//pdf/pdfium/fuzzers",
"//ppapi/cpp:objects",
"//ppapi/cpp/private:internal_module",
"//printing",
@ -319,7 +318,6 @@ if (enable_pdf) {
":internal",
":ppapi_migration",
"//base",
"//base:i18n",
"//build:chromeos_buildflags",
"//net",
"//pdf:buildflags",