style plugin: Don't suppress all warnings for checkouts in /usr/local/google

BUG=none
TEST=The plugin tests pass on my linux box.

Review URL: http://codereview.chromium.org/8369015

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

1 files changed, 11 insertions, 0 deletions

diff --git a/tools/clang/plugins/ChromeClassTester.cpp b/tools/clang/plugins/ChromeClassTester.cpp
index f7b9e02..d1fd4f9a 100644
--- a/tools/clang/plugins/ChromeClassTester.cpp
+++ b/tools/clang/plugins/ChromeClassTester.cpp
@@ -19,6 +19,12 @@ bool starts_with(const std::string& one, const std::string& two) {
   return one.compare(0, two.size(), two) == 0;
 }
 
+std::string lstrip(const std::string& one, const std::string& two) {
+  if (starts_with(one, two))
+    return one.substr(two.size());
+  return one;
+}
+
 bool ends_with(const std::string& one, const std::string& two) {
   if (two.size() > one.size())
     return false;
@@ -224,6 +230,11 @@ bool ChromeClassTester::InBannedDirectory(SourceLocation loc) {
       b = resolvedPath;
     }
 
+    // On linux, chrome is often checked out to /usr/local/google. Due to the
+    // "usr" rule in banned_directories_, all diagnostics would be suppressed
+    // in that case. As a workaround, strip that prefix.
+    b = lstrip(b, "/usr/local/google");
+
     for (std::vector<std::string>::const_iterator it =
              banned_directories_.begin();
          it != banned_directories_.end(); ++it) {