[Downloads] Retain a BlobDataHandle in DownloadUrlParameters.

[Downloads] Retain a BlobDataHandle in DownloadUrlParameters.

This allows a caller to extend the lifetime of a blob until the
downloads / resource loader subsystems take over. Without this, an
unfortunately timed revokeObjectURL() call from the originating page may
invalidate the URL before the download has a chance to start the
associated download request.

BUG=595305
Committed: https://crrev.com/6150c52a037d3b05eeac0faaf792ab0203b5580f
Cr-Commit-Position: refs/heads/master@{#383370}

[asanka, 2016-03-25 19:17:21]

asanka@chromium.org changed reviewers:
+ jam@chromium.org

[asanka, 2016-03-25 19:18:42]

Description was changed from

==========
[Downloads] Retain a BlobDataHandle in DownloadUrlParameters.

This allows a caller to extend the lifetime of a blob URL until the
downloads / resource loader subsystems take over.

BUG=595305
==========

to

==========
[Downloads] Retain a BlobDataHandle in DownloadUrlParameters.

This allows a caller to extend the lifetime of a blob until the
downloads / resource loader subsystems take over. Without this, if an
unfortunately timed revokeObjectURL() call from the originating page may
invalidate the URL before the download has a chance to start the
associated download request.

BUG=595305
==========

[asanka, 2016-03-25 19:19:41]

Description was changed from

==========
[Downloads] Retain a BlobDataHandle in DownloadUrlParameters.

This allows a caller to extend the lifetime of a blob until the
downloads / resource loader subsystems take over. Without this, if an
unfortunately timed revokeObjectURL() call from the originating page may
invalidate the URL before the download has a chance to start the
associated download request.

BUG=595305
==========

to

==========
[Downloads] Retain a BlobDataHandle in DownloadUrlParameters.

This allows a caller to extend the lifetime of a blob until the
downloads / resource loader subsystems take over. Without this, an
unfortunately timed revokeObjectURL() call from the originating page may
invalidate the URL before the download has a chance to start the
associated download request.

BUG=595305
==========

[asanka, 2016-03-25 19:20:56]

jam: PTAL?

[jam, 2016-03-25 19:31:16]

lgtm

[asanka, 2016-03-25 20:10:01]

The CQ bit was checked by asanka@chromium.org

[commit-bot: I haz the power, 2016-03-25 20:10:25]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1829413002/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1829413002/40001

[commit-bot: I haz the power, 2016-03-25 21:41:13]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-03-25 21:45:40]

Description was changed from

==========
[Downloads] Retain a BlobDataHandle in DownloadUrlParameters.

This allows a caller to extend the lifetime of a blob until the
downloads / resource loader subsystems take over. Without this, an
unfortunately timed revokeObjectURL() call from the originating page may
invalidate the URL before the download has a chance to start the
associated download request.

BUG=595305
==========

to

==========
[Downloads] Retain a BlobDataHandle in DownloadUrlParameters.

This allows a caller to extend the lifetime of a blob until the
downloads / resource loader subsystems take over. Without this, an
unfortunately timed revokeObjectURL() call from the originating page may
invalidate the URL before the download has a chance to start the
associated download request.

BUG=595305

Committed: https://crrev.com/6150c52a037d3b05eeac0faaf792ab0203b5580f
Cr-Commit-Position: refs/heads/master@{#383370}
==========

[commit-bot: I haz the power, 2016-03-25 21:45:41]

Patchset 3 (id:??) landed as
https://crrev.com/6150c52a037d3b05eeac0faaf792ab0203b5580f
Cr-Commit-Position: refs/heads/master@{#383370}