| Index: content/browser/media/session/media_session_service_impl.cc
|
| diff --git a/content/browser/media/session/media_session_service_impl.cc b/content/browser/media/session/media_session_service_impl.cc
|
| index f0f5431c2300325fc6dee976a70fdfd53b47f572..589823fab2b78f83bf7575e5044575a57642f1eb 100644
|
| --- a/content/browser/media/session/media_session_service_impl.cc
|
| +++ b/content/browser/media/session/media_session_service_impl.cc
|
| @@ -14,9 +14,17 @@ namespace content {
|
|
|
| MediaSessionServiceImpl::MediaSessionServiceImpl(
|
| RenderFrameHost* render_frame_host)
|
| - : render_frame_host_(render_frame_host) {}
|
| + : render_frame_host_(render_frame_host) {
|
| + MediaSessionImpl* session = GetMediaSession();
|
| + if (session)
|
| + session->SetMediaSessionService(this);
|
| +}
|
|
|
| -MediaSessionServiceImpl::~MediaSessionServiceImpl() = default;
|
| +MediaSessionServiceImpl::~MediaSessionServiceImpl() {
|
| + MediaSessionImpl* session = GetMediaSession();
|
| + if (session && session->GetMediaSessionService() == this)
|
| + session->SetMediaSessionService(nullptr);
|
| +}
|
|
|
| // static
|
| void MediaSessionServiceImpl::Create(
|
| @@ -43,22 +51,33 @@ void MediaSessionServiceImpl::SetMetadata(
|
| return;
|
| }
|
|
|
| - WebContentsImpl* contents = static_cast<WebContentsImpl*>(
|
| - WebContentsImpl::FromRenderFrameHost(render_frame_host_));
|
| - if (contents)
|
| - MediaSessionImpl::Get(contents)->SetMetadata(metadata);
|
| + MediaSessionImpl* session = GetMediaSession();
|
| + if (session)
|
| + session->SetMetadata(metadata);
|
| }
|
|
|
| void MediaSessionServiceImpl::EnableAction(
|
| blink::mojom::MediaSessionAction action) {
|
| - // TODO(zqzhang): Plumb this signal to Java. See https://crbug.com/656563
|
| - NOTIMPLEMENTED();
|
| + MediaSessionImpl* session = GetMediaSession();
|
| + if (session)
|
| + session->OnMediaSessionEnabledAction(action);
|
| }
|
|
|
| void MediaSessionServiceImpl::DisableAction(
|
| blink::mojom::MediaSessionAction action) {
|
| - // TODO(zqzhang): Plumb this signal to Java. See https://crbug.com/656563
|
| - NOTIMPLEMENTED();
|
| + MediaSessionImpl* session = GetMediaSession();
|
| + if (session)
|
| + session->OnMediaSessionDisabledAction(action);
|
| +}
|
| +
|
| +MediaSessionImpl* MediaSessionServiceImpl::GetMediaSession() {
|
| + WebContentsImpl* contents = static_cast<WebContentsImpl*>(
|
| + WebContentsImpl::FromRenderFrameHost(render_frame_host_));
|
| + if (!contents)
|
| + return nullptr;
|
| + if (render_frame_host_ != contents->GetMainFrame())
|
| + return nullptr;
|
| + return MediaSessionImpl::Get(contents);
|
| }
|
|
|
| void MediaSessionServiceImpl::Bind(
|
|
|