Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(73)

Unified Diff: media/webm/webm_stream_parser.h

Issue 11471006: Log MediaSource parsing errors to the MediaLog so they can appear in chrome:media-internals. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: . Created 8 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: media/webm/webm_stream_parser.h
diff --git a/media/webm/webm_stream_parser.h b/media/webm/webm_stream_parser.h
index 36401510075340f79380b5cab663d23f8d14aa8c..7da1f6ac223163f3708167afa03e223d2559f613 100644
--- a/media/webm/webm_stream_parser.h
+++ b/media/webm/webm_stream_parser.h
@@ -27,7 +27,8 @@ class WebMStreamParser : public StreamParser {
const NewBuffersCB& video_cb,
const NeedKeyCB& need_key_cb,
const NewMediaSegmentCB& new_segment_cb,
- const base::Closure& end_of_segment_cb) OVERRIDE;
+ const base::Closure& end_of_segment_cb,
+ const ErrorLogCB& error_cb) OVERRIDE;
virtual void Flush() OVERRIDE;
virtual bool Parse(const uint8* buf, int size) OVERRIDE;
@@ -71,6 +72,7 @@ class WebMStreamParser : public StreamParser {
NeedKeyCB need_key_cb_;
NewMediaSegmentCB new_segment_cb_;
base::Closure end_of_segment_cb_;
+ ErrorLogCB error_cb_;
// True if a new cluster id has been seen, but no audio or video buffers have
// been parsed yet.

Powered by Google App Engine
This is Rietveld 408576698