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

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

Issue 2616403004: [WPT Export] Add proper author to exported commits (Closed)
Patch Set: [WPT Export] Add proper author to exported commits Created 3 years, 11 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 | third_party/WebKit/Tools/Scripts/webkitpy/w3c/local_wpt.py » ('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 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 from webkitpy.common.memoized import memoized 5 from webkitpy.common.memoized import memoized
6 from webkitpy.common.webkit_finder import WebKitFinder 6 from webkitpy.common.webkit_finder import WebKitFinder
7 from webkitpy.w3c.deps_updater import DepsUpdater 7 from webkitpy.w3c.deps_updater import DepsUpdater
8 8
9 CHROMIUM_WPT_DIR = 'third_party/WebKit/LayoutTests/imported/wpt/' 9 CHROMIUM_WPT_DIR = 'third_party/WebKit/LayoutTests/imported/wpt/'
10 10
(...skipping 40 matching lines...) Expand 10 before | Expand all | Expand 10 after
51 def subject(self): 51 def subject(self):
52 return self.host.executive.run_command([ 52 return self.host.executive.run_command([
53 'git', 'show', '--format=%s', '--no-patch', self.sha 53 'git', 'show', '--format=%s', '--no-patch', self.sha
54 ]) 54 ])
55 55
56 def body(self): 56 def body(self):
57 return self.host.executive.run_command([ 57 return self.host.executive.run_command([
58 'git', 'show', '--format=%b', '--no-patch', self.sha 58 'git', 'show', '--format=%b', '--no-patch', self.sha
59 ]) 59 ])
60 60
61 def author(self):
62 return self.host.executive.run_command([
63 'git', 'show', '--format="%aN <%aE>"', '--no-patch', self.sha
64 ])
65
61 def message(self): 66 def message(self):
62 """Returns a string with a commit's subject and body.""" 67 """Returns a string with a commit's subject and body."""
63 return self.host.executive.run_command([ 68 return self.host.executive.run_command([
64 'git', 'show', '--format=%B', '--no-patch', self.sha 69 'git', 'show', '--format=%B', '--no-patch', self.sha
65 ]) 70 ])
66 71
67 def filtered_changed_files(self): 72 def filtered_changed_files(self):
68 """Makes a patch with just changes in files in the WPT dir for a given c ommit.""" 73 """Makes a patch with just changes in files in the WPT dir for a given c ommit."""
69 changed_files = self.host.executive.run_command([ 74 changed_files = self.host.executive.run_command([
70 'git', 'diff-tree', '--name-only', '--no-commit-id', '-r', self.sha, 75 'git', 'diff-tree', '--name-only', '--no-commit-id', '-r', self.sha,
(...skipping 20 matching lines...) Expand all
91 96
92 @memoized 97 @memoized
93 def absolute_chromium_wpt_dir(self): 98 def absolute_chromium_wpt_dir(self):
94 finder = WebKitFinder(self.host.filesystem) 99 finder = WebKitFinder(self.host.filesystem)
95 return finder.path_from_webkit_base('LayoutTests', 'imported', 'wpt') 100 return finder.path_from_webkit_base('LayoutTests', 'imported', 'wpt')
96 101
97 @memoized 102 @memoized
98 def absolute_chromium_dir(self): 103 def absolute_chromium_dir(self):
99 finder = WebKitFinder(self.host.filesystem) 104 finder = WebKitFinder(self.host.filesystem)
100 return finder.chromium_base() 105 return finder.chromium_base()
OLDNEW
« no previous file with comments | « no previous file | third_party/WebKit/Tools/Scripts/webkitpy/w3c/local_wpt.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698