Set callback interface on WebCDMSessionImpl after creation.

content/renderer/media/webcontentdecryptionmodulesession_impl.cc

 // Copyright 2013 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 "content/renderer/media/webcontentdecryptionmodulesession_impl.h"
 
 #include "base/bind.h"
 #include "base/callback_helpers.h"
 #include "base/logging.h"
 #include "base/strings/string_util.h"
@@ skipping 28 matching lines @@
       return blink::WebContentDecryptionModuleExceptionClientError;
     case media::MediaKeys::OUTPUT_ERROR:
       return blink::WebContentDecryptionModuleExceptionOutputError;
     default:
       NOTREACHED();
       return blink::WebContentDecryptionModuleExceptionUnknownError;
   }
 }
 
 WebContentDecryptionModuleSessionImpl::WebContentDecryptionModuleSessionImpl(
-    Client* client,
     const scoped_refptr<CdmSessionAdapter>& adapter)
     : adapter_(adapter),
-      client_(client),
       is_closed_(false),
       next_available_result_index_(1),
       weak_ptr_factory_(this) {
 }
 
 WebContentDecryptionModuleSessionImpl::
     ~WebContentDecryptionModuleSessionImpl() {
   if (!web_session_id_.empty())
     adapter_->RemoveSession(web_session_id_);
 
   // Release any WebContentDecryptionModuleResult objects that are left. Their
   // index will have been passed down via a CdmPromise, but it uses a WeakPtr.
   DLOG_IF(WARNING, outstanding_results_.size() > 0)
       << "Clearing " << outstanding_results_.size() << " results";
   for (ResultMap::iterator it = outstanding_results_.begin();
        it != outstanding_results_.end();
        ++it) {
     it->second.completeWithError(
         blink::WebContentDecryptionModuleExceptionInvalidStateError,
         0,
         "Outstanding request being cancelled.");
   }
   outstanding_results_.clear();
 }
 
+void WebContentDecryptionModuleSessionImpl::setClientInterface(Client* client) {
+  client_ = client;
+}
+
 blink::WebString WebContentDecryptionModuleSessionImpl::sessionId() const {
   return blink::WebString::fromUTF8(web_session_id_);
 }
 
 void WebContentDecryptionModuleSessionImpl::initializeNewSession(
     const blink::WebString& init_data_type,
     const uint8* init_data,
     size_t init_data_length) {
   // TODO(ddorwin): Guard against this in supported types check and remove this.
   // Chromium only supports ASCII MIME types.
@@ skipping 114 matching lines @@
           result_index),
       base::Bind(&WebContentDecryptionModuleSessionImpl::SessionError,
                  weak_ptr_factory_.GetWeakPtr(),
                  result_index)));
   adapter_->ReleaseSession(web_session_id_, promise.Pass());
 }
 
 void WebContentDecryptionModuleSessionImpl::OnSessionMessage(
     const std::vector<uint8>& message,
     const GURL& destination_url) {
   client_->message(

[ddorwin, 2014/07/10 23:22:41]

This is the interesting point, especially after the refactoring. We might add:
DCHECK(client_) << "Client not set before message event";

[jrummell, 2014/07/17 00:56:54]

Done.

       message.empty() ? NULL : &message[0], message.size(), destination_url);
 }
 
 void WebContentDecryptionModuleSessionImpl::OnSessionReady() {
   client_->ready();
 }
 
 void WebContentDecryptionModuleSessionImpl::OnSessionClosed() {
   if (!is_closed_) {
     is_closed_ = true;
@@ skipping 72 matching lines @@
 
 uint32 WebContentDecryptionModuleSessionImpl::AddResult(
     blink::WebContentDecryptionModuleResult result) {
   uint32 result_index = next_available_result_index_++;
   DCHECK(result_index != kReservedIndex);
   outstanding_results_.insert(std::make_pair(result_index, result));
   return result_index;
 }
 
 }  // namespace content