Remove MediaRouterAction

Desktop / Chromium - Takumi Fujimoto [chromium.org] - 22 February 2019 22:11 EST

The new CastToolbarButton in the toolbar's trusted area is now enabled by default, so this CL removes the old Media Router toolbar action that was replaced by it.

MediaRouterActionController is still used by the new icon, and will be renamed in a separate patch.

Bug: 919185 Change-Id: I95842e657379f823bd483d0a3da00015f76d8255 Reviewed-on: https://chromium-review.googlesource.com/c/1395419

906aab7c6bbe Remove MediaRouterAction
chrome/browser/policy/policy_browsertest.cc | 17 +-
chrome/browser/ui/BUILD.gn | 2 -
.../media_router_dialog_controller_impl_base.cc | 19 +-
.../media_router_dialog_controller_impl_base.h | 16 -
.../toolbar/component_toolbar_actions_factory.cc | 24 +-
.../ui/toolbar/component_toolbar_actions_factory.h | 6 +-
chrome/browser/ui/toolbar/media_router_action.cc | 332 ---------------
chrome/browser/ui/toolbar/media_router_action.h | 156 --------
.../ui/toolbar/media_router_action_controller.cc | 35 +-
.../ui/toolbar/media_router_action_controller.h | 34 +-
.../media_router_action_controller_unittest.cc | 47 +--
.../ui/toolbar/media_router_action_unittest.cc | 348 ----------------
.../ui/toolbar/media_router_contextual_menu.cc | 55 +--
.../ui/toolbar/media_router_contextual_menu.h | 26 +-
.../media_router_contextual_menu_unittest.cc | 53 +--
.../toolbar/mock_media_router_action_controller.cc | 6 +-
.../ui/views/media_router/cast_toolbar_button.cc | 2 +-
.../media_router/cast_toolbar_button_unittest.cc | 2 +-
.../media_router_dialog_controller_views.cc | 1 -
.../media_router/media_router_ui_browsertest.cc | 443 ++++++---------------
chrome/browser/ui/views/toolbar/toolbar_view.cc | 4 +-
.../media_router_dialog_controller_webui_impl.cc | 2 -
chrome/test/BUILD.gn | 1 -
.../media_router_integration_browsertest.h | 1 -
24 files changed, 185 insertions(+), 1447 deletions(-)

Upstream: git.chromium.org


  • Share