Commit 516ef6d9 authored by xjz's avatar xjz Committed by Commit bot

Inform MediaObserver when video natural size changes.

BUG=678339

Review-Url: https://codereview.chromium.org/2620433002
Cr-Commit-Position: refs/heads/master@{#442107}
parent df4f7e43
...@@ -1312,14 +1312,20 @@ void WebMediaPlayerImpl::OnVideoNaturalSizeChange(const gfx::Size& size) { ...@@ -1312,14 +1312,20 @@ void WebMediaPlayerImpl::OnVideoNaturalSizeChange(const gfx::Size& size) {
if (overlay_enabled_ && surface_manager_) if (overlay_enabled_ && surface_manager_)
surface_manager_->NaturalSizeChanged(rotated_size); surface_manager_->NaturalSizeChanged(rotated_size);
if (pipeline_metadata_.natural_size.IsEmpty()) { gfx::Size old_size = pipeline_metadata_.natural_size;
pipeline_metadata_.natural_size = rotated_size;
if (old_size.IsEmpty()) {
// WatchTimeReporter doesn't report metrics for empty videos. Re-create // WatchTimeReporter doesn't report metrics for empty videos. Re-create
// |watch_time_reporter_| if we didn't originally know the video size. // |watch_time_reporter_| if we didn't originally know the video size.
CreateWatchTimeReporter(); CreateWatchTimeReporter();
} }
pipeline_metadata_.natural_size = rotated_size;
client_->sizeChanged(); client_->sizeChanged();
if (observer_) {
PipelineMetadata metadata = pipeline_metadata_;
metadata.natural_size = size;
observer_->OnMetadataChanged(metadata);
}
} }
void WebMediaPlayerImpl::OnVideoOpacityChange(bool opaque) { void WebMediaPlayerImpl::OnVideoOpacityChange(bool opaque) {
......
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