Commit 37f2b6ec authored by Jennifer Apacible's avatar Jennifer Apacible Committed by Commit Bot

[Picture in Picture] Show correct media controls on window open.

Currently, the initial media control state for play/pause is incorrect.
It shows the opposite control than intended.

Tests forthcoming in a future patch once we have test infrastructure
set up.

BUG: 836389
Change-Id: Ie7be9bfb384f62ac925d6cb1c1ee30cc6baf2cfc
Reviewed-on: https://chromium-review.googlesource.com/1060244Reviewed-by: default avatarBecca Hughes <beccahughes@chromium.org>
Commit-Queue: apacible <apacible@chromium.org>
Cr-Commit-Position: refs/heads/master@{#558894}
parent 3d7caa70
...@@ -214,7 +214,7 @@ void OverlayWindowViews::SetUpViews() { ...@@ -214,7 +214,7 @@ void OverlayWindowViews::SetUpViews() {
kPlayPauseIconSize.width(), SK_ColorWHITE); kPlayPauseIconSize.width(), SK_ColorWHITE);
play_pause_controls_view_->SetToggledImage(views::Button::STATE_NORMAL, play_pause_controls_view_->SetToggledImage(views::Button::STATE_NORMAL,
&pause_icon); &pause_icon);
play_pause_controls_view_->SetToggled(controller_->IsPlayerActive()); play_pause_controls_view_->SetToggled(!controller_->IsPlayerActive());
// Paint to ui::Layers to use in the OverlaySurfaceEmbedder. // Paint to ui::Layers to use in the OverlaySurfaceEmbedder.
video_view_->SetPaintToLayer(ui::LAYER_TEXTURED); video_view_->SetPaintToLayer(ui::LAYER_TEXTURED);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment