Commit 72909afd authored by CJ DiMeglio's avatar CJ DiMeglio Committed by Commit Bot

Passes opacity information on EnableSubmission.

We had incorrectly obmitted an update to the opacity when we first start
sending frames. This CL adds that necessary call, and also neatens up
the logic to update opacity and rotation, making the effect of these
calls clearer.

Bug: 893056
Change-Id: I1b6edb4292f1a3a25aa22b61e82c20381d395eb7
Reviewed-on: https://chromium-review.googlesource.com/c/1275525
Commit-Queue: CJ DiMeglio <lethalantidote@chromium.org>
Reviewed-by: default avatarEmircan Uysaler <emircan@chromium.org>
Cr-Commit-Position: refs/heads/master@{#602112}
parent 5adeafff
...@@ -1112,11 +1112,12 @@ void WebMediaPlayerMS::OnFirstFrameReceived(media::VideoRotation video_rotation, ...@@ -1112,11 +1112,12 @@ void WebMediaPlayerMS::OnFirstFrameReceived(media::VideoRotation video_rotation,
DVLOG(1) << __func__; DVLOG(1) << __func__;
DCHECK(thread_checker_.CalledOnValidThread()); DCHECK(thread_checker_.CalledOnValidThread());
OnRotationChanged(video_rotation);
OnOpacityChanged(is_opaque);
if (surface_layer_mode_ == blink::WebMediaPlayer::SurfaceLayerMode::kAlways) { if (surface_layer_mode_ == blink::WebMediaPlayer::SurfaceLayerMode::kAlways) {
ActivateSurfaceLayerForVideo(); ActivateSurfaceLayerForVideo();
} }
OnRotationChanged(video_rotation);
OnOpacityChanged(is_opaque);
SetReadyState(WebMediaPlayer::kReadyStateHaveMetadata); SetReadyState(WebMediaPlayer::kReadyStateHaveMetadata);
SetReadyState(WebMediaPlayer::kReadyStateHaveEnoughData); SetReadyState(WebMediaPlayer::kReadyStateHaveEnoughData);
......
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