Chromecast: change BrowserCdmCast threading model.

chromecast/media/cma/pipeline/media_pipeline_impl.cc

 // Copyright 2014 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 "chromecast/media/cma/pipeline/media_pipeline_impl.h"
 
 #include "base/bind.h"
 #include "base/callback.h"
 #include "base/callback_helpers.h"
 #include "base/location.h"
@@ skipping 95 matching lines @@
 
 void MediaPipelineImpl::SetCdm(int cdm_id) {
   CMALOG(kLogControl) << __FUNCTION__ << " cdm_id=" << cdm_id;
   DCHECK(thread_checker_.CalledOnValidThread());
   NOTIMPLEMENTED();
   // TODO(gunsch): SetCdm(int) is not implemented.
   // One possibility would be a GetCdmByIdCB that's passed in.
 }
 
 void MediaPipelineImpl::SetCdm(::media::BrowserCdm* media_keys) {
+  DCHECK(thread_checker_.CalledOnValidThread());

[erickung1, 2015/02/07 01:31:10]

Very NIT: may switch line with CMALOG to follow other function implementation

[gunsch, 2015/02/07 01:44:27]

Done.

   CMALOG(kLogControl) << __FUNCTION__;
   audio_pipeline_->SetCdm(static_cast<BrowserCdmCast*>(media_keys));
   video_pipeline_->SetCdm(static_cast<BrowserCdmCast*>(media_keys));
 }
 
 AudioPipeline* MediaPipelineImpl::GetAudioPipeline() const {
   return audio_pipeline_.get();
 }
 
 VideoPipeline* MediaPipelineImpl::GetVideoPipeline() const {
@@ skipping 237 matching lines @@
 
 void MediaPipelineImpl::OnError(::media::PipelineStatus error) {
   DCHECK(thread_checker_.CalledOnValidThread());
   DCHECK_NE(error, ::media::PIPELINE_OK) << "PIPELINE_OK is not an error!";
   if (!client_.error_cb.is_null())
     client_.error_cb.Run(error);
 }
 
 }  // namespace media
 }  // namespace chromecast