MSE: Plumb ChunkDemuxer appendData failures into append Error algorithm

third_party/WebKit/Source/modules/mediasource/SourceBuffer.h

Index: third_party/WebKit/Source/modules/mediasource/SourceBuffer.h
diff --git a/third_party/WebKit/Source/modules/mediasource/SourceBuffer.h b/third_party/WebKit/Source/modules/mediasource/SourceBuffer.h
index 5add19dff48d8a5284479b754ee72dff2b73b765..5d6d57ff8952a3aebed04c2b026933d9aed981d1 100644
--- a/third_party/WebKit/Source/modules/mediasource/SourceBuffer.h
+++ b/third_party/WebKit/Source/modules/mediasource/SourceBuffer.h
@@ -131,7 +131,10 @@ private:
 
     void appendStreamInternal(Stream*, ExceptionState&);
     void appendStreamAsyncPart();
-    void appendStreamDone(bool success);
+
+    // |decodeError| is only valid if |runAppendError| is true.

[foolip, 2016/06/20 12:04:17]

Smells like this should be a 3-state enum?

[wolenetz, 2016/06/21 19:38:09]

I was thinking similarly and certainly would have done that already if this
method were public. I changed to using an enum. And will do the same in future
per Blink code style that recommends this :)

I also updated the ::appendError(..) method to take an enum instead of a bool,
following that same code style.

[foolip, 2016/06/22 12:39:52]

Yay :)

+    void appendStreamDone(bool runAppendError, bool decodeError);
+
     void clearAppendStreamState();
 
     void removeMediaTracks();