Enforced new rules for braces in conditional and loop bodies in style checker.

PRESUBMIT.py

 # Copyright (c) 2013 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.
 
 """Top-level presubmit script for Blink.
 
 See http://dev.chromium.org/developers/how-tos/depottools/presubmit-scripts
 for more details about the presubmit API built into gcl.
 """
 
@@ skipping 31 matching lines @@
     license_header = r'.*'
 
     results = []
     results.extend(input_api.canned_checks.PanProjectChecks(
         input_api, output_api, excluded_paths=_EXCLUDED_PATHS,
         maxlen=800, license_header=license_header))
     results.extend(_CheckForVersionControlConflicts(input_api, output_api))
     results.extend(_CheckPatchFiles(input_api, output_api))
     results.extend(_CheckTestExpectations(input_api, output_api))
     results.extend(_CheckUnwantedDependencies(input_api, output_api))
+    results.extend(_CheckStyle(input_api, output_api))

[abarth-chromium, 2013/05/24 06:06:39]

This seems like a separate issue from the rest of this CL.  I'm not sure we want
to enforce style on commit at this point.  Enforcing it on upload seems to be
working well given that not all our code passes the style checker at the moment.

     return results
 
 
 def _CheckSubversionConfig(input_api, output_api):
   """Verifies the subversion config file is correctly setup.
 
   Checks that autoprops are enabled, returns an error otherwise.
   """
   join = input_api.os_path.join
   if input_api.platform == 'win32':
@@ skipping 132 matching lines @@
                 'You added one or more #includes of files that are temporarily\n'
                 'allowed but being removed. Can you avoid introducing the\n'
                 '#include? See relevant DEPS file(s) for details and contacts.',
                 warning_descriptions))
     return results
 
 
 def CheckChangeOnUpload(input_api, output_api):
     results = []
     results.extend(_CommonChecks(input_api, output_api))
-    results.extend(_CheckStyle(input_api, output_api))
     return results
 
 
 def CheckChangeOnCommit(input_api, output_api):
     results = []
     results.extend(_CommonChecks(input_api, output_api))
     results.extend(input_api.canned_checks.CheckTreeIsOpen(
         input_api, output_api,
         json_url='http://blink-status.appspot.com/current?format=json'))
     results.extend(input_api.canned_checks.CheckChangeHasDescription(
         input_api, output_api))
     results.extend(_CheckSubversionConfig(input_api, output_api))
     return results
 
 def GetPreferredTrySlaves(project, change):
     return ['linux_layout_rel', 'mac_layout_rel', 'win_layout_rel']