fix: replay case (#60)

zhihu · Apr 9, 2019 · e2693ad · e2693ad
1 parent e295326
commit e2693ad
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 3 deletions.
diff --git a/packages/griffith-mp4/src/mse/controller.js b/packages/griffith-mp4/src/mse/controller.js
@@ -151,6 +151,8 @@ export default class MSE {
       return
     }
 
+    this.handleReplayCase()
+
     this.loadData(start, end).then(mdatBuffer => {
       if (!mdatBuffer) {
         return
@@ -200,7 +202,8 @@ export default class MSE {
       videoInterval: {offsetInterVal = []} = [],
       mp4Data: {videoSamplesLength},
     } = this.mp4Probe
-    if (this.mediaSource.readyState === 'open') {
+
+    if (this.mediaSource.readyState !== 'closed') {
       if (offsetInterVal[1] === videoSamplesLength && !this.mseUpdating) {
         this.destroy()
       } else if (this.shouldFetchNextSegment()) {
@@ -209,7 +212,15 @@ export default class MSE {
     }
   }
 
+  handleReplayCase = () => {
+    if (this.mediaSource.readyState === 'ended') {
+      this.sourceBuffers.video.appendBuffer(new Uint8Array(0))
+    }
+  }
+
   shouldFetchNextSegment = () => {
+    this.handleReplayCase()
+
     if (!this.mseUpdating && this.mp4Probe.isDraining(this.video.currentTime)) {
       return true
     }

diff --git a/packages/griffith-mp4/src/player.js b/packages/griffith-mp4/src/player.js
@@ -43,8 +43,8 @@ export default class Player extends Component {
   }
 
   handlePlay = e => {
-    const {currentTime, buffered = []} = this.video
-    if (currentTime === 0 && !buffered.length) {
+    const {currentTime} = this.video
+    if (currentTime === 0) {
       this.mse.seek(0)
     }