Improve GN public header file checking

This makes the header checker and include iterator work from InputFiles (which basically just hold the file buffer) rather than just raw strings. This allows us to reference these files from Err.

Some extra line/char tracking is now in the include iterator so we can actually quote from and annotate the place in the source file where the bad include is, which looks much nicer than "the file blah included blah". It also makes it possible to write much shorter error messages that still make sense.

This adds visibility checking as previously documented in the help for "public" and a unit test for that.

This updates the documentation for "public" which was wrong (it referred to patterns which was not used).

R=scottmg@chromium.org

Review URL: https://codereview.chromium.org/229423002

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@262747 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 6 insertions, 1 deletions

diff --git a/BUILD.gn b/BUILD.gn
index 224dedb..e8dbc1d 100644
--- a/BUILD.gn
+++ b/BUILD.gn
@@ -32,7 +32,6 @@ group("root") {
     #"//sdch",
     #"//skia",
     #"//third_party/WebKit/Source/platform",
-    "//third_party/freetype2",
     #"//third_party/icu:icudata",
     #"//third_party/leveldatabase",
     "//third_party/libpng",
@@ -47,4 +46,10 @@ group("root") {
     "//ui/gfx/geometry",
     "//url",
   ]
+
+  if (is_linux) {
+    deps += [
+      "//third_party/freetype2",
+    ]
+  }
 }