New issue
Advanced search Search tips

Issue 820277 link

Starred by 1 user

Issue metadata

Status: Assigned
Owner:
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug

Blocking:
issue 790766



Sign in to add a comment

Merge GetRouteController and GetMediaController APIs

Project Member Reported by tguilbert@chromium.org, Mar 8 2018

Issue description

Cc: -dcheng@chromium.org imch...@chromium.org
This bug will also track separating the core MediaStatus/MediaStatusObserver interface from the Mirorring/Hangouts extra data, in media_status.mojom
Project Member

Comment 3 by bugdroid1@chromium.org, Jul 12

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/2784a2c6630a8de538a7b2bf6d993823d321a489

commit 2784a2c6630a8de538a7b2bf6d993823d321a489
Author: Thomas Guilbert <tguilbert@chromium.org>
Date: Thu Jul 12 20:02:40 2018

Move content::MediaController to media/base

The MediaController interface and the upcoming MediaStatusObserver do
not have to live in content/public/browser. The media/base folder and
the media namespace is a more generic place for them to live, even if
most of the code that will use them lives in content.

Bug: 820277
Change-Id: I2f91cd88c6f5238723aae8aeff1c323c02f738d4
Reviewed-on: https://chromium-review.googlesource.com/1134090
Reviewed-by: Scott Violet <sky@chromium.org>
Commit-Queue: Thomas Guilbert <tguilbert@chromium.org>
Cr-Commit-Position: refs/heads/master@{#574690}
[modify] https://crrev.com/2784a2c6630a8de538a7b2bf6d993823d321a489/chrome/browser/media/android/remote/media_controller_bridge.h
[modify] https://crrev.com/2784a2c6630a8de538a7b2bf6d993823d321a489/chrome/browser/media/android/router/media_router_android.cc
[modify] https://crrev.com/2784a2c6630a8de538a7b2bf6d993823d321a489/chrome/browser/media/android/router/media_router_android.h
[modify] https://crrev.com/2784a2c6630a8de538a7b2bf6d993823d321a489/chrome/browser/media/android/router/media_router_android_bridge.cc
[modify] https://crrev.com/2784a2c6630a8de538a7b2bf6d993823d321a489/chrome/browser/media/android/router/media_router_android_bridge.h
[modify] https://crrev.com/2784a2c6630a8de538a7b2bf6d993823d321a489/chrome/browser/media/router/media_router.h
[modify] https://crrev.com/2784a2c6630a8de538a7b2bf6d993823d321a489/chrome/browser/media/router/media_router_base.cc
[modify] https://crrev.com/2784a2c6630a8de538a7b2bf6d993823d321a489/chrome/browser/media/router/media_router_base.h
[modify] https://crrev.com/2784a2c6630a8de538a7b2bf6d993823d321a489/chrome/browser/media/router/presentation/presentation_service_delegate_impl.cc
[modify] https://crrev.com/2784a2c6630a8de538a7b2bf6d993823d321a489/chrome/browser/media/router/presentation/presentation_service_delegate_impl.h
[modify] https://crrev.com/2784a2c6630a8de538a7b2bf6d993823d321a489/content/browser/media/flinging_renderer.cc
[modify] https://crrev.com/2784a2c6630a8de538a7b2bf6d993823d321a489/content/browser/media/flinging_renderer.h
[modify] https://crrev.com/2784a2c6630a8de538a7b2bf6d993823d321a489/content/browser/media/flinging_renderer_unittest.cc
[modify] https://crrev.com/2784a2c6630a8de538a7b2bf6d993823d321a489/content/browser/presentation/presentation_service_impl_unittest.cc
[modify] https://crrev.com/2784a2c6630a8de538a7b2bf6d993823d321a489/content/public/browser/BUILD.gn
[modify] https://crrev.com/2784a2c6630a8de538a7b2bf6d993823d321a489/content/public/browser/presentation_service_delegate.h
[modify] https://crrev.com/2784a2c6630a8de538a7b2bf6d993823d321a489/media/base/BUILD.gn
[rename] https://crrev.com/2784a2c6630a8de538a7b2bf6d993823d321a489/media/base/media_controller.h

Status: Assigned (was: Available)
Labels: Videostack-RemotePlayback
Cc: -imch...@chromium.org

Sign in to add a comment