Home | History | Annotate | Download | only in app
      1 # Copyright 2013 The Chromium Authors. All rights reserved.
      2 # Use of this source code is governed by a BSD-style license that can be
      3 # found in the LICENSE file.
      4 
      5 """Presubmit script for changes affecting chrome/app/
      6 
      7 See http://dev.chromium.org/developers/how-tos/depottools/presubmit-scripts
      8 for more details about the presubmit API built into gcl.
      9 """
     10 
     11 import os
     12 
     13 def _CheckNoProductNameInGeneratedResources(input_api, output_api):
     14   """Check that no PRODUCT_NAME placeholders are found in resources files.
     15 
     16   These kinds of strings prevent proper localization in some languages. For
     17   more information, see the following chromium-dev thread:
     18   https://groups.google.com/a/chromium.org/forum/#!msg/chromium-dev/PBs5JfR0Aoc/NOcIHII9u14J
     19   """
     20 
     21   problems = []
     22   filename_filter = lambda x: x.LocalPath().endswith('.grd')
     23 
     24   for f, line_num, line in input_api.RightHandSideLines(filename_filter):
     25     if 'PRODUCT_NAME' in line:
     26       problems.append('%s:%d' % (f.LocalPath(), line_num))
     27 
     28   if problems:
     29     return [output_api.PresubmitPromptWarning(
     30         "Don't use PRODUCT_NAME placeholders in string resources. Instead, add "
     31         "separate strings to google_chrome_strings.grd and "
     32         "chromium_strings.grd. See http://goo.gl/6614MQ for more information."
     33         "Problems with this check? Contact dubroy (at] chromium.org.",
     34         items=problems)]
     35   return []
     36 
     37 def _CommonChecks(input_api, output_api):
     38   """Checks common to both upload and commit."""
     39   results = []
     40   results.extend(_CheckNoProductNameInGeneratedResources(input_api, output_api))
     41   return results
     42 
     43 def CheckChangeOnUpload(input_api, output_api):
     44   return _CommonChecks(input_api, output_api)
     45 
     46 def CheckChangeOnCommit(input_api, output_api):
     47   return _CommonChecks(input_api, output_api)
     48