Add PipelineStatus UMA logging to media_internals

media/base/media_log.h

Index: media/base/media_log.h
diff --git a/media/base/media_log.h b/media/base/media_log.h
index f342ee84fc44185d8590c3ce5c6c2700a7904d45..79f3a70d85540cece3d055cb4e939bdb84093d06 100644
--- a/media/base/media_log.h
+++ b/media/base/media_log.h
@@ -41,7 +41,7 @@ class MEDIA_EXPORT MediaLog : public base::RefCountedThreadSafe<MediaLog> {
   // Convert various enums to strings.
   static const char* EventTypeToString(MediaLogEvent::Type type);
   static const char* PipelineStatusToString(PipelineStatus);
-
+  static bool StringToPipelineStatus(const std::string&, PipelineStatus&);

[DaleCurtis, 2014/11/06 23:49:08]

No need for & for primitive type like status. Both parameters need names too,
can you fix the one above as well?

[prabhur1, 2014/11/07 23:43:24]

Done.

[DaleCurtis, 2014/11/11 20:48:58]

This is backwards of what I meant, std::string should be const&, while
PipelineStatus should be a pointer. We don't use non-const & in functions.

[prabhur1, 2014/11/12 21:32:37]

Done.

   MediaLog();
 
   // Add an event to this log. Overriden by inheritors to actually do something