[Chromecast] Allow CMA backend to notify of key expiry

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 137 matching lines @@
 }
 
 void MediaPipelineImpl::OnDecoderError(MediaPipelineBackend::Decoder* decoder) {
   if (decoder == audio_decoder_) {
     audio_pipeline_->OnError();
   } else if (decoder == video_decoder_) {
     video_pipeline_->OnError();
   }
 }
 
+void MediaPipelineImpl::OnKeyStatusChanged(const std::string& key_id,
+                                           CastKeyStatus key_status,
+                                           uint32_t system_code) {
+  CMALOG(kLogControl) << __FUNCTION__;
+  DCHECK(thread_checker_.CalledOnValidThread());
+  if (cdm_)

[yucliu1, 2015/11/18 01:06:36]

Is it possible for !cdm_ happen? If this did happen, I think we need to fire it
later when cdm_ is ready. Otherwise, add a DCHECK here?

[halliwell, 2015/11/18 02:32:18]

Agreed, I don't think it's possible.  A key can only change status if we are
using a CDM in the first place.  Changed to a DCHECK.

+    cdm_->SetKeyStatus(key_id, key_status, system_code);
+}
+
 void MediaPipelineImpl::SetCdm(BrowserCdmCast* cdm) {
   CMALOG(kLogControl) << __FUNCTION__;
   DCHECK(thread_checker_.CalledOnValidThread());
   cdm_ = cdm;
   if (audio_pipeline_)
     audio_pipeline_->SetCdm(cdm);
   if (video_pipeline_)
     video_pipeline_->SetCdm(cdm);
 }
 
@@ skipping 281 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