Do not retry uploading to blobstore on HTTP 500, regenerate a new url first.

Do not retry uploading to blobstore on HTTP 500, regenerate a new url first.

This is important since after an HTTP 500, the temporary url is likely invalid.

R=csharp@chromium.org
BUG=

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=196770

[M-A Ruel, 2013-04-26 16:39:01]

The CL exploded a bit so I'm fine if you prefer me to split it in two, the
removal of *args caused the majority of the changes.

[csharp, 2013-04-26 17:14:06]

On 2013/04/26 16:39:01, Marc-Antoine Ruel wrote:
> The CL exploded a bit so I'm fine if you prefer me to split it in two, the
> removal of *args caused the majority of the changes.

Two cls would be awesome :)

[M-A Ruel, 2013-04-26 17:31:36]

Now much simpler

[csharp, 2013-04-26 17:51:58]

lgtm

https://codereview.chromium.org/14455006/diff/4001/isolateserver_archive.py
File isolateserver_archive.py (right):

https://codereview.chromium.org/14455006/diff/4001/isolateserver_archive.py#n...
isolateserver_archive.py:139: content_type, body = encode_multipart_formdata(
Move this out of the loop

[M-A Ruel, 2013-04-26 17:56:22]

https://codereview.chromium.org/14455006/diff/4001/isolateserver_archive.py
File isolateserver_archive.py (right):

https://codereview.chromium.org/14455006/diff/4001/isolateserver_archive.py#n...
isolateserver_archive.py:139: content_type, body = encode_multipart_formdata(
On 2013/04/26 17:51:58, csharp wrote:
> Move this out of the loop

Oops, done.

[commit-bot: I haz the power, 2013-04-26 17:56:30]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/maruel@chromium.org/14455006/8001

[commit-bot: I haz the power, 2013-04-26 17:57:43]

Change committed as 196770