Fix crrev and add test coverage for the main function.

Fix crrev and add test coverage for the main function.

Specifically, what was broken was a simple typo: json.lad instead of json.load. I didn't catch this before because there was no test coverage for the `main` function.

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

[qyearsley, 2016-03-14 20:23:36]

qyearsley@chromium.org changed reviewers:
+ prasadv@chromium.org, robertocn@chromium.org

[qyearsley, 2016-03-15 18:03:11]

The CQ bit was checked by qyearsley@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-03-15 18:03:14]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1796993002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1796993002/1

[commit-bot: I haz the power, 2016-03-15 18:06:44]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-03-15 18:06:45]

Dry run: This issue passed the CQ dry run.

[qyearsley, 2016-03-15 21:01:38]

On 2016/03/15 at 18:06:45, commit-bot wrote:
> Dry run: This issue passed the CQ dry run.

Ping Prasad, Roberto

[qyearsley, 2016-03-18 19:49:16]

Ping again

[RobertoCN, 2016-03-23 02:11:19]

lgtm

[qyearsley, 2016-03-23 02:29:07]

The CQ bit was checked by qyearsley@chromium.org

[commit-bot: I haz the power, 2016-03-23 02:29:13]

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

[commit-bot: I haz the power, 2016-03-23 02:35:08]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-03-23 02:35:09]

Try jobs failed on following builders:
  Build Presubmit on tryserver.infra (JOB_FAILED,
https://build.chromium.org/p/tryserver.infra/builders/Build%20Presubmit/build...)

[qyearsley, 2016-03-23 21:12:45]

qyearsley@chromium.org changed reviewers:
+ iannucci@chromium.org

[qyearsley, 2016-03-23 21:13:24]

Description was changed from

==========
Fix crrev and add test coverage for the main function.

BUG=594605
==========

to

==========
Fix crrev and add test coverage for the main function.

Specifically, what was broken was a simple typo: json.lad instead of json.load.
I didn't catch this before because there was no test coverage for the `main`
function.

BUG=594605
==========

[qyearsley, 2016-03-23 21:14:15]

On 2016/03/23 at 02:35:09, commit-bot wrote:
> Try jobs failed on following builders:
>   Build Presubmit on tryserver.infra (JOB_FAILED,
https://build.chromium.org/p/tryserver.infra/builders/Build%20Presubmit/build...)

This change requires infra OWNERS LGTM.

Robbie, could you take a quick look at this when you have time and stamp it if
it looks OK?

[qyearsley, 2016-03-25 21:25:38]

On 2016/03/23 at 21:14:15, qyearsley wrote:
> On 2016/03/23 at 02:35:09, commit-bot wrote:
> > Try jobs failed on following builders:
> >   Build Presubmit on tryserver.infra (JOB_FAILED,
https://build.chromium.org/p/tryserver.infra/builders/Build%20Presubmit/build...)
> 
> This change requires infra OWNERS LGTM.
> 
> Robbie, could you take a quick look at this when you have time and stamp it if
it looks OK?

Ping iannucci@

[qyearsley, 2016-03-29 16:10:08]

qyearsley@chromium.org changed reviewers:
+ dnj@chromium.org

[qyearsley, 2016-03-29 16:13:06]

+dnj

This is a fix (json.lad -> json.load) with a test to cover the lines where the
error occurred, and I need infra/build OWNERS stamp to land.

[dnj, 2016-03-29 16:25:49]

lgtm

[qyearsley, 2016-03-29 16:29:34]

The CQ bit was checked by qyearsley@chromium.org

[commit-bot: I haz the power, 2016-03-29 16:29:47]

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

[qyearsley, 2016-03-29 16:29:56]

On 2016/03/29 at 16:25:49, dnj wrote:
> lgtm

Thanks :-)

[commit-bot: I haz the power, 2016-03-29 16:33:33]

Description was changed from

==========
Fix crrev and add test coverage for the main function.

Specifically, what was broken was a simple typo: json.lad instead of json.load.
I didn't catch this before because there was no test coverage for the `main`
function.

BUG=594605
==========

to

==========
Fix crrev and add test coverage for the main function.

Specifically, what was broken was a simple typo: json.lad instead of json.load.
I didn't catch this before because there was no test coverage for the `main`
function.

BUG=594605

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

[commit-bot: I haz the power, 2016-03-29 16:33:35]

Committed patchset #1 (id:1) as
http://src.chromium.org/viewvc/chrome?view=rev&revision=299538