MD Downloads: ignore empty URLs

MD Downloads: ignore empty URLs

R=mmenke@chromium.org
BUG=635220
Committed: https://crrev.com/d7f1230e51b779d8f81b140be7cd9db8734d5841
Cr-Commit-Position: refs/heads/master@{#411242}

[mmenke, 2016-08-11 01:20:05]

LGTM, that works.  Doesn't look like there's a test fixture for it (Though this
is one rare case where I'm not sure a test is worth the effort, anyways).

[Dan Beam, 2016-08-11 01:21:26]

On 2016/08/11 01:20:05, mmenke wrote:
> LGTM, that works.  Doesn't look like there's a test fixture for it (Though
this
> is one rare case where I'm not sure a test is worth the effort, anyways).

after 30m for a 1s fix, let me assure you it's not

[Dan Beam, 2016-08-11 01:21:40]

On 2016/08/11 01:21:26, Dan Beam wrote:
> On 2016/08/11 01:20:05, mmenke wrote:
> > LGTM, that works.  Doesn't look like there's a test fixture for it (Though
> this
> > is one rare case where I'm not sure a test is worth the effort, anyways).
> 
> after 30m for a 1s fix, let me assure you it's not

30m of trying to test it*

[Dan Beam, 2016-08-11 01:21:50]

The CQ bit was checked by dbeam@chromium.org

[commit-bot: I haz the power, 2016-08-11 01:22:52]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-11 03:04:31]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-08-11 03:07:46]

Description was changed from

==========
MD Downloads: ignore empty URLs

R=mmenke@chromium.org
BUG=635220
==========

to

==========
MD Downloads: ignore empty URLs

R=mmenke@chromium.org
BUG=635220

Committed: https://crrev.com/d7f1230e51b779d8f81b140be7cd9db8734d5841
Cr-Commit-Position: refs/heads/master@{#411242}
==========

[commit-bot: I haz the power, 2016-08-11 03:07:47]

Patchset 1 (id:??) landed as
https://crrev.com/d7f1230e51b779d8f81b140be7cd9db8734d5841
Cr-Commit-Position: refs/heads/master@{#411242}