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

Side by Side Diff: extensions/common/api/externs_checker.py

Issue 1761663002: [Closure Externs] Add more APIs to the presubmit warning list (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Test fixes Created 4 years, 9 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
OLDNEW
1 # Copyright 2016 The Chromium Authors. All rights reserved. 1 # Copyright 2016 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 class ExternsChecker(object): 5 class ExternsChecker(object):
6 _UPDATE_MESSAGE = """To update the externs, run: 6 _UPDATE_MESSAGE = """To update the externs, run:
7 src/ $ python tools/json_schema_compiler/compiler.py\ 7 src/ $ python tools/json_schema_compiler/compiler.py\
8 %s --root=. --generator=externs > %s""" 8 %s --root=. --generator=externs > %s"""
9 9
10 def __init__(self, input_api, output_api, api_pairs): 10 def __init__(self, input_api, output_api, api_pairs):
11 self._input_api = input_api 11 self._input_api = input_api
12 self._output_api = output_api 12 self._output_api = output_api
13 self._api_pairs = api_pairs 13 self._api_pairs = api_pairs
14 14
15 for path in api_pairs.keys() + api_pairs.values():
16 if not input_api.os_path.exists(path):
17 raise OSError('Path Not Found: %s' % path)
18
15 def RunChecks(self): 19 def RunChecks(self):
16 bad_files = [] 20 bad_files = []
17 affected = [f.AbsoluteLocalPath() for f in self._input_api.AffectedFiles()] 21 affected = [f.AbsoluteLocalPath() for f in self._input_api.AffectedFiles()]
18 for path in affected: 22 for path in affected:
19 pair = self._api_pairs.get(path) 23 pair = self._api_pairs.get(path)
20 if pair != None and pair not in affected: 24 if pair != None and pair not in affected:
21 bad_files.append({'source': path, 'extern': pair}) 25 bad_files.append({'source': path, 'extern': pair})
22 results = [] 26 results = []
23 if bad_files: 27 if bad_files:
24 replacements = (('<source_file>', '<output_file>') if len(bad_files) > 1 28 replacements = (('<source_file>', '<output_file>') if len(bad_files) > 1
25 else (bad_files[0]['source'], bad_files[0]['extern'])) 29 else (bad_files[0]['source'], bad_files[0]['extern']))
26 long_text = self._UPDATE_MESSAGE % replacements 30 long_text = self._UPDATE_MESSAGE % replacements
27 results.append(self._output_api.PresubmitPromptWarning( 31 results.append(self._output_api.PresubmitPromptWarning(
28 str('Found updated extension api files without updated extern files. ' 32 str('Found updated extension api files without updated extern files. '
29 'Please update the extern files.'), 33 'Please update the extern files.'),
30 [f['source'] for f in bad_files], 34 [f['source'] for f in bad_files],
31 long_text)) 35 long_text))
32 return results 36 return results
OLDNEW
« no previous file with comments | « chrome/common/extensions/api/PRESUBMIT.py ('k') | extensions/common/api/externs_checker_test.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698