[email protected] | 2fac375 | 2011-11-27 20:56:51 | [diff] [blame] | 1 | # Copyright (c) 2011 The Chromium Authors. All rights reserved. |
[email protected] | 93b2d737 | 2009-11-17 18:54:53 | [diff] [blame] | 2 | # Use of this source code is governed by a BSD-style license that can be |
| 3 | # found in the LICENSE file. |
| 4 | |
[email protected] | 5a011450 | 2011-11-29 13:01:24 | [diff] [blame] | 5 | """Presubmit script for changes affecting chrome/ |
| 6 | |
| 7 | See http://dev.chromium.org/developers/how-tos/depottools/presubmit-scripts |
tfarina | 78bb92f4 | 2015-01-31 00:20:48 | [diff] [blame^] | 8 | for more details about the presubmit API built into depot_tools. |
[email protected] | 5a011450 | 2011-11-29 13:01:24 | [diff] [blame] | 9 | """ |
| 10 | |
| 11 | import re |
[email protected] | 93b2d737 | 2009-11-17 18:54:53 | [diff] [blame] | 12 | |
| 13 | INCLUDE_CPP_FILES_ONLY = ( |
| 14 | r'.*\.cc$', r'.*\.h$' |
| 15 | ) |
| 16 | |
| 17 | EXCLUDE = ( |
| 18 | # Objective C confuses everything. |
| 19 | r'.*cocoa.*', |
| 20 | r'.*_mac\.(cc|h)$', |
| 21 | r'.*_mac_.*', |
| 22 | # All the messages files do weird multiple include trickery |
[email protected] | 3139e9f | 2013-10-10 21:29:47 | [diff] [blame] | 23 | r'.*_messages.*\.h$', |
[email protected] | 93b2d737 | 2009-11-17 18:54:53 | [diff] [blame] | 24 | r'render_messages.h$', |
| 25 | # Autogenerated window resources files are off limits |
| 26 | r'.*resource.h$', |
[email protected] | 93b2d737 | 2009-11-17 18:54:53 | [diff] [blame] | 27 | # Header trickery |
| 28 | r'.*-inl\.h$', |
[email protected] | 93b2d737 | 2009-11-17 18:54:53 | [diff] [blame] | 29 | # Has safe printf usage that cpplint complains about |
| 30 | r'safe_browsing_util\.cc$', |
[email protected] | 93b2d737 | 2009-11-17 18:54:53 | [diff] [blame] | 31 | ) |
| 32 | |
[email protected] | 5a011450 | 2011-11-29 13:01:24 | [diff] [blame] | 33 | def _CheckChangeLintsClean(input_api, output_api): |
| 34 | """Makes sure that the chrome/ code is cpplint clean.""" |
[email protected] | 93b2d737 | 2009-11-17 18:54:53 | [diff] [blame] | 35 | black_list = input_api.DEFAULT_BLACK_LIST + EXCLUDE |
| 36 | sources = lambda x: input_api.FilterSourceFile( |
| 37 | x, white_list=INCLUDE_CPP_FILES_ONLY, black_list=black_list) |
[email protected] | 5a011450 | 2011-11-29 13:01:24 | [diff] [blame] | 38 | return input_api.canned_checks.CheckChangeLintsClean( |
| 39 | input_api, output_api, sources) |
| 40 | |
| 41 | def _CheckNoContentUnitTestsInChrome(input_api, output_api): |
| 42 | """Makes sure that no unit tests from content/ are included in unit_tests.""" |
| 43 | problems = [] |
| 44 | for f in input_api.AffectedFiles(): |
| 45 | if not f.LocalPath().endswith('chrome_tests.gypi'): |
| 46 | continue |
| 47 | |
| 48 | for line_num, line in f.ChangedContents(): |
| 49 | m = re.search(r"'(.*\/content\/.*unittest.*)'", line) |
| 50 | if m: |
| 51 | problems.append(m.group(1)) |
| 52 | |
| 53 | if not problems: |
| 54 | return [] |
| 55 | return [output_api.PresubmitPromptWarning( |
| 56 | 'Unit tests located in content/ should be added to the ' + |
| 57 | 'content_tests.gypi:content_unittests target.', |
| 58 | items=problems)] |
| 59 | |
| 60 | def _CommonChecks(input_api, output_api): |
| 61 | """Checks common to both upload and commit.""" |
| 62 | results = [] |
| 63 | results.extend(_CheckNoContentUnitTestsInChrome(input_api, output_api)) |
| 64 | return results |
| 65 | |
| 66 | def CheckChangeOnUpload(input_api, output_api): |
| 67 | results = [] |
| 68 | results.extend(_CommonChecks(input_api, output_api)) |
| 69 | results.extend(_CheckChangeLintsClean(input_api, output_api)) |
| 70 | return results |
| 71 | |
| 72 | def CheckChangeOnCommit(input_api, output_api): |
| 73 | results = [] |
| 74 | results.extend(_CommonChecks(input_api, output_api)) |
[email protected] | 93b2d737 | 2009-11-17 18:54:53 | [diff] [blame] | 75 | return results |