MSE: Log buffered audio splice generation to media-internals

media/base/media_log_event.h

Index: media/base/media_log_event.h
diff --git a/media/base/media_log_event.h b/media/base/media_log_event.h
index 66aa5f7826d1dcfcf7c25219c248a0632a4059d1..5f57fad659ef605ef5b15f651ecdf548d7d332bc 100644
--- a/media/base/media_log_event.h
+++ b/media/base/media_log_event.h
@@ -77,6 +77,18 @@ struct MediaLogEvent {
     //         "buffer_end": <last buffered byte>.
     BUFFERED_EXTENTS_CHANGED,
 
+    // Buffered audio splice frame statistics have changed.
+    // params: "audio_splice_count": <count of splice frames buffered>
+    //         "audio_splice_degenerate_count": <count of degenerate splices>
+    //           A degenerate splice is any splice that was prevented from
+    //           generation due to exact buffer overlap, overlap of previous
+    //           splice, overlap of preroll, or overlap duration below minimum
+    //           threshold.
+    //         "audio_splice_duration_average": <seconds>
+    //         "audio_splice_duration_min": <seconds>
+    //         "audio_splice_duration_max": <seconds>
+    BUFFERED_AUDIO_SPLICE_STATISTICS_CHANGED,

[xhwang, 2015/07/15 00:46:50]

+dalecurtis

When I first looked at this list (Type), I found it a bit arbitrary. 
For example, what's the fundamental difference between an event with a
specialized "type", and those general ERROR/INFO/DEBUG events below? Somehow I
feel we have two systems here.

Can we simply replace the "Type type" with a "string type" so that we don't need
most of this list here? The downside is that we'll need to send a "type" string
over the wire instead of a "type" integer, but that should be small compared to
the whole data we are sending. We can still keep ERROR/INFO/DEBUG as log
"Level"s so that we can filter them if we need to.

+
     // Error log reported by media code such as reasons of playback error.
     MEDIA_ERROR_LOG_ENTRY,
     // params: "error": Error string describing the error detected.