Skip to content

Commit

Permalink
fix: limit fastSeek to Safari based browsers only (#6752)
Browse files Browse the repository at this point in the history
Fixes #6722
  • Loading branch information
gkatsev authored Jul 13, 2020
1 parent 559b3d7 commit 94bea35
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion src/js/tech/html5.js
Original file line number Diff line number Diff line change
Expand Up @@ -536,7 +536,7 @@ class Html5 extends Tech {
*/
setCurrentTime(seconds) {
try {
if (this.isScrubbing_ && this.el_.fastSeek) {
if (this.isScrubbing_ && this.el_.fastSeek && browser.IS_ANY_SAFARI) {
this.el_.fastSeek(seconds);
} else {
this.el_.currentTime = seconds;
Expand Down
2 changes: 1 addition & 1 deletion test/unit/tech/html5.test.js
Original file line number Diff line number Diff line change
Expand Up @@ -51,7 +51,7 @@ QUnit.module('HTML5', {
}
});

QUnit.test('if setScrubbing is true and fastSeek is available, use it', function(assert) {
QUnit[browser.IS_ANY_SAFARI ? 'test' : 'skip']('if setScrubbing is true and fastSeek is available, use it', function(assert) {
Object.defineProperty(tech.el(), 'currentTime', {
get: () => {},
set: () => {},
Expand Down

0 comments on commit 94bea35

Please sign in to comment.