Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(430)

Side by Side Diff: PRESUBMIT.py

Issue 2047633002: Revert of Remove base/move.h (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « no previous file | base/BUILD.gn » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 # Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 # Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 # Use of this source code is governed by a BSD-style license that can be 2 # Use of this source code is governed by a BSD-style license that can be
3 # found in the LICENSE file. 3 # found in the LICENSE file.
4 4
5 """Top-level presubmit script for Chromium. 5 """Top-level presubmit script for Chromium.
6 6
7 See http://dev.chromium.org/developers/how-tos/depottools/presubmit-scripts 7 See http://dev.chromium.org/developers/how-tos/depottools/presubmit-scripts
8 for more details about the presubmit API built into depot_tools. 8 for more details about the presubmit API built into depot_tools.
9 """ 9 """
10 10
(...skipping 1833 matching lines...) Expand 10 before | Expand all | Expand 10 after
1844 results.extend(_CheckFilePermissions(input_api, output_api)) 1844 results.extend(_CheckFilePermissions(input_api, output_api))
1845 results.extend(_CheckNoAuraWindowPropertyHInHeaders(input_api, output_api)) 1845 results.extend(_CheckNoAuraWindowPropertyHInHeaders(input_api, output_api))
1846 results.extend(_CheckIncludeOrder(input_api, output_api)) 1846 results.extend(_CheckIncludeOrder(input_api, output_api))
1847 results.extend(_CheckForVersionControlConflicts(input_api, output_api)) 1847 results.extend(_CheckForVersionControlConflicts(input_api, output_api))
1848 results.extend(_CheckPatchFiles(input_api, output_api)) 1848 results.extend(_CheckPatchFiles(input_api, output_api))
1849 results.extend(_CheckHardcodedGoogleHostsInLowerLayers(input_api, output_api)) 1849 results.extend(_CheckHardcodedGoogleHostsInLowerLayers(input_api, output_api))
1850 results.extend(_CheckNoAbbreviationInPngFileName(input_api, output_api)) 1850 results.extend(_CheckNoAbbreviationInPngFileName(input_api, output_api))
1851 results.extend(_CheckForInvalidOSMacros(input_api, output_api)) 1851 results.extend(_CheckForInvalidOSMacros(input_api, output_api))
1852 results.extend(_CheckForInvalidIfDefinedMacros(input_api, output_api)) 1852 results.extend(_CheckForInvalidIfDefinedMacros(input_api, output_api))
1853 results.extend(_CheckFlakyTestUsage(input_api, output_api)) 1853 results.extend(_CheckFlakyTestUsage(input_api, output_api))
1854 # TODO(danakj): Remove this when base/move.h is removed.
1855 results.extend(_CheckForUsingPass(input_api, output_api))
1854 results.extend(_CheckAddedDepsHaveTargetApprovals(input_api, output_api)) 1856 results.extend(_CheckAddedDepsHaveTargetApprovals(input_api, output_api))
1855 results.extend( 1857 results.extend(
1856 input_api.canned_checks.CheckChangeHasNoTabs( 1858 input_api.canned_checks.CheckChangeHasNoTabs(
1857 input_api, 1859 input_api,
1858 output_api, 1860 output_api,
1859 source_file_filter=lambda x: x.LocalPath().endswith('.grd'))) 1861 source_file_filter=lambda x: x.LocalPath().endswith('.grd')))
1860 results.extend(_CheckSpamLogging(input_api, output_api)) 1862 results.extend(_CheckSpamLogging(input_api, output_api))
1861 results.extend(_CheckForAnonymousVariables(input_api, output_api)) 1863 results.extend(_CheckForAnonymousVariables(input_api, output_api))
1862 results.extend(_CheckCygwinShell(input_api, output_api)) 1864 results.extend(_CheckCygwinShell(input_api, output_api))
1863 results.extend(_CheckUserActionUpdate(input_api, output_api)) 1865 results.extend(_CheckUserActionUpdate(input_api, output_api))
(...skipping 146 matching lines...) Expand 10 before | Expand all | Expand 10 after
2010 2012
2011 if not bad_macros: 2013 if not bad_macros:
2012 return [] 2014 return []
2013 2015
2014 return [output_api.PresubmitError( 2016 return [output_api.PresubmitError(
2015 'Found ifdef check on always-defined macro[s]. Please fix your code\n' 2017 'Found ifdef check on always-defined macro[s]. Please fix your code\n'
2016 'or check the list of ALWAYS_DEFINED_MACROS in src/PRESUBMIT.py.', 2018 'or check the list of ALWAYS_DEFINED_MACROS in src/PRESUBMIT.py.',
2017 bad_macros)] 2019 bad_macros)]
2018 2020
2019 2021
2022 def _CheckForUsingPass(input_api, output_api):
2023 """Check all affected files for using side effects of Pass."""
2024 errors = []
2025 for f in input_api.AffectedFiles():
2026 if f.LocalPath().endswith(('.h', '.c', '.cc', '.m', '.mm')):
2027 for lnum, line in f.ChangedContents():
2028 # Warn on any use of foo.Pass().
2029 if input_api.re.search(r'[a-zA-Z0-9_]+\.Pass\(\)', line):
2030 errors.append(output_api.PresubmitError(
2031 ('%s:%d uses Pass(); please use std::move() instead. ' +
2032 'See crbug.com/557422.') % (f.LocalPath(), lnum)))
2033 return errors
2034
2035
2020 def _CheckForIPCRules(input_api, output_api): 2036 def _CheckForIPCRules(input_api, output_api):
2021 """Check for same IPC rules described in 2037 """Check for same IPC rules described in
2022 http://www.chromium.org/Home/chromium-security/education/security-tips-for-ipc 2038 http://www.chromium.org/Home/chromium-security/education/security-tips-for-ipc
2023 """ 2039 """
2024 base_pattern = r'IPC_ENUM_TRAITS\(' 2040 base_pattern = r'IPC_ENUM_TRAITS\('
2025 inclusion_pattern = input_api.re.compile(r'(%s)' % base_pattern) 2041 inclusion_pattern = input_api.re.compile(r'(%s)' % base_pattern)
2026 comment_pattern = input_api.re.compile(r'//.*(%s)' % base_pattern) 2042 comment_pattern = input_api.re.compile(r'//.*(%s)' % base_pattern)
2027 2043
2028 problems = [] 2044 problems = []
2029 for f in input_api.AffectedSourceFiles(None): 2045 for f in input_api.AffectedSourceFiles(None):
(...skipping 101 matching lines...) Expand 10 before | Expand all | Expand 10 after
2131 results.extend(input_api.canned_checks.CheckTreeIsOpen( 2147 results.extend(input_api.canned_checks.CheckTreeIsOpen(
2132 input_api, 2148 input_api,
2133 output_api, 2149 output_api,
2134 json_url='http://chromium-status.appspot.com/current?format=json')) 2150 json_url='http://chromium-status.appspot.com/current?format=json'))
2135 2151
2136 results.extend(input_api.canned_checks.CheckChangeHasBugField( 2152 results.extend(input_api.canned_checks.CheckChangeHasBugField(
2137 input_api, output_api)) 2153 input_api, output_api))
2138 results.extend(input_api.canned_checks.CheckChangeHasDescription( 2154 results.extend(input_api.canned_checks.CheckChangeHasDescription(
2139 input_api, output_api)) 2155 input_api, output_api))
2140 return results 2156 return results
OLDNEW
« no previous file with comments | « no previous file | base/BUILD.gn » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698