Make apply_issue fail if the content of a large file is missing.

rietveld.py

Index: rietveld.py
diff --git a/rietveld.py b/rietveld.py
index 8e99e4bc2a1663b8fb8d094b5d9155154a6aecca..9adc9c5ca1346a99c79552dc62ae77c17c576871 100644
--- a/rietveld.py
+++ b/rietveld.py
@@ -148,7 +148,7 @@ class Rietveld(object):
           out.append(patch.FilePatchDelete(filename, state['is_binary']))
         else:
           content = self.get_file_content(issue, patchset, state['id'])
-          if not content:
+          if not content or content == 'None':

[tandrii(chromium), 2016/01/22 21:08:25]

ugh, but whatever works.

[Sébastien Marchand, 2016/01/22 21:11:20]

Yeah, this is ugly and it took me quite some time to debug this (because I was
printing the value of |content|, so I was confused by the fact that no exception
was raised although content was 'None'...). I don't know who's emitting this
string, but we should probably change it to something less confusing (or set it
to the 'real' None).

             # As a precaution due to a bug in upload.py for git checkout, refuse
             # empty files. If it's empty, it's not a binary file.
             raise patch.UnsupportedPatchFormat(