Remove readability/inheritance from the lint blacklist

presubmit_canned_checks.py

 # 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.
 
 """Generic presubmit checks that can be reused by other presubmit checks."""
 
 import os as _os
 _HERE = _os.path.dirname(_os.path.abspath(__file__))
 
 # 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?
 # - whitespace/braces   : We have a lot of explicit scoping in chrome code.
-# - readability/inheritance : Temporary, while the OVERRIDE and FINAL fixup
-#                             is in progress.
 DEFAULT_LINT_FILTERS = [
   '-build/include',
   '-build/include_order',
   '-build/namespace',
   '-readability/casting',
   '-runtime/int',
   '-runtime/virtual',
   '-whitespace/braces',
-  '-readability/inheritance'
 ]
 
 ### Description checks
 
 def CheckChangeHasTestField(input_api, output_api):
   """Requires that the changelist have a TEST= field."""
   if input_api.change.TEST:
     return []
   else:
     return [output_api.PresubmitNotifyResult(
@@ skipping 1078 matching lines @@
   for f in affected_files:
     cmd = ['gn', 'format', '--dry-run', f.AbsoluteLocalPath()]
     rc = gn.main(cmd)
     if rc == 2:
       warnings.append(output_api.PresubmitPromptWarning(
           '%s requires formatting. Please run `gn format --in-place %s`.' % (
               f.AbsoluteLocalPath(), f.LocalPath())))
   # It's just a warning, so ignore other types of failures assuming they'll be
   # caught elsewhere.
   return warnings