| Index: media/blink/webmediaplayer_impl.cc
|
| diff --git a/media/blink/webmediaplayer_impl.cc b/media/blink/webmediaplayer_impl.cc
|
| index 4bbc1aad4b2d4173e780141d84cf6483d564ffc3..4127fba19f4e0b98416be476f33d64d379fa81ac 100644
|
| --- a/media/blink/webmediaplayer_impl.cc
|
| +++ b/media/blink/webmediaplayer_impl.cc
|
| @@ -174,9 +174,8 @@ WebMediaPlayerImpl::WebMediaPlayerImpl(
|
| network_state_(WebMediaPlayer::NetworkStateEmpty),
|
| ready_state_(WebMediaPlayer::ReadyStateHaveNothing),
|
| highest_ready_state_(WebMediaPlayer::ReadyStateHaveNothing),
|
| - preload_(BufferedDataSource::AUTO),
|
| - buffering_strategy_(
|
| - BufferedDataSourceInterface::BUFFERING_STRATEGY_NORMAL),
|
| + preload_(MultibufferDataSource::AUTO),
|
| + buffering_strategy_(MultibufferDataSource::BUFFERING_STRATEGY_NORMAL),
|
| main_task_runner_(base::ThreadTaskRunnerHandle::Get()),
|
| media_task_runner_(params.media_task_runner()),
|
| worker_task_runner_(params.worker_task_runner()),
|
| @@ -363,24 +362,10 @@ void WebMediaPlayerImpl::DoLoad(LoadType load_type,
|
| supports_save_ = false;
|
| StartPipeline();
|
| } else {
|
| - // TODO(hubbe): This experiment is temporary and should be removed once
|
| - // we have enough data to support the primacy of the new media cache.
|
| - // See http://crbug.com/514719 for details.
|
| - // Otherwise it's a regular request which requires resolving the URL first.
|
| - if (base::FeatureList::IsEnabled(kUseNewMediaCache)) {
|
| - // Remove this when MultiBufferDataSource becomes default.
|
| - LOG(WARNING) << "Using MultibufferDataSource";
|
| - data_source_.reset(new MultibufferDataSource(
|
| - url, static_cast<UrlData::CORSMode>(cors_mode), main_task_runner_,
|
| - url_index_, frame_, media_log_.get(), &buffered_data_source_host_,
|
| - base::Bind(&WebMediaPlayerImpl::NotifyDownloading, AsWeakPtr())));
|
| - } else {
|
| - data_source_.reset(new BufferedDataSource(
|
| - url, static_cast<BufferedResourceLoader::CORSMode>(cors_mode),
|
| - main_task_runner_, frame_, media_log_.get(),
|
| - &buffered_data_source_host_,
|
| - base::Bind(&WebMediaPlayerImpl::NotifyDownloading, AsWeakPtr())));
|
| - }
|
| + data_source_.reset(new MultibufferDataSource(
|
| + url, static_cast<UrlData::CORSMode>(cors_mode), main_task_runner_,
|
| + url_index_, frame_, media_log_.get(), &buffered_data_source_host_,
|
| + base::Bind(&WebMediaPlayerImpl::NotifyDownloading, AsWeakPtr())));
|
| data_source_->SetPreload(preload_);
|
| data_source_->SetBufferingStrategy(buffering_strategy_);
|
| data_source_->Initialize(
|
| @@ -563,24 +548,24 @@ void WebMediaPlayerImpl::setSinkId(
|
| callback));
|
| }
|
|
|
| -STATIC_ASSERT_ENUM(WebMediaPlayer::PreloadNone, BufferedDataSource::NONE);
|
| +STATIC_ASSERT_ENUM(WebMediaPlayer::PreloadNone, MultibufferDataSource::NONE);
|
| STATIC_ASSERT_ENUM(WebMediaPlayer::PreloadMetaData,
|
| - BufferedDataSource::METADATA);
|
| -STATIC_ASSERT_ENUM(WebMediaPlayer::PreloadAuto, BufferedDataSource::AUTO);
|
| + MultibufferDataSource::METADATA);
|
| +STATIC_ASSERT_ENUM(WebMediaPlayer::PreloadAuto, MultibufferDataSource::AUTO);
|
|
|
| void WebMediaPlayerImpl::setPreload(WebMediaPlayer::Preload preload) {
|
| DVLOG(1) << __func__ << "(" << preload << ")";
|
| DCHECK(main_task_runner_->BelongsToCurrentThread());
|
|
|
| - preload_ = static_cast<BufferedDataSource::Preload>(preload);
|
| + preload_ = static_cast<MultibufferDataSource::Preload>(preload);
|
| if (data_source_)
|
| data_source_->SetPreload(preload_);
|
| }
|
|
|
| STATIC_ASSERT_ENUM(WebMediaPlayer::BufferingStrategy::Normal,
|
| - BufferedDataSource::BUFFERING_STRATEGY_NORMAL);
|
| + MultibufferDataSource::BUFFERING_STRATEGY_NORMAL);
|
| STATIC_ASSERT_ENUM(WebMediaPlayer::BufferingStrategy::Aggressive,
|
| - BufferedDataSource::BUFFERING_STRATEGY_AGGRESSIVE);
|
| + MultibufferDataSource::BUFFERING_STRATEGY_AGGRESSIVE);
|
|
|
| void WebMediaPlayerImpl::setBufferingStrategy(
|
| WebMediaPlayer::BufferingStrategy buffering_strategy) {
|
| @@ -592,10 +577,10 @@ void WebMediaPlayerImpl::setBufferingStrategy(
|
| // of the platform media player and may have data usage penalties.
|
| // TODO(dalecurtis, hubbe): We should probably stop using "pause-and-buffer"
|
| // everywhere. See http://crbug.com/594669 for more details.
|
| - buffering_strategy_ = BufferedDataSource::BUFFERING_STRATEGY_NORMAL;
|
| + buffering_strategy_ = MultibufferDataSource::BUFFERING_STRATEGY_NORMAL;
|
| #else
|
| buffering_strategy_ =
|
| - static_cast<BufferedDataSource::BufferingStrategy>(buffering_strategy);
|
| + static_cast<MultibufferDataSource::BufferingStrategy>(buffering_strategy);
|
| #endif
|
|
|
| if (data_source_)
|
| @@ -1000,7 +985,7 @@ void WebMediaPlayerImpl::OnPipelineSuspended() {
|
| // If we're not in an aggressive buffering state, tell the data source we have
|
| // enough data so that it may release the connection.
|
| if (buffering_strategy_ !=
|
| - BufferedDataSource::BUFFERING_STRATEGY_AGGRESSIVE) {
|
| + MultibufferDataSource::BUFFERING_STRATEGY_AGGRESSIVE) {
|
| if (data_source_)
|
| data_source_->OnBufferingHaveEnough(true);
|
| }
|
|
|