Increase coverage to 91%; Much stricter about header parsing.

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 115 matching lines @@
         out.append(patch.FilePatchDelete(filename, state['is_binary']))
       elif status in ('A', 'M'):
         # TODO(maruel): Rietveld support is still weird, add this line once it's
         # safe to use.
         # props = state.get('property_changes', '').splitlines() or []
         props = []
         if state['is_binary']:
           out.append(patch.FilePatchBinary(
               filename,
               self.get_file_content(issue, patchset, state['id']),
-              props))
+              props,
+              is_new=(status[0] == 'A')))
         else:
           if state['num_chunks']:
             diff = self.get_file_diff(issue, patchset, state['id'])
           else:
             raise patch.UnsupportedPatchFormat(
                 filename, 'File doesn\'t have a diff.')
           out.append(patch.FilePatchDiff(filename, diff, props))
       else:
         # Line too long (N/80)
         # pylint: disable=C0301
@@ skipping 57 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