Add media log messages to the main log

media/base/media_log.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "media/base/media_log.h"
 
 #include <string>
 
 #include "base/atomic_sequence_num.h"
 #include "base/logging.h"
+#include "base/strings/string_number_conversions.h"
 #include "base/values.h"
 
 namespace media {
 
 // A count of all MediaLogs created in the current process. Used to generate
 // unique IDs.
 static base::StaticAtomicSequenceNumber g_media_log_count;
 
 std::string MediaLog::EventTypeToString(MediaLogEvent::Type type) {
   switch (type) {
@@ skipping 70 matching lines @@
       return "demuxer: could not parse";
     case DEMUXER_ERROR_NO_SUPPORTED_STREAMS:
       return "demuxer: no supported streams";
     case DECODER_ERROR_NOT_SUPPORTED:
       return "decoder: not supported";
   }
   NOTREACHED();
   return NULL;
 }
 
+std::string MediaLog::MediaEventToLogString(const MediaLogEvent& event) {

[wolenetz, 2015/02/04 22:04:18]

This method seems quite fragile to event parameter type changes elsewhere in
code. What did the JSON look like? Might there be a better way to dump all event
properties, that isn't so fragile?

+  std::string param;
+  switch (event.type) {
+    case MediaLogEvent::WEBMEDIAPLAYER_CREATED:
+    case MediaLogEvent::WEBMEDIAPLAYER_DESTROYED:
+    case MediaLogEvent::PIPELINE_CREATED:
+    case MediaLogEvent::PIPELINE_DESTROYED:
+    case MediaLogEvent::PLAY:
+    case MediaLogEvent::PAUSE:
+    case MediaLogEvent::ENDED:
+    case MediaLogEvent::TEXT_ENDED:
+    case MediaLogEvent::BUFFERED_EXTENTS_CHANGED:
+    case MediaLogEvent::TOTAL_BYTES_SET:
+    case MediaLogEvent::NETWORK_ACTIVITY_SET:
+    case MediaLogEvent::PROPERTY_CHANGE:
+      return EventTypeToString(event.type);
+
+    case MediaLogEvent::LOAD:
+      event.params.GetString("url", &param);
+      return EventTypeToString(event.type) + " url=" + param;
+
+    case MediaLogEvent::SEEK:
+      {
+        double seek_target = 0.0;
+        if (event.params.GetDouble("seek_target", &seek_target)) {
+          return EventTypeToString(event.type) + " seek_target=" +
+              base::DoubleToString(seek_target);
+        }
+        return EventTypeToString(event.type) + " seek_target=unknown";
+      }
+
+    case MediaLogEvent::PIPELINE_STATE_CHANGED:
+      event.params.GetString("pipeline_state", &param);
+      return EventTypeToString(event.type) + " pipeline_state=" + param;
+
+    case MediaLogEvent::PIPELINE_ERROR:
+      event.params.GetString("pipeline_error", &param);
+      return EventTypeToString(event.type) + " pipeline_error=" + param;
+
+    case MediaLogEvent::VIDEO_SIZE_SET:
+      {
+        int width = 0;
+        int height = 0;
+        event.params.GetInteger("width", &width);
+        event.params.GetInteger("height", &height);
+        return EventTypeToString(event.type) + " " + base::IntToString(width) +
+            "x" + base::IntToString(height);
+      }
+
+    case MediaLogEvent::DURATION_SET:
+      {
+        double duration = 0.0;
+        if (event.params.GetDouble("duration", &duration)) {
+          return EventTypeToString(event.type) + " duration=" +
+              base::DoubleToString(duration);
+        }
+        return EventTypeToString(event.type) + " duration=unknown";
+      }
+
+    case MediaLogEvent::MEDIA_SOURCE_ERROR:
+      event.params.GetString("error", &param);
+      return EventTypeToString(event.type) + " error=" + param;
+
+    default:

[wolenetz, 2015/02/04 22:04:18]

Please drop the default case and instead let build fail if not all enum
MediaLogEvent::Type are handled in cases in this switch.

+      return "UNKNOWN_EVENT_TYPE!!";
+  }
+  NOTREACHED();
+  return NULL;
+}
+
 LogHelper::LogHelper(const LogCB& log_cb) : log_cb_(log_cb) {}
 
 LogHelper::~LogHelper() {
   if (log_cb_.is_null())
     return;
   log_cb_.Run(stream_.str());
 }
 
 MediaLog::MediaLog() : id_(g_media_log_count.GetNext()) {}
 
@@ skipping 126 matching lines @@
     const std::string& key, base::TimeDelta value) {
   scoped_ptr<MediaLogEvent> event(CreateEvent(MediaLogEvent::PROPERTY_CHANGE));
   if (value.is_max())
     event->params.SetString(key, "unknown");
   else
     event->params.SetDouble(key, value.InSecondsF());
   AddEvent(event.Pass());
 }
 
 }  //namespace media