Skip to content

Commit

Permalink
fix(FEC-7970): Change media from DRM to DRM is not working (#202)
Browse files Browse the repository at this point in the history
Remove destroy from DrmProtocol interface since it has no meaning and we're overriding the old protocol with the new protocol on change media.
  • Loading branch information
Dan Ziv committed Mar 4, 2018
1 parent 2be251d commit b9974ad
Show file tree
Hide file tree
Showing 3 changed files with 0 additions and 8 deletions.
1 change: 0 additions & 1 deletion flow-typed/interfaces/drm-protocol.js
Original file line number Diff line number Diff line change
Expand Up @@ -2,5 +2,4 @@
declare interface IDrmProtocol {
static canPlayDrm(drmData: Array<Object>): boolean;
static setDrmPlayback(...any): void;
static destroy(): void;
}
3 changes: 0 additions & 3 deletions src/drm/base-drm-protocol.js
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,4 @@ export default class BaseDrmProtocol implements IDrmProtocol {
static setDrmPlayback(...args): void {
throw new Error(Error.Severity.CRITICAL, Error.Category.PLAYER, Error.Code.RUNTIME_ERROR_METHOD_NOT_IMPLEMENTED, 'static setDrmPlayback');
}

static destroy(): void {
}
}
4 changes: 0 additions & 4 deletions src/engines/html5/media-source/adapters/native-adapter.js
Original file line number Diff line number Diff line change
Expand Up @@ -281,10 +281,6 @@ export default class NativeAdapter extends BaseMediaSourceAdapter {
clearInterval(this._liveDurationChangeInterval);
this._liveDurationChangeInterval = null;
}
if (NativeAdapter._drmProtocol) {
NativeAdapter._drmProtocol.destroy();
NativeAdapter._drmProtocol = null;
}
});
}

Expand Down

0 comments on commit b9974ad

Please sign in to comment.