Require folks to run `git cl format` on ui/webui/resources

ui/webui/resources/PRESUBMIT.py

 # Copyright 2016 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.
 
 import re
 
 
 def PostUploadHook(cl, change, output_api):
   rietveld_obj = cl.RpcServer()
   description = rietveld_obj.get_description(cl.issue)
@@ skipping 12 matching lines @@
 
   if new_description == description:
     return []
 
   rietveld_obj.update_description(cl.issue, new_description)
   return [output_api.PresubmitNotifyResult(
       'Automatically added optional Closure bots to run on CQ.')]
 
 
 def CheckChangeOnUpload(input_api, output_api):
-  return _CheckForTranslations(input_api, output_api)
+  return _CommonChecks(input_api, output_api)
 
 
 def CheckChangeOnCommit(input_api, output_api):
-  return _CheckForTranslations(input_api, output_api)
+  return _CommonChecks(input_api, output_api)
 
 
 def _CheckForTranslations(input_api, output_api):
   shared_keywords = ['i18n(']
   html_keywords = shared_keywords + ['$118n{']
   js_keywords = shared_keywords + ['I18nBehavior', 'loadTimeData.']
 
   errors = []
 
   for f in input_api.AffectedFiles():
@@ skipping 13 matching lines @@
         errors.append("%s:%d\n%s" % (f.LocalPath(), lnum, line))
 
   if not errors:
     return []
 
   return [output_api.PresubmitError("\n".join(errors) + """
 
 Don't embed translations directly in shared UI code. Instead, inject your
 translation from the place using the shared code. For an example: see
 <cr-dialog>#closeText (http://bit.ly/2eLEsqh).""")]
+
+
+def _CommonChecks(input_api, output_api):
+  results = []
+  results += _CheckForTranslations(input_api, output_api)
+  results += input_api.canned_checks.CheckPatchFormatted(input_api, output_api)

[dpapad, 2016/12/22 00:59:13]

Can you ensure that this does not create a problem if your patch includes a
vulcanized/crispified file? Or if it does, is there an easy way to bypass?

I guess this is done only for files under ui/webui/resources (even if the CL has
other files too), but it is worth double checking.

[Dan Beam, 2016/12/22 00:59:54]

right, there are no crisper files in ui/webui/resources

+  return results