Split MediaContentType and AudioFocusType

content/renderer/media/webmediaplayer_ms.cc

Index: content/renderer/media/webmediaplayer_ms.cc
diff --git a/content/renderer/media/webmediaplayer_ms.cc b/content/renderer/media/webmediaplayer_ms.cc
index d437247d18053d97a778c326d8bc707e6b520d81..e0d998d6d49949b3e5f3b200925166a1180f1d0c 100644
--- a/content/renderer/media/webmediaplayer_ms.cc
+++ b/content/renderer/media/webmediaplayer_ms.cc
@@ -23,6 +23,7 @@
 #include "content/renderer/media/webmediaplayer_ms_compositor.h"
 #include "content/renderer/render_frame_impl.h"
 #include "content/renderer/render_thread_impl.h"
+#include "media/base/media_content_type.h"
 #include "media/base/media_log.h"
 #include "media/base/video_frame.h"
 #include "media/blink/webmediaplayer_util.h"
@@ -181,7 +182,7 @@ void WebMediaPlayerMS::play() {
     // here, but that is treated as an unknown duration and assumed to be
     // interactive. See http://crbug.com/595297 for more details.
     delegate_->DidPlay(delegate_id_, hasVideo(), hasAudio(), false,
-                       base::TimeDelta::FromSeconds(1));
+                       media::MediaContentType::Uncontrollable);

[mlamouri (slow - plz ping), 2016/08/18 14:33:15]

Should this be transient to make sure the behaviour doesn't change? We can then
ping perkj@ and magjed@ from the TODO to pick the behaviour they would prefer.

[Zhiqiang Zhang (Slow), 2016/08/18 17:31:45]

Changed the logic in MediaSession, so that all types other than CONTENT will be
treated as TRANSIENT.
Let's leave it for now and refine later.

   }
 
   paused_ = false;