Make upload.py use --no-renames. This generates more SVN-like patches,

Make upload.py use --no-renames. This generates more SVN-like patches, 
ones that are broken down into add+delete operations.  The result will 
be reviews that are more consistent with what gcl produces on svn.

Also, fix the 'git show' command for renames (renames shouldn't happen 
with --no-renames) where there is no corresponding edit in the diff.  
Previously this would crash.

[nickcarter, 2009-09-25 22:58:04]

To see what renames look like in Rietveld before and after this change, see the
various patch sets of the test issue: http://codereview.chromium.org/246010/show

[nickcarter, 2009-09-25 23:01:21]

Actually, I realized that --no-renames means that we lose history, particularly
in the edit case.  That's actually pretty valuable.

So the solution will be something more like adding "D" deletion records for the
old filenames in 'renames' to the svn diff.  I'll code that up.

[evan, 2009-09-25 23:19:49]

Since this file is in Rietveld, you'll need to make the patch against Rietveld
(static/upload.py in their tree).  I can review+commit your patches on that.