Move url_open with dependencies to utils.net module.

Move url_open with dependencies to utils.net module.

Also add back run_isolated.zip support to swarming.py.

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

[Vadim Sh., 2013-08-28 04:22:05]

Please take a look.

It's a successor of https://codereview.chromium.org/23431002/ with a clean diff.

Rebasing on top of tree changes indeed sucks...

[Vadim Sh., 2013-08-28 04:56:57]

Also this CL should fix canary. All test pass and I explicitly verified that
run_example_swarm.py runs as expected. So feel free to commit this change if it
looks good to you.

[M-A Ruel, 2013-08-28 15:01:48]

lgtm, thanks for fixing.

I noted optional nits you can do as a follow up if you want.

I'll prepare a CL for swarming.py collect error.

https://codereview.chromium.org/23657003/diff/1/run_isolated.py
File run_isolated.py (right):

https://codereview.chromium.org/23657003/diff/1/run_isolated.py#newcode448
run_isolated.py:448: connection = net.url_open(zipped_source, retry_404=True,
I'd align:
connection = net.url_open(
    zipped_source,
    retry_404=True,
    read_timeout=DOWNLOAD_READ_TIMEOUT)

https://codereview.chromium.org/23657003/diff/1/swarming.py
File swarming.py (right):

https://codereview.chromium.org/23657003/diff/1/swarming.py#newcode145
swarming.py:145: data=binascii.unhexlify(self._zip_file_hash),
You could use .digest() instead.

[commit-bot: I haz the power, 2013-08-28 15:02:04]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/vadimsh@chromium.org/23657003/1

[commit-bot: I haz the power, 2013-08-28 15:03:47]

Change committed as 220027