Merge pull request Streamedian#62 from mburger81/patch-3

Failed to execute 'remove' on 'SourceBuffer'
acguokailin · Oct 29, 2017 · 226618c · 226618c
2 parents 3189f4d + fdc9f07
commit 226618c
Showing 1 changed file with 12 additions and 3 deletions.
diff --git a/src/core/presentation/mse.js b/src/core/presentation/mse.js
@@ -102,8 +102,17 @@ export class MSEBuffer {
             this.cleaning = true;
             promises.push(new Promise((resolve, reject)=>{
                 this.cleanResolvers.push(resolve);
-                this.sourceBuffer.remove(this.sourceBuffer.buffered.start(i), this.sourceBuffer.buffered.end(i));
-                resolve();
+                if (!this.sourceBuffer.updating) {
+                    this.sourceBuffer.remove(this.sourceBuffer.buffered.start(i), this.sourceBuffer.buffered.end(i));
+                    resolve();
+                } else {
+                    this.sourceBuffer.onupdateend = () => {
+                        if (this.sourceBuffer) {
+                            this.sourceBuffer.remove(this.sourceBuffer.buffered.start(i), this.sourceBuffer.buffered.end(i));
+                        }
+                        resolve();
+                    };
+                }
             }));
         }
         return Promise.all(promises);
@@ -378,4 +387,4 @@ export class MSE {
             this.buffers[track].feed(data);
         }
     }
-}
+}