Download archives from the page_set's specified bucket in page_set_archive_info.

Download archives from the page_set's specified bucket in page_set_archive_info.

BUG=434478
Committed: https://crrev.com/9f53699000cf07f07aab14629d5ab3a7b09f6f58
Cr-Commit-Position: refs/heads/master@{#304962}

[aiolos (Not reviewing), 2014-11-19 23:14:39]

aiolos@chromium.org changed reviewers:
+ dtu@chromium.org

[dtu, 2014-11-19 23:44:10]

lgtm nit

https://codereview.chromium.org/726373005/diff/1/tools/telemetry/telemetry/pa...
File tools/telemetry/telemetry/page/page_set_archive_info.py (right):

https://codereview.chromium.org/726373005/diff/1/tools/telemetry/telemetry/pa...
tools/telemetry/telemetry/page/page_set_archive_info.py:16: def __init__(self,
file_path, data, bucket=None, ignore_archive=False):
Can bucket be required?

[aiolos (Not reviewing), 2014-11-20 00:31:30]

https://codereview.chromium.org/726373005/diff/1/tools/telemetry/telemetry/pa...
File tools/telemetry/telemetry/page/page_set_archive_info.py (right):

https://codereview.chromium.org/726373005/diff/1/tools/telemetry/telemetry/pa...
tools/telemetry/telemetry/page/page_set_archive_info.py:16: def __init__(self,
file_path, data, bucket=None, ignore_archive=False):
On 2014/11/19 23:44:10, dtu wrote:
> Can bucket be required?

Done.

[dtu, 2014-11-20 01:28:11]

The CQ bit was checked by dtu@chromium.org

[dtu, 2014-11-20 01:28:11]

lgtm

[commit-bot: I haz the power, 2014-11-20 01:29:27]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/726373005/20001

[commit-bot: I haz the power, 2014-11-20 03:40:22]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2014-11-20 03:40:53]

Patchset 2 (id:??) landed as
https://crrev.com/9f53699000cf07f07aab14629d5ab3a7b09f6f58
Cr-Commit-Position: refs/heads/master@{#304962}