Commit 138363ff authored by Becca Hughes's avatar Becca Hughes Committed by Commit Bot

Media Controls: Re-create the play button

As per steimel@'s comment in crrev.com/c/991293 we should always
create the play button as we use it when the video element is
very small.

BUG=821961,821414

Change-Id: Icec781d0cd49529c72dd21c88d54727e81236890
Reviewed-on: https://chromium-review.googlesource.com/1000722
Commit-Queue: Becca Hughes <beccahughes@chromium.org>
Reviewed-by: default avatarMounir Lamouri <mlamouri@chromium.org>
Cr-Commit-Position: refs/heads/master@{#549266}
parent eb70ffc0
...@@ -601,13 +601,7 @@ void MediaControlsImpl::PopulatePanel() { ...@@ -601,13 +601,7 @@ void MediaControlsImpl::PopulatePanel() {
button_panel = media_button_panel_; button_panel = media_button_panel_;
} }
// The play button should only be on the button panel if we are playing audio button_panel->ParserAppendChild(play_button_);
// only or are using the legacy controls.
if (!IsModern() || is_acting_as_audio_controls_ ||
MediaElement().IsHTMLAudioElement()) {
button_panel->ParserAppendChild(play_button_);
}
button_panel->ParserAppendChild(current_time_display_); button_panel->ParserAppendChild(current_time_display_);
button_panel->ParserAppendChild(duration_display_); button_panel->ParserAppendChild(duration_display_);
......
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