Add a media log event for SUSPENDED for clarity on internals page.

This causes the summary for a player to show up as "SUSPENDED"
when looking at the list of players in chrome://media-internals.
Making it easier to see which players are active and which are
now suspended.

Change-Id: I142efaaec97f533488831b9e26d77a95283c28c6
Reviewed-on: https://chromium-review.googlesource.com/c/1357615Reviewed-by: Thomas Guilbert <tguilbert@chromium.org>
Commit-Queue: Dale Curtis <dalecurtis@chromium.org>
Cr-Commit-Position: refs/heads/master@{#612890}

media/base/media_log.cc

@@ -84,6 +84,8 @@ std::string MediaLog::EventTypeToString(MediaLogEvent::Type type) {
       return "MEDIA_DEBUG_LOG_ENTRY";
     case MediaLogEvent::PROPERTY_CHANGE:
       return "PROPERTY_CHANGE";
+    case MediaLogEvent::SUSPENDED:
+      return "SUSPENDED";
   }
   NOTREACHED();
   return NULL;

media/base/media_log_event.h

@@ -89,7 +89,10 @@ struct MediaLogEvent {
     // A property has changed without any special event occurring.
     PROPERTY_CHANGE,
 
-    TYPE_LAST = PROPERTY_CHANGE
+    // Issued when a player is suspended.
+    SUSPENDED,
+
+    TYPE_LAST = SUSPENDED
   };
 
   int32_t id;

media/blink/webmediaplayer_impl.cc

@@ -1509,6 +1509,9 @@ void WebMediaPlayerImpl::OnPipelineSeeked(bool time_updated) {
 }
 
 void WebMediaPlayerImpl::OnPipelineSuspended() {
+  // Add a log event so the player shows up as "SUSPENDED" in media-internals.
+  media_log_->AddEvent(media_log_->CreateEvent(MediaLogEvent::SUSPENDED));
+
 #if defined(OS_ANDROID)
   if (IsRemote() && !IsNewRemotePlaybackPipelineEnabled()) {
     scoped_refptr<VideoFrame> frame = cast_impl_.GetCastingBanner();