MSE: Log buffered audio splice generation to media-internals

content/browser/media/media_internals.cc

Index: content/browser/media/media_internals.cc
diff --git a/content/browser/media/media_internals.cc b/content/browser/media/media_internals.cc
index 4b4d6cd6b3d3cb4e415a97c43fa0bc93e10a97f2..f9d15bf9037de730139695fd4d03ec43cc578cfc 100644
--- a/content/browser/media/media_internals.cc
+++ b/content/browser/media/media_internals.cc
@@ -641,6 +641,22 @@ void MediaInternals::SaveEvent(int process_id,
   }
 
   auto& pending_events = pending_events_map_[process_id];
+
+  // Only save the most recent audio splicer statistics, if any. If the new
+  // event updates the statistics, then remove at most one previously saved
+  // statistics event.
+  if (event.type ==
+      media::MediaLogEvent::BUFFERED_AUDIO_SPLICE_STATISTICS_CHANGED) {
+    for (auto iter = pending_events.begin(); iter != pending_events.end();

[wolenetz, 2015/07/14 23:23:37]

This isn't a very efficient mechanism for enforcing at-most-one. wdyt?

[DaleCurtis, 2015/07/14 23:37:47]

I think you want to do something similar to what RendererMediaLog does and just
drop these events instead of saving them if media internals isn't open.

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

Maybe it's time to drop all DEBUG logs when the UI is not open?

+         ++iter) {
+      if (iter->type ==
+          media::MediaLogEvent::BUFFERED_AUDIO_SPLICE_STATISTICS_CHANGED) {
+        pending_events.erase(iter);
+        break;
+      }
+    }
+  }
+
   // TODO(xhwang): Notify user that some old logs could have been truncated.
   // See http://crbug.com/498520
   if (pending_events.size() >= kMaxNumEvents)