Bugzilla

[Try run where I overload the assert to do some logging](https://treeherder.mozilla.org/#/jobs?repo=try&selectedJob=254471969&revision=40a95c2a44796bd7933eb3a84662d9c3ad252164). It looks like we're not ever firing `waitingForKey` which is a real head-scratcher.

Trimmed log from when the assert fires:
```
video.duration: 5.083333, video.currentTime: 5.077332, everWaitingForKey:false, waitingForKeyTime: 0
```

In local runs where I see `waitingForKey` fired just before 2 seconds. So the test is not seeing this event, and we're also continuing to play the video to the end.

I'm going to retrigger some more test runs on the cases mentioned in comment 4 to see if we get any more fails with a bigger sample.

[Try run where I overload the assert to do some logging](https://treeherder.mozilla.org/#/jobs?repo=try&selectedJob=254471969&revision=40a95c2a44796bd7933eb3a84662d9c3ad252164). It looks like we're not ever firing `waitingForKey` which is a real head-scratcher.

Trimmed log from when the assert fires:
```
video.duration: 5.083333, video.currentTime: 5.077332, everWaitingForKey:false, waitingForKeyTime: 0
```

In local runs I see `waitingForKey` fired just before 2 seconds. So the test is not seeing this event, and we're also continuing to play the video to the end.

I'm going to retrigger some more test runs on the cases mentioned in comment 4 to see if we get any more fails with a bigger sample.