Clang plugin: Support for Blink in web/.

tools/clang/plugins/ChromeClassTester.cpp

Index: tools/clang/plugins/ChromeClassTester.cpp
diff --git a/tools/clang/plugins/ChromeClassTester.cpp b/tools/clang/plugins/ChromeClassTester.cpp
index c9388067234f44a1985587dd660edfd1417b1e1c..e41c92a3929245e6b5f532314099f2d3316e7749 100644
--- a/tools/clang/plugins/ChromeClassTester.cpp
+++ b/tools/clang/plugins/ChromeClassTester.cpp
@@ -157,6 +157,11 @@ ChromeClassTester::LocationType ChromeClassTester::ClassifyLocation(
   std::replace(filename.begin(), filename.end(), '\\', '/');
 #endif
 
+  if (options_.check_for_blink)

[dcheng, 2017/04/17 02:40:57]

Hmm, so the potential issue with this is that Blink files include other
third_party code--and we want those SourceLocations to still be treated as
kThirdParty. Otherwise, this could cause the style checker plugin to warn on
third-party code if a Blink file includes a header from third_party.

I've been thinking about it more, and I have two main ideas to fix this:
- Use a namespace check to determine if something should be Blink or not. e.g.
namespace blink == Blink, otherwise something else
- Specify "web/public", "web/Source" as the blink directory paths.

I kind of prefer using "namespace blink" as the main check if possible, though
that's a bit clunky to implement atm. I think once
https://chromium-review.googlesource.com/c/478907/ land, this should be pretty
easy to do though. WDYT?

[tkent, 2017/04/17 08:00:23]

Ah, I see.
Patch Set 5 implemented this.
We'd like to omit "Source/". So, this would be "web/public, web/bindings,
web/core, web/devtools, web/modules, web/platform, web/web, or web/wtf".  It
wouldn't be robust for Blink-internal structure changes.  I'd like to assume
it's the last fallback option.

+    return LocationType::kBlink;
+  // TODO(tkent): Remove blink_directories_. We specify "-Xclang
+  // -plugin-arg-find-bad-constructs -Xclang check-for-blink" in BUILD.gn
+  // instead.
   for (const std::string& blink_dir : blink_directories_) {
     // If any of the allowed directories occur as a component in filename,
     // this file is allowed.