Accept svn:mime-type as a valid svn property.

rietveld.py

 # Copyright (c) 2011 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.
 """Defines class Rietveld to easily access a rietveld instance.
 
 Security implications:
 
 The following hypothesis are made:
 - Rietveld enforces:
   - Nobody else than issue owner can upload a patch set
@@ skipping 180 matching lines @@
       raise patch.UnsupportedPatchFormat(
           filename, 'Failed to parse svn properties.')
 
     while rietveld_svn_props:
       # Something like 'Added: svn:eol-style'. Note the action is localized.
       # *sigh*.
       action = rietveld_svn_props.pop(0)
       match = re.match(r'^(\w+): (.+)$', action)
       if not match or not rietveld_svn_props:
         raise patch.UnsupportedPatchFormat(
-            filename, 'Failed to parse svn properties.')
+            filename,
+            'Failed to parse svn properties: %s, %s' % (action, svn_props))
 
       if match.group(2) == 'svn:mergeinfo':
         # Silently ignore the content.
         rietveld_svn_props.pop(0)
         continue
 
       if match.group(1) not in ('Added', 'Modified'):
         # Will fail for our French friends.
         raise patch.UnsupportedPatchFormat(
             filename, 'Unsupported svn property operation.')
 
-      if match.group(2) in ('svn:eol-style', 'svn:executable'):
+      if match.group(2) in ('svn:eol-style', 'svn:executable', 'svn:mime-type'):
         # '   + foo' where foo is the new value. That's fragile.
         content = rietveld_svn_props.pop(0)
         match2 = re.match(r'^   \+ (.*)$', content)
         if not match2:
           raise patch.UnsupportedPatchFormat(
               filename, 'Unsupported svn property format.')
         svn_props.append((match.group(2), match2.group(1)))
     return svn_props
 
   def update_description(self, issue, description):
@@ skipping 103 matching lines @@
         if retry >= (maxtries - 1):
           raise
         if not 'Name or service not known' in e.reason:
           # Usually internal GAE flakiness.
           raise
       # If reaching this line, loop again. Uses a small backoff.
       time.sleep(1+maxtries*2)
 
   # DEPRECATED.
   Send = get