Add url/ to banned_directories in clang style checker.

tools/clang/plugins/ChromeClassTester.cpp

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 // A general interface for filtering and only acting on classes in Chromium C++
 // code.
 
 #include "ChromeClassTester.h"
 
 #include <sys/param.h>
@@ skipping 123 matching lines @@
 
   return false;
 }
 
 void ChromeClassTester::BuildBannedLists() {
   banned_namespaces_.push_back("std");
   banned_namespaces_.push_back("__gnu_cxx");
   banned_namespaces_.push_back("WebKit");
   banned_namespaces_.push_back("WebTestRunner");
 
-  banned_directories_.push_back("third_party/");
-  banned_directories_.push_back("native_client/");
   banned_directories_.push_back("breakpad/");
   banned_directories_.push_back("courgette/");
+  banned_directories_.push_back("dart/");
+  banned_directories_.push_back("frameworks/");
+  banned_directories_.push_back("googleurl/");
+  banned_directories_.push_back("icu4c/");
+  banned_directories_.push_back("native_client/");
   banned_directories_.push_back("pdf/");
   banned_directories_.push_back("ppapi/");
+  banned_directories_.push_back("sdch/");
+  banned_directories_.push_back("testing/");
+  banned_directories_.push_back("third_party/");
+  // TODO(tfarina): Remove this when ready. crbug.com/229660
+  banned_directories_.push_back("url/");

[Nico, 2013/04/16 01:51:50]

Do you want to put this behind a flag so that you can remove it without a clang
roll in the future?

   banned_directories_.push_back("usr/");
-  banned_directories_.push_back("testing/");
-  banned_directories_.push_back("googleurl/");
   banned_directories_.push_back("v8/");
-  banned_directories_.push_back("dart/");
-  banned_directories_.push_back("sdch/");
-  banned_directories_.push_back("icu4c/");
-  banned_directories_.push_back("frameworks/");
 
   // Don't check autogenerated headers.
   // Make puts them below $(builddir_name)/.../gen and geni.
   // Ninja puts them below OUTPUT_DIR/.../gen
   // Xcode has a fixed output directory for everything.
   banned_directories_.push_back("gen/");
   banned_directories_.push_back("geni/");
   banned_directories_.push_back("xcodebuild/");
 
   // You are standing in a mazy of twisty dependencies, all resolved by
@@ skipping 122 matching lines @@
   PresumedLoc ploc = source_manager.getPresumedLoc(spelling_location);
   if (ploc.isInvalid()) {
     // If we're in an invalid location, we're looking at things that aren't
     // actually stated in the source.
     return false;
   }
 
   *filename = ploc.getFilename();
   return true;
 }