commit 2bd36f5a92338c33a221e92a5d95e74dcd7d6f7c
author Edward Lesmes <ehmaldonado@google.com> Wed Jul 29 17:47:01 2020 +0000
committer Chromium commit bot <commit-bot@chromium.org> Wed Jul 29 17:47:01 2020 +0000
tree e452e77d36264e06e74e3b70878f1500a43f1ba8
parent cf4f47bdd5e363d8fafb6db7b8ba98f63c9e709f

Enable gn check for //samples:pdfium_test

gn check doesn't understand conditional includes, so `// nogncheck` is
needed.
If `pdf_enable_skia` is false, it will still complain that the headers
files live under `//skia`, which isn't included as in deps.

Change-Id: I2a6e1dd27dccf681d93b01204da15ccbc824b257
Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/71751
Reviewed-by: dsinclair <dsinclair@chromium.org>
Commit-Queue: dsinclair <dsinclair@chromium.org>

samples/pdfium_test_write_helper.h

diff --git a/samples/pdfium_test_write_helper.h b/samples/pdfium_test_write_helper.h
index 0de3418..bb4b1ac 100644
--- a/samples/pdfium_test_write_helper.h
+++ b/samples/pdfium_test_write_helper.h
@@ -10,8 +10,8 @@
 #include "public/fpdfview.h"
 
 #ifdef PDF_ENABLE_SKIA
-#include "third_party/skia/include/core/SkPictureRecorder.h"
-#include "third_party/skia/include/core/SkStream.h"
+#include "third_party/skia/include/core/SkPictureRecorder.h"  // nogncheck
+#include "third_party/skia/include/core/SkStream.h"  // nogncheck
 #endif
 
 std::string WritePpm(const char* pdf_name,