Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(670)

Unified Diff: media/blink/webmediaplayer_impl.cc

Issue 2611783002: Turn off video overlay on android for rotated video. (Closed)
Patch Set: fixed |force_video_overlays_| Created 3 years, 12 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « media/blink/webmediaplayer_impl.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/blink/webmediaplayer_impl.cc
diff --git a/media/blink/webmediaplayer_impl.cc b/media/blink/webmediaplayer_impl.cc
index 654ba89050299e2a31ec49243c759482815edc6c..c6b8b571cea6a261dc31fc9e629025bf7185660a 100644
--- a/media/blink/webmediaplayer_impl.cc
+++ b/media/blink/webmediaplayer_impl.cc
@@ -346,14 +346,21 @@ void WebMediaPlayerImpl::DisableOverlay() {
}
void WebMediaPlayerImpl::enteredFullscreen() {
- if (!force_video_overlays_ && enable_fullscreen_video_overlays_)
+ // |force_video_overlays_| implies that we're already in overlay mode, so take
+ // no action here. Otherwise, switch to an overlay if it's allowed and if
+ // it will display properly.
+ if (!force_video_overlays_ && enable_fullscreen_video_overlays_ &&
+ DoesOverlaySupportMetadata()) {
EnableOverlay();
+ }
if (observer_)
observer_->OnEnteredFullscreen();
}
void WebMediaPlayerImpl::exitedFullscreen() {
- if (!force_video_overlays_ && enable_fullscreen_video_overlays_)
+ // If we're in overlay mode, then exit it unless we're supposed to be in
+ // overlay mode all the time.
+ if (!force_video_overlays_ && overlay_enabled_)
DisableOverlay();
if (observer_)
observer_->OnExitedFullscreen();
@@ -1155,8 +1162,15 @@ void WebMediaPlayerImpl::OnMetadata(PipelineMetadata metadata) {
pipeline_metadata_.natural_size = GetRotatedVideoSize(
pipeline_metadata_.video_rotation, pipeline_metadata_.natural_size);
- if (overlay_enabled_ && surface_manager_)
- surface_manager_->NaturalSizeChanged(pipeline_metadata_.natural_size);
+ if (overlay_enabled_) {
+ // SurfaceView doesn't support rotated video, so transition back if
+ // the video is now rotated. If |force_video_overlays_|, we keep the
+ // overlay anyway so that the state machine keeps working.
+ if (!force_video_overlays_ && !DoesOverlaySupportMetadata())
+ DisableOverlay();
+ else if (surface_manager_)
+ surface_manager_->NaturalSizeChanged(pipeline_metadata_.natural_size);
+ }
DCHECK(!video_weblayer_);
video_weblayer_.reset(new cc_blink::WebLayerImpl(cc::VideoLayer::Create(
@@ -2011,6 +2025,10 @@ bool WebMediaPlayerImpl::IsHidden() const {
return delegate_ && delegate_->IsHidden();
}
+bool WebMediaPlayerImpl::DoesOverlaySupportMetadata() const {
+ return pipeline_metadata_.video_rotation == VIDEO_ROTATION_0;
+}
+
void WebMediaPlayerImpl::ActivateViewportIntersectionMonitoring(bool activate) {
DCHECK(main_task_runner_->BelongsToCurrentThread());
« no previous file with comments | « media/blink/webmediaplayer_impl.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698