Include more Skia files in GN checks
Includes more Skia source files within the scope of "gn check", which
only considers headers reachable from the GN build graph. This avoids
later roll issues when Chromium has stricter checks than PDFium.
This change introduces a new top-level "gn_check" target that is only
used for GN reachability, and is not built by default.
Change-Id: I9057509461ba5b256610c21048458e7fc685e615
Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/101050
Reviewed-by: Lei Zhang <thestig@chromium.org>
Commit-Queue: Lei Zhang <thestig@chromium.org>
Auto-Submit: K. Moon <kmoon@chromium.org>
diff --git a/BUILD.gn b/BUILD.gn
index 3c49e84..689ea87 100644
--- a/BUILD.gn
+++ b/BUILD.gn
@@ -572,3 +572,16 @@
]
}
}
+
+# Makes additional targets reachable only for "gn check". These are not always
+# built by the "all" Ninja target, which uses the "default" group, which in turn
+# depends on the "pdfium_all" group.
+group("gn_check") {
+ deps = []
+
+ # TODO(crbug.com/pdfium/1832): Remove !is_android when //third_party/expat is
+ # available.
+ if (!is_android) {
+ deps += [ "//skia" ]
+ }
+}
