Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(3)

Unified Diff: tools/clang/plugins/ChromeClassTester.cpp

Issue 24020005: Teach Clang plugin about blink namespace (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Add TODO Created 7 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/clang/plugins/ChromeClassTester.cpp
diff --git a/tools/clang/plugins/ChromeClassTester.cpp b/tools/clang/plugins/ChromeClassTester.cpp
index ee8452dbc1e926fc587998c932c73d4f4c57bdff..e44e7da7e50b0e2b527ff1967e713c31ad3582e8 100644
--- a/tools/clang/plugins/ChromeClassTester.cpp
+++ b/tools/clang/plugins/ChromeClassTester.cpp
@@ -140,9 +140,13 @@ bool ChromeClassTester::InImplementationFile(SourceLocation record_location) {
void ChromeClassTester::BuildBannedLists() {
banned_namespaces_.push_back("std");
banned_namespaces_.push_back("__gnu_cxx");
- banned_namespaces_.push_back("WebKit");
banned_namespaces_.push_back("WebTestRunner");
+ // We're in the process of renaming WebKit to blink.
+ // TODO(abarth): Remove WebKit once the rename is complete.
+ banned_namespaces_.push_back("WebKit");
+ banned_namespaces_.push_back("blink");
+
banned_directories_.push_back("third_party/");
banned_directories_.push_back("native_client/");
banned_directories_.push_back("breakpad/");
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698