Change ownership of PipelineImpl

media/filters/pipeline_controller.cc

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "media/filters/pipeline_controller.h"
 
 #include "base/bind.h"
 #include "media/base/demuxer.h"
 
 namespace media {
 
 PipelineController::PipelineController(
-    Pipeline* pipeline,
+    std::unique_ptr<Pipeline> pipeline,
     const RendererFactoryCB& renderer_factory_cb,
     const SeekedCB& seeked_cb,
     const SuspendedCB& suspended_cb,
     const BeforeResumeCB& before_resume_cb,
     const ResumedCB& resumed_cb,
     const PipelineStatusCB& error_cb)
-    : pipeline_(pipeline),
+    : pipeline_(std::move(pipeline)),
       renderer_factory_cb_(renderer_factory_cb),
       seeked_cb_(seeked_cb),
       suspended_cb_(suspended_cb),
       before_resume_cb_(before_resume_cb),
       resumed_cb_(resumed_cb),
       error_cb_(error_cb),
       weak_factory_(this) {
   DCHECK(pipeline_);
   DCHECK(!renderer_factory_cb_.is_null());
   DCHECK(!seeked_cb_.is_null());
   DCHECK(!suspended_cb_.is_null());
   DCHECK(!before_resume_cb_.is_null());
   DCHECK(!resumed_cb_.is_null());
   DCHECK(!error_cb_.is_null());
 }
 
 PipelineController::~PipelineController() {
   DCHECK(thread_checker_.CalledOnValidThread());
 }
 
-// TODO(sandersd): If there is a pending suspend, don't call pipeline_.Start()
+// TODO(sandersd): If there is a pending suspend, don't call pipeline_->Start()
 // until Resume().
 void PipelineController::Start(Demuxer* demuxer,
                                Pipeline::Client* client,
                                bool is_streaming,
                                bool is_static) {
   DCHECK(thread_checker_.CalledOnValidThread());
   DCHECK(state_ == State::CREATED);
   DCHECK(demuxer);
 
   // Once the pipeline is started, we want to call the seeked callback but
@@ skipping 187 matching lines @@
       // immediately.
       pending_seeked_cb_ = false;
       bool was_pending_time_updated = pending_time_updated_;
       pending_time_updated_ = false;
       seeked_cb_.Run(was_pending_time_updated);
       return;
     }
   }
 }
 
+void PipelineController::Stop() {
+  pipeline_->Stop();

[sandersd (OOO until July 31), 2017/02/23 23:11:46]

Should we perhaps be resetting |state_| here?

[tguilbert, 2017/02/24 02:13:59]

I added a comment, a TODO, and opened crbug.com/695734 to track the work
officially.

+}
+
+bool PipelineController::IsPipelineRunning() const {
+  return pipeline_->IsRunning();
+}
+
+double PipelineController::GetPlaybackRate() const {
+  return pipeline_->GetPlaybackRate();
+}
+
+void PipelineController::SetPlaybackRate(double playback_rate) {
+  pipeline_->SetPlaybackRate(playback_rate);
+}
+
+float PipelineController::GetVolume() const {
+  return pipeline_->GetVolume();
+}
+
+void PipelineController::SetVolume(float volume) {
+  pipeline_->SetVolume(volume);
+}
+
+base::TimeDelta PipelineController::GetMediaTime() const {
+  return pipeline_->GetMediaTime();
+}
+
+Ranges<base::TimeDelta> PipelineController::GetBufferedTimeRanges() const {
+  return pipeline_->GetBufferedTimeRanges();
+}
+
+base::TimeDelta PipelineController::GetMediaDuration() const {
+  return pipeline_->GetMediaDuration();
+}
+
+bool PipelineController::DidLoadingProgress() {
+  return pipeline_->DidLoadingProgress();
+}
+
+PipelineStatistics PipelineController::GetStatistics() const {
+  return pipeline_->GetStatistics();
+}
+
+void PipelineController::SetCdm(CdmContext* cdm_context,
+                                const CdmAttachedCB& cdm_attached_cb) {
+  pipeline_->SetCdm(cdm_context, cdm_attached_cb);
+}
+
+void PipelineController::OnEnabledAudioTracksChanged(
+    const std::vector<MediaTrack::Id>& enabledTrackIds) {
+  pipeline_->OnEnabledAudioTracksChanged(enabledTrackIds);
+}
+
+void PipelineController::OnSelectedVideoTrackChanged(
+    const std::vector<MediaTrack::Id>& selectedTrackId) {
+  pipeline_->OnSelectedVideoTrackChanged(selectedTrackId);
+}
+
 }  // namespace media