app_list: Exclude Objective-C from cpplint pass.

ui/app_list/PRESUBMIT.py

Index: ui/app_list/PRESUBMIT.py
diff --git a/ui/app_list/PRESUBMIT.py b/ui/app_list/PRESUBMIT.py
index 591fad1229fcb3f6207577c2f50d221257319237..a5b21e0b5ff305f2b5b4cb5ae0e43af8590c1090 100644
--- a/ui/app_list/PRESUBMIT.py
+++ b/ui/app_list/PRESUBMIT.py
@@ -7,14 +7,23 @@
 See http://dev.chromium.org/developers/how-tos/depottools/presubmit-scripts
 for more details about the presubmit API built into depot_tools.
 """
+
 INCLUDE_CPP_FILES_ONLY = (
   r'.*\.cc$', r'.*\.h$'
 )
 
+EXCLUDE = (
+  # Objective C confuses everything.
+  r'.*cocoa.*',
+  r'.*_mac\.(cc|h)$',

[tapted, 2014/12/12 00:12:42]

a foo_mac.cc file won't have any Objective C in it - they should still get
checked. (I know chrome/PRESUBMIT.py has this, but it could be improved).

For app list I think we just need to ignore the cocoa folder:

EXCLUDE = (
  r'.*/cocoa/.*'
)

Everything in there is objective C and there's nothing that isn't.

Approach otherwise looks nice.

[tfarina, 2014/12/12 00:21:28]

Done.

+  r'.*_mac_.*',
+)
+
 def CheckChangeLintsClean(input_api, output_api):
   """Makes sure that the ui/app_list/ code is cpplint clean."""
+  black_list = input_api.DEFAULT_BLACK_LIST + EXCLUDE
   sources = lambda x: input_api.FilterSourceFile(
-    x, white_list = INCLUDE_CPP_FILES_ONLY, black_list = None)
+    x, white_list = INCLUDE_CPP_FILES_ONLY, black_list = black_list)
   return input_api.canned_checks.CheckChangeLintsClean(
       input_api, output_api, sources)