In LayoutTestResult, change is_missing_* methods to accept explicit False.

In LayoutTestResult, change is_missing_* methods to accept explicit False.

This doesn't change any behavior in practice, since whenever is_missing_*
is present in a layout test dict, it's always true, as far as I know.

This change also changes did_run_as_expected.

So, this is just a minor tweak for correctness in case those fields are
ever explicitly given as False.

Committed: https://crrev.com/57f8a91b6ca44aca59df7c39b8907e35d33bfaf5
Cr-Commit-Position: refs/heads/master@{#417967}

[qyearsley, 2016-09-09 22:06:22]

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

[Dirk Pranke, 2016-09-10 16:21:27]

I'd probably add in the default 'False' values just to be slightly clearer. I
don't want to have to think about what the default is (None) and then the
boolean conversion.

lgtm though.

[qyearsley, 2016-09-12 16:19:06]

The CQ bit was checked by qyearsley@chromium.org

[qyearsley, 2016-09-12 16:19:06]

The patchset sent to the CQ was uploaded after l-g-t-m from dpranke@chromium.org
Link to the patchset: https://codereview.chromium.org/2326063003/#ps20001
(title: "Specify False as the default value.")

[commit-bot: I haz the power, 2016-09-12 16:19:35]

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-09-12 17:36:23]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-09-12 17:38:14]

Description was changed from

==========
In LayoutTestResult, change is_missing_* methods to accept explicit False.

This doesn't change any behavior in practice, since whenever is_missing_*
is present in a layout test dict, it's always true, as far as I know.

This change also changes did_run_as_expected.

So, this is just a minor tweak for correctness in case those fields are
ever explicitly given as False.
==========

to

==========
In LayoutTestResult, change is_missing_* methods to accept explicit False.

This doesn't change any behavior in practice, since whenever is_missing_*
is present in a layout test dict, it's always true, as far as I know.

This change also changes did_run_as_expected.

So, this is just a minor tweak for correctness in case those fields are
ever explicitly given as False.

Committed: https://crrev.com/57f8a91b6ca44aca59df7c39b8907e35d33bfaf5
Cr-Commit-Position: refs/heads/master@{#417967}
==========

[commit-bot: I haz the power, 2016-09-12 17:38:15]

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