Cleanup verifyKeyStatuses() to take KeyId/status pairs

Cleanup verifyKeyStatuses() to take KeyId/status pairs

As suggested in a previous CL, the status for each keyId should have it's own
status value rather than relying on all the KeyIds to have the same status.

No longer passing |unexpected|, as the size check verifies that the correct
number of expected KeyIds matches.

BUG=616166
TEST=modified tests pass

Review-Url: https://codereview.chromium.org/2847803005
Cr-Commit-Position: refs/heads/master@{#468105}
Committed: https://chromium.googlesource.com/chromium/src/+/42f79db5eabcab9856321dcbdec9953c156784a6

[jrummell, 2017-04-28 01:09:14]

Description was changed from

==========
Cleanup verifyKeyStatuses() to take KeyId/status pairs

As suggested in a previous CL, the status for each keyId should have it's own
status value rather than relying on all the KeyIds to have the same status.

BUG=616166
TEST=modified tests pass
==========

to

==========
Cleanup verifyKeyStatuses() to take KeyId/status pairs

As suggested in a previous CL, the status for each keyId should have it's own
status value rather than relying on all the KeyIds to have the same status.

No longer passing |unexpected|, as the size check verifies that the correct
number of expected KeyIds matches.

BUG=616166
TEST=modified tests pass
==========

[jrummell, 2017-04-28 01:09:30]

jrummell@chromium.org changed reviewers:
+ xhwang@chromium.org

[jrummell, 2017-04-28 01:09:31]

PTAL

[xhwang, 2017-04-28 16:23:28]

Nice cleanup! LGTM

[jrummell, 2017-04-28 18:04:27]

The CQ bit was checked by jrummell@chromium.org

[commit-bot: I haz the power, 2017-04-28 18:05:31]

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, 2017-04-28 20:12:06]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1493402667513540, "parent_rev":
"f9bf5ca85577c34702afe733dcb6d1d98ed728a0", "commit_rev":
"42f79db5eabcab9856321dcbdec9953c156784a6"}

[commit-bot: I haz the power, 2017-04-28 20:12:34]

Description was changed from

==========
Cleanup verifyKeyStatuses() to take KeyId/status pairs

As suggested in a previous CL, the status for each keyId should have it's own
status value rather than relying on all the KeyIds to have the same status.

No longer passing |unexpected|, as the size check verifies that the correct
number of expected KeyIds matches.

BUG=616166
TEST=modified tests pass
==========

to

==========
Cleanup verifyKeyStatuses() to take KeyId/status pairs

As suggested in a previous CL, the status for each keyId should have it's own
status value rather than relying on all the KeyIds to have the same status.

No longer passing |unexpected|, as the size check verifies that the correct
number of expected KeyIds matches.

BUG=616166
TEST=modified tests pass

Review-Url: https://codereview.chromium.org/2847803005
Cr-Commit-Position: refs/heads/master@{#468105}
Committed:
https://chromium.googlesource.com/chromium/src/+/42f79db5eabcab9856321dcbdec9...
==========

[commit-bot: I haz the power, 2017-04-28 20:12:42]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/42f79db5eabcab9856321dcbdec9...