[WPT Export] Fix cwd on run_command calls in ChromiumCommit

third_party/WebKit/Tools/Scripts/webkitpy/w3c/chromium_commit.py

 # Copyright 2016 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 from webkitpy.common.memoized import memoized
 from webkitpy.common.webkit_finder import WebKitFinder
 
 CHROMIUM_WPT_DIR = 'third_party/WebKit/LayoutTests/external/wpt/'
 
 
@@ skipping 13 matching lines @@
 
         assert sha or position, 'requires sha or position'
         assert not (sha and position), 'cannot accept both sha and position'
 
         if position and not sha:
             if position.startswith('Cr-Commit-Position: '):
                 position = position[len('Cr-Commit-Position: '):]
 
             sha = self.position_to_sha(position)
 
+        assert len(sha) == 40, 'Expected SHA-1 hash, got {}'.format(sha)
         self.sha = sha
         self.position = position
 
     def num_behind_master(self):
         """Returns the number of commits this commit is behind origin/master.
         It is inclusive of this commit and of the latest commit.
         """
         return len(self.host.executive.run_command([
             'git', 'rev-list', '{}..origin/master'.format(self.sha)
-        ]).splitlines())
+        ], cwd=self.absolute_chromium_dir()).splitlines())
 
     def position_to_sha(self, commit_position):
         return self.host.executive.run_command([
             'git', 'crrev-parse', commit_position
-        ]).strip()
+        ], cwd=self.absolute_chromium_dir()).strip()
 
     def subject(self):
         return self.host.executive.run_command([
             'git', 'show', '--format=%s', '--no-patch', self.sha
-        ])
+        ], cwd=self.absolute_chromium_dir())
 
     def body(self):
         return self.host.executive.run_command([
             'git', 'show', '--format=%b', '--no-patch', self.sha
-        ])
+        ], cwd=self.absolute_chromium_dir())
 
     def author(self):
         return self.host.executive.run_command([
             'git', 'show', '--format="%aN <%aE>"', '--no-patch', self.sha
-        ])
+        ], cwd=self.absolute_chromium_dir())
 
     def message(self):
         """Returns a string with a commit's subject and body."""
         return self.host.executive.run_command([
             'git', 'show', '--format=%B', '--no-patch', self.sha
-        ])
+        ], cwd=self.absolute_chromium_dir())
 
     def filtered_changed_files(self):
         """Makes a patch with just changes in files in the WPT dir for a given commit."""
         changed_files = self.host.executive.run_command([
             'git', 'diff-tree', '--name-only', '--no-commit-id', '-r', self.sha,
             '--', self.absolute_chromium_wpt_dir()
-        ]).splitlines()
+        ], cwd=self.absolute_chromium_dir()).splitlines()
 
         blacklist = [
             'MANIFEST.json',
             self.host.filesystem.join('resources', 'testharnessreport.js'),
         ]
         qualified_blacklist = [CHROMIUM_WPT_DIR + f for f in blacklist]
 
         return [f for f in changed_files if f not in qualified_blacklist and not self.is_baseline(f)]
 
     @staticmethod
@@ skipping 14 matching lines @@
 
     @memoized
     def absolute_chromium_wpt_dir(self):
         finder = WebKitFinder(self.host.filesystem)
         return finder.path_from_webkit_base('LayoutTests', 'external', 'wpt')
 
     @memoized
     def absolute_chromium_dir(self):
         finder = WebKitFinder(self.host.filesystem)
         return finder.chromium_base()