Remove more logic out of trychange.py into scm.py.

trychange.py

 #!/usr/bin/python
 # Copyright (c) 2009 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.
 """Client-side script to send a try job to the try server. It communicates to
 the try server by either writting to a svn repository or by directly connecting
 to the server by HTTP.
 """
 
 import datetime
@@ skipping 67 matching lines @@
   def GetFileNames(self):
     """Return the list of files in the diff."""
     return self.options.files
 
 
 class SVN(SCM):
   """Gathers the options and diff for a subversion checkout."""
   def __init__(self, *args, **kwargs):
     SCM.__init__(self, *args, **kwargs)
     self.checkout_root = scm.SVN.GetCheckoutRoot(os.getcwd())
-    self.options.files
     if not self.options.diff:
       # Generate the diff from the scm.
       self.options.diff = self._GenerateDiff()
     if not self.options.email:
       # Assumes the svn credential is an email address.
       self.options.email = scm.SVN.GetEmail(self.checkout_root)
 
   def _GenerateDiff(self):
     """Returns a string containing the diff for the given file list.
 
@@ skipping 24 matching lines @@
         return gclient_utils.FileRead(os.path.join(self.checkout_root,
                                                    'PRESUBMIT.py'))
       except (IOError, OSError):
         return None
 
 
 class GIT(SCM):
   """Gathers the options and diff for a git checkout."""
   def __init__(self, *args, **kwargs):
     SCM.__init__(self, *args, **kwargs)
-    self.checkout_root = os.path.abspath(
-        gclient_utils.CheckCall(['git', 'rev-parse', '--show-cdup']).strip())
+    self.checkout_root = scm.GIT.GetCheckoutRoot(os.getcwd())
     if not self.options.diff:
-      self.options.diff = self._GenerateDiff()
+      self.options.diff = scm.GIT.GenerateDiff(self.checkout_root)
     if not self.options.name:
-      self.options.name = self._GetPatchName()
+      self.options.name = scm.GIT.GetPatchName(self.checkout_root)
     if not self.options.email:
-      self.options.email = scm.GIT.GetEmail('.')
-
-  def _GenerateDiff(self):
-    """Get the diff we'll send to the try server. We ignore the files list."""
-    return scm.GIT.GenerateDiff(self.checkout_root)
-
-  def _GetPatchName(self):
-    """Construct a name for this patch."""
-    # TODO: perhaps include the hash of the current commit, to distinguish
-    # patches?
-    branch = gclient_utils.CheckCall(['git', 'symbolic-ref', 'HEAD']).strip()
-    if not branch.startswith('refs/heads/'):
-      # TODO(maruel): Find a better type.
-      raise NoTryServerAccess("Couldn't figure out branch name")
-    branch = branch[len('refs/heads/'):]
-    return branch
+      self.options.email = scm.GIT.GetEmail(self.checkout_root)
 
   def GetLocalRoot(self):
     """Return the path of the repository root."""
     return self.checkout_root
 
   def GetBots(self):
     try:
       return gclient_utils.FileRead(os.path.join(self.checkout_root,
                                                  'PRESUBMIT.py'))
     except (IOError, OSError):
@@ skipping 327 matching lines @@
   except (InvalidScript, NoTryServerAccess), e:
     if swallow_exception:
       return 1
     print e
     return 1
   return 0
 
 
 if __name__ == "__main__":
   sys.exit(TryChange(None, [], False))