[Findit] Adding test to improve code coverage

[Findit] Adding test to improve code coverage

BUG=604096
Committed: https://chromium.googlesource.com/infra/infra/+/6b287728967ff7839d3442240e5b24219e600095

[lijeffrey, 2016-04-27 05:48:05]

lijeffrey@chromium.org changed reviewers:
+ chanli@chromium.org, stgao@chromium.org

[lijeffrey, 2016-04-27 05:48:06]

It seems Findit still report 100.0% code coverage despite line 458 missing the
implied else branch.

I'm going to cq this change first to unblock
https://codereview.chromium.org/1917383002/ which enforces 100% code coverage
again, and had to re-added the # pragma: no branch statement which we should
have a test for

[lijeffrey, 2016-04-27 05:48:11]

The CQ bit was checked by lijeffrey@chromium.org

[commit-bot: I haz the power, 2016-04-27 05:48:22]

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

[commit-bot: I haz the power, 2016-04-27 05:48:23]

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

[commit-bot: I haz the power, 2016-04-27 05:48:23]

No L-G-T-M from a valid reviewer yet. 
CQ run can only be started by full committers or once the patch has
received an L-G-T-M from a full committer.
Even if an L-G-T-M may have been provided, it was from a non-committer,
_not_ a full super star committer.
Committers are members of the group "project-infra-committers".
Note that this has nothing to do with OWNERS files.

[lijeffrey, 2016-04-27 05:49:15]

Nevermind, I guess I don't have permission to do that, ptal as soon as you have
the chance :)

[stgao, 2016-04-27 17:09:49]

The CQ bit was checked by stgao@chromium.org

[stgao, 2016-04-27 17:09:49]

lgtm

[commit-bot: I haz the power, 2016-04-27 17:09:52]

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

[stgao, 2016-04-27 17:10:58]

On 2016/04/27 05:49:15, lijeffrey wrote:
> Nevermind, I guess I don't have permission to do that, ptal as soon as you
have
> the chance :)

You could use TBR=ID@chromium.org to bypass the l-g-t-m check.

[commit-bot: I haz the power, 2016-04-27 17:14:10]

Description was changed from

==========
[Findit] Adding test to improve code coverage

BUG=
==========

to

==========
[Findit] Adding test to improve code coverage

BUG=

Committed:
https://chromium.googlesource.com/infra/infra/+/6b287728967ff7839d3442240e5b2...
==========

[commit-bot: I haz the power, 2016-04-27 17:14:11]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/infra/infra/+/6b287728967ff7839d3442240e5b2...

[lijeffrey, 2016-05-02 17:24:19]

Description was changed from

==========
[Findit] Adding test to improve code coverage

BUG=

Committed:
https://chromium.googlesource.com/infra/infra/+/6b287728967ff7839d3442240e5b2...
==========

to

==========
[Findit] Adding test to improve code coverage

BUG=604096

Committed:
https://chromium.googlesource.com/infra/infra/+/6b287728967ff7839d3442240e5b2...
==========