Disable the remaining pylint warnings locally and enable pylint warnings in PRESUBMIT.

presubmit_canned_checks.py

 # Copyright (c) 2010 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.
 
 """Generic presubmit checks that can be reused by other presubmit checks."""
 
 ### Description checks
 
 def CheckChangeHasTestField(input_api, output_api):
   """Requires that the changelist have a TEST= field."""
@@ skipping 63 matching lines @@
   return []
 
 
 def CheckChangeLintsClean(input_api, output_api, source_file_filter=None):
   """Checks that all '.cc' and '.h' files pass cpplint.py."""
   _RE_IS_TEST = input_api.re.compile(r'.*tests?.(cc|h)$')
   result = []
 
   # Initialize cpplint.
   import cpplint
+  # Access to a protected member _XX of a client class
+  # pylint: disable=W0212
   cpplint._cpplint_state.ResetErrorCounts()
 
   # Justifications for each filter:
   #
   # - build/include       : Too many; fix in the future.
   # - build/include_order : Not happening; #ifdefed includes.
   # - build/namespace     : I'm surprised by how often we violate this rule.
   # - readability/casting : Mistakes a whole bunch of function pointer.
   # - runtime/int         : Can be fixed long term; volume of errors too high
   # - runtime/virtual     : Broken now, but can be fixed in the future?
@@ skipping 430 matching lines @@
     pending_builds_len = len(builder.get('pending_builds', []))
     if pending_builds_len > max_pendings:
       out.append('%s has %d build(s) pending' %
                   (builder_name, pending_builds_len))
   if out:
     return [output_api.PresubmitPromptWarning(
         'Build(s) pending. It is suggested to wait that no more than %d '
             'builds are pending.' % max_pendings,
         long_text='\n'.join(out))]
   return []