Skip to content

Commit

Permalink
fix: remove hack (#424)
Browse files Browse the repository at this point in the history
  • Loading branch information
incompl authored Aug 31, 2018
1 parent 852e6c5 commit 578ee12
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 5 deletions.
3 changes: 0 additions & 3 deletions src/states/BeforePreroll.js
Original file line number Diff line number Diff line change
Expand Up @@ -85,9 +85,6 @@ export default class BeforePreroll extends ContentState {

onContentChanged() {
this.init(this.player);

// TODO: Handle case where player does not play on content change
this.onPlay(this.player);
}

}
2 changes: 0 additions & 2 deletions test/unit/states/test.BeforePreroll.js
Original file line number Diff line number Diff line change
Expand Up @@ -96,10 +96,8 @@ QUnit.test('skips the preroll', function(assert) {

QUnit.test('handles content change', function(assert) {
sinon.spy(this.beforePreroll, "init");
sinon.spy(this.beforePreroll, "onPlay");
this.beforePreroll.onContentChanged(this.player);
assert.equal(this.beforePreroll.init.calledOnce, true);
assert.equal(this.beforePreroll.onPlay.calledOnce, true);
});

QUnit.test('sets _shouldBlockPlay to true', function(assert) {
Expand Down

0 comments on commit 578ee12

Please sign in to comment.