Skip to content

Commit

Permalink
Remove flaky assertion in EME polyfill.
Browse files Browse the repository at this point in the history
This assertion did not add much value, and it sometimes triggers a
test failure.  Since prefixed EME does not have actual session
management, it's entirely possible for a stray message to come in
for a session we've already destroyed in the polyfill.  Rather than
assert, just ignore it.

Change-Id: I4779d3f36a25bc21ade95c4ea3f25ad1f8a96e7b
  • Loading branch information
joeyparrish committed Mar 31, 2015
1 parent 1c8e72d commit 9940d96
Showing 1 changed file with 0 additions and 1 deletion.
1 change: 0 additions & 1 deletion lib/polyfill/patchedmediakeys_v01b.js
Original file line number Diff line number Diff line change
Expand Up @@ -390,7 +390,6 @@ shaka.polyfill.PatchedMediaKeys.v01b.MediaKeys.prototype.onWebkitKeyMessage_ =
shaka.log.debug('v01b.onWebkitKeyMessage_', event);

var session = this.findSession_(event.sessionId);
shaka.asserts.assert(session);
if (!session) {
shaka.log.error('Session not found', event.sessionId);
return;
Expand Down

0 comments on commit 9940d96

Please sign in to comment.