commit bb85de36d277a449690e167f63a4ee988877f78d
author Dirk Pranke <dpranke@google.com> Fri Aug 21 00:12:44 2020 +0000
committer Chromium commit bot <commit-bot@chromium.org> Fri Aug 21 00:12:44 2020 +0000
tree 609dbcd912d4ede39c5685b7cf771c781aa3755d
parent eb7e2f0aa932426b9f35e3bb8a331cf589963888

Fix visibility on //xfa:xfa_warnings.

The config had a `visibility` set on it that had no
affect. Fortunately for it, GN wasn't enforcing
visibility of configs.

Less fortunately for it, the latest GN does, so we
need to fix this before we can roll to the latest GN.

Bug: crbug.com/1119933
Change-Id: I4ea21ed9d9a7d17a3e7a98b03818676f0722e87f
Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/73010
Auto-Submit: Dirk Pranke <dpranke@google.com>
Commit-Queue: Lei Zhang <thestig@chromium.org>
Reviewed-by: Lei Zhang <thestig@chromium.org>

xfa/BUILD.gn

diff --git a/xfa/BUILD.gn b/xfa/BUILD.gn
index df29737..08986ac 100644
--- a/xfa/BUILD.gn
+++ b/xfa/BUILD.gn
@@ -7,7 +7,7 @@
 assert(pdf_enable_xfa)
 
 config("xfa_warnings") {
-  visibility = [ ":*" ]
+  visibility = [ "./*" ]
   if (is_posix && !is_clang) {  # When GCC.
     cflags = [ "-Wno-strict-overflow" ]
   }