Make move_source_file.py work in blink again.

tools/git/move_source_file.py

Index: tools/git/move_source_file.py
diff --git a/tools/git/move_source_file.py b/tools/git/move_source_file.py
index f1dde3b3e11ff39023dba95e6b9b6ec50b9fd889..4df87322335ccbccd0a4fa8659d4aa0bcc713bbc 100755
--- a/tools/git/move_source_file.py
+++ b/tools/git/move_source_file.py
@@ -8,8 +8,7 @@ to them, and re-ordering headers as needed.  If multiple source files are
 specified, the destination must be a directory.  Updates include guards in
 moved header files.  Assumes Chromium coding style.
 
-Attempts to update paths used in .gyp(i) files, but does not reorder
-or restructure .gyp(i) files in any way.
+Attempts to update and reorder paths used in .gyp(i) files.
 
 Updates full-path references to files in // comments in source files.
 
@@ -213,8 +212,10 @@ def UpdateIncludeGuard(old_path, new_path):
     f.write(new_contents)
 
 def main():
-  if not os.path.isdir('.git'):
-    print 'Fatal: You must run from the root of a git checkout.'
+  # git rev-parse returns 0 even when run in the .git directory. We don't want

[satorux1, 2016/01/21 05:52:07]

how about commenting that we use 'git rev-parse' to check if the script is run
inside of a git checkout.

[dgrogan, 2016/01/21 19:50:53]

Done.

+  # people running this in the .git directory.
+  if not os.system('git rev-parse') == 0 or os.getcwd().endswith(".git"):

[satorux1, 2016/01/21 05:52:07]

This probably matches "foo/bar.git".


how about os.path.basename(os.getcwd()) == '.git' ?

[dgrogan, 2016/01/21 19:50:53]

Good point. Fixed.

+    print 'Fatal: You must run in a git checkout.'
     return 1
 
   in_blink = os.getcwd().endswith("third_party/WebKit")

[satorux1, 2016/01/21 05:52:07]

'/' may not match on Windows? while you are at it, could you make it friendly
for Windows? something like below would work:

  cwd = os.getcwd()
  parent = os.path.dirname(cwd)
  os.path.basename(parent) == 'third_party' and
  os.path.basename(cwd) == 'WebKit'

[dgrogan, 2016/01/21 19:50:53]

Done.