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

Side by Side Diff: third_party/WebKit/Tools/Scripts/webkitpy/w3c/deps_updater.py

Issue 2512563002: Add WPTEXPORT footer automatically in DepsUpdater and imported/PRESUBMIT (Closed)
Patch Set: Created 4 years, 1 month 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 2014 The Chromium Authors. All rights reserved. 1 # Copyright 2014 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 """Fetches a copy of the latest state of a W3C test repository and commits. 5 """Fetches a copy of the latest state of a W3C test repository and commits.
6 6
7 If this script is given the argument --auto-update, it will also attempt to 7 If this script is given the argument --auto-update, it will also attempt to
8 upload a CL, triggery try jobs, and make any changes that are required for 8 upload a CL, triggery try jobs, and make any changes that are required for
9 new failing tests before committing. 9 new failing tests before committing.
10 """ 10 """
(...skipping 324 matching lines...) Expand 10 before | Expand all | Expand 10 after
335 '--rietveld', 335 '--rietveld',
336 '-m', 336 '-m',
337 description, 337 description,
338 ] + ['--cc=' + email for email in cc_list]) 338 ] + ['--cc=' + email for email in cc_list])
339 339
340 def _cl_description(self): 340 def _cl_description(self):
341 description = self.check_run(['git', 'log', '-1', '--format=%B']) 341 description = self.check_run(['git', 'log', '-1', '--format=%B'])
342 build_link = self._build_link() 342 build_link = self._build_link()
343 if build_link: 343 if build_link:
344 description += 'Build: %s\n\n' % build_link 344 description += 'Build: %s\n\n' % build_link
345 description += 'TBR=qyearsley@chromium.org' 345 description += ('TBR=qyearsley@chromium.org\n'
346 'WPTEXPORT=false')
346 return description 347 return description
347 348
348 def _build_link(self): 349 def _build_link(self):
349 """Returns a link to a job, if running on buildbot.""" 350 """Returns a link to a job, if running on buildbot."""
350 master_name = self.host.environ.get('BUILDBOT_MASTERNAME') 351 master_name = self.host.environ.get('BUILDBOT_MASTERNAME')
351 builder_name = self.host.environ.get('BUILDBOT_BUILDERNAME') 352 builder_name = self.host.environ.get('BUILDBOT_BUILDERNAME')
352 build_number = self.host.environ.get('BUILDBOT_BUILDNUMBER') 353 build_number = self.host.environ.get('BUILDBOT_BUILDNUMBER')
353 if not (master_name and builder_name and build_number): 354 if not (master_name and builder_name and build_number):
354 return None 355 return None
355 return 'https://build.chromium.org/p/%s/builders/%s/builds/%s' % (master _name, builder_name, build_number) 356 return 'https://build.chromium.org/p/%s/builders/%s/builds/%s' % (master _name, builder_name, build_number)
(...skipping 91 matching lines...) Expand 10 before | Expand all | Expand 10 after
447 """Returns a dict mapping source to dest name for layout tests that have been renamed.""" 448 """Returns a dict mapping source to dest name for layout tests that have been renamed."""
448 out = self.check_run(['git', 'diff', 'origin/master', '-M100%', '--diff- filter=R', '--name-status']) 449 out = self.check_run(['git', 'diff', 'origin/master', '-M100%', '--diff- filter=R', '--name-status'])
449 renamed_tests = {} 450 renamed_tests = {}
450 for line in out.splitlines(): 451 for line in out.splitlines():
451 _, source_path, dest_path = line.split() 452 _, source_path, dest_path = line.split()
452 source_test = self.finder.layout_test_name(source_path) 453 source_test = self.finder.layout_test_name(source_path)
453 dest_test = self.finder.layout_test_name(dest_path) 454 dest_test = self.finder.layout_test_name(dest_path)
454 if source_test and dest_test: 455 if source_test and dest_test:
455 renamed_tests[source_test] = dest_test 456 renamed_tests[source_test] = dest_test
456 return renamed_tests 457 return renamed_tests
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698