[MediaSession] Keep MediaSession controllable when losing audio focus

content/browser/media/session/audio_focus_delegate_android.cc

Index: content/browser/media/session/audio_focus_delegate_android.cc
diff --git a/content/browser/media/session/audio_focus_delegate_android.cc b/content/browser/media/session/audio_focus_delegate_android.cc
index dfaba7d1588f001685927deaef243d030285d94d..ae8e42578354ec0ed73a77e0a962959d623e8b79 100644
--- a/content/browser/media/session/audio_focus_delegate_android.cc
+++ b/content/browser/media/session/audio_focus_delegate_android.cc
@@ -61,11 +61,9 @@ void AudioFocusDelegateAndroid::OnSuspend(JNIEnv*,
   if (!media_session_->IsActive())
     return;
 
-  if (temporary) {
-    media_session_->Suspend(MediaSession::SuspendType::SYSTEM);
-  } else {
-    media_session_->Stop(MediaSession::SuspendType::SYSTEM);

[mlamouri (slow - plz ping), 2017/01/09 14:33:37]

Do we still use `Stop()` somewhere?

[Zhiqiang Zhang (Slow), 2017/01/09 18:11:59]

Yes, but Stop() is now only called with SuspendType::UI (from the notification).
Maybe we should also remove the SuspendType argument from Stop().

-  }
+  // Suspending the session instead of stopping regardless of |temporary| to

[mlamouri (slow - plz ping), 2017/01/09 14:33:37]

Could you remove `temporary` from the method if it's not used? :)

[Zhiqiang Zhang (Slow), 2017/01/09 18:11:59]

Done.

+  // keep the session controllable.
+  media_session_->Suspend(MediaSession::SuspendType::SYSTEM);
 }
 
 void AudioFocusDelegateAndroid::OnResume(JNIEnv*,