media: Add static MojoCdmService::GetCdm().

media/mojo/services/mojo_cdm_service.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 "media/mojo/services/mojo_cdm_service.h"
 
+#include <map>
+
 #include "base/bind.h"
+#include "base/lazy_instance.h"
 #include "media/base/cdm_config.h"
 #include "media/base/cdm_context.h"
 #include "media/base/cdm_factory.h"
 #include "media/base/cdm_key_information.h"
 #include "media/base/key_systems.h"
 #include "media/mojo/services/media_type_converters.h"
 #include "media/mojo/services/mojo_cdm_service_context.h"
 #include "mojo/common/common_type_converters.h"
 #include "mojo/common/url_type_converters.h"
 #include "url/gurl.h"
 
 namespace media {
 
+namespace {
+
+// A static map between the CDM ID and the CDM. Can be accessed on different
+// threads (protected by the |g_cdm_map_lock_|).
+using CdmMap = std::map<int, scoped_refptr<MediaKeys>>;
+
+base::LazyInstance<CdmMap>::Leaky g_cdm_map = LAZY_INSTANCE_INITIALIZER;

[DaleCurtis, 2015/11/17 23:57:05]

Instead of two lazy instances you could put this in a class that returns the id
based on an internal lock?

[xhwang, 2015/11/18 00:52:53]

It does make the main impl (MojoCdmService) cleaner. Done!

+
+base::LazyInstance<base::Lock>::Leaky g_cdm_map_lock =
+    LAZY_INSTANCE_INITIALIZER;
+}
+
 using SimpleMojoCdmPromise = MojoCdmPromise<>;
 using CdmIdMojoCdmPromise = MojoCdmPromise<int>;
 using NewSessionMojoCdmPromise = MojoCdmPromise<std::string>;
 
 int MojoCdmService::next_cdm_id_ = CdmContext::kInvalidCdmId + 1;
 
+// static
+scoped_refptr<MediaKeys> MojoCdmService::GetCdm(int cdm_id) {
+  DVLOG(1) << __FUNCTION__ << ": " << cdm_id;
+  base::AutoLock lock(g_cdm_map_lock.Get());
+  return g_cdm_map.Get().count(cdm_id) ? g_cdm_map.Get()[cdm_id] : nullptr;
+}
+
 MojoCdmService::MojoCdmService(
     base::WeakPtr<MojoCdmServiceContext> context,
     mojo::ServiceProvider* service_provider,
     CdmFactory* cdm_factory,
     mojo::InterfaceRequest<interfaces::ContentDecryptionModule> request)
     : binding_(this, request.Pass()),
       context_(context),
       service_provider_(service_provider),
       cdm_factory_(cdm_factory),
       cdm_id_(CdmContext::kInvalidCdmId),
       weak_factory_(this) {
   DCHECK(context_);
   DCHECK(cdm_factory_);
 }
 
 MojoCdmService::~MojoCdmService() {
+  {
+    base::AutoLock lock(g_cdm_map_lock.Get());
+    DCHECK(g_cdm_map.Get().count(cdm_id_));
+    g_cdm_map.Get().erase(cdm_id_);
+  }
+
   if (cdm_id_ != CdmContext::kInvalidCdmId && context_)
     context_->UnregisterCdm(cdm_id_);
 }
 
 void MojoCdmService::SetClient(
     interfaces::ContentDecryptionModuleClientPtr client) {
   client_ = client.Pass();
 }
 
 void MojoCdmService::Initialize(
@@ skipping 92 matching lines @@
   // TODO(xhwang): This should not happen when KeySystemInfo is properly
   // populated. See http://crbug.com/469366
   if (!cdm || !context_) {
     promise->reject(MediaKeys::NOT_SUPPORTED_ERROR, 0, error_message);
     return;
   }
 
   cdm_ = cdm;
   cdm_id_ = next_cdm_id_++;
   context_->RegisterCdm(cdm_id_, this);
+
+  {
+    base::AutoLock lock(g_cdm_map_lock.Get());
+    DCHECK(!g_cdm_map.Get().count(cdm_id_));
+    g_cdm_map.Get()[cdm_id_] = cdm;
+  }
+
+  DVLOG(1) << __FUNCTION__ << ": CDM successfully created with ID " << cdm_id_;
   promise->resolve(cdm_id_);
 }
 
 void MojoCdmService::OnSessionMessage(const std::string& session_id,
                                       MediaKeys::MessageType message_type,
                                       const std::vector<uint8_t>& message,
                                       const GURL& legacy_destination_url) {
   DVLOG(2) << __FUNCTION__;
   client_->OnSessionMessage(
       session_id, static_cast<interfaces::CdmMessageType>(message_type),
@@ skipping 29 matching lines @@
                                           MediaKeys::Exception exception,
                                           uint32_t system_code,
                                           const std::string& error_message) {
   DVLOG(2) << __FUNCTION__;
   client_->OnLegacySessionError(
       session_id, static_cast<interfaces::CdmException>(exception), system_code,
       error_message);
 }
 
 }  // namespace media