media: Add MojoCdmServiceContext.

media/mojo/services/mojo_cdm_service_context.cc

+// Copyright 2014 The Chromium Authors. All rights reserved.

[ddorwin, 2015/06/08 18:33:10]

nit: year is inconsistent with .h. Is that correct?

[xhwang, 2015/06/08 22:12:33]

Done.

+// 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_context.h"
+
+#include "base/bind.h"
+#include "base/logging.h"
+
+namespace media {
+
+MojoCdmServiceContext::MojoCdmServiceContext() {
+}
+
+MojoCdmServiceContext::~MojoCdmServiceContext() {
+}
+
+MojoCdmService* MojoCdmServiceContext::Create(
+    const mojo::String& key_system,
+    const mojo::String& security_origin,
+    int32_t cdm_id,
+    mojo::InterfaceRequest<mojo::ContentDecryptionModule> request) {
+  DVLOG(1) << __FUNCTION__ << ": " << key_system;
+
+  // TODO(xhwang): pass |security_origin| down because CdmFactory needs it.
+  MojoCdmService* cdm_service =
+      MojoCdmService::Create(this, key_system, request.Pass());

[ddorwin, 2015/06/08 18:33:10]

Should this function be returning a scoped_ptr?

[xhwang, 2015/06/08 22:12:33]

Done.

+  if (cdm_service)
+    services_.add(cdm_id, make_scoped_ptr(cdm_service));
+  return cdm_service;
+}
+
+// TODO(xhwang): The returned CdmContext can be destroyed at any time if the

[ddorwin, 2015/06/08 18:33:10]

This seems worse than what is implied by the comments in the interface defn. You
might want to add this type of warning there.

[xhwang, 2015/06/08 22:12:33]

Done.

+// pipe is disconnected. When implementing SetCdm(), make sure we never
+// dereference garbage. For example, use media::PlayerTracker.
+CdmContext* MojoCdmServiceContext::GetCdmContext(int32_t cdm_id) {
+  MojoCdmService* service = services_.get(cdm_id);
+  if (!service) {
+    LOG(ERROR) << "CDM context not found: " << cdm_id;
+    return nullptr;
+  }
+
+  return service->GetCdmContext();
+}
+
+void MojoCdmServiceContext::ServiceHadConnectionError(MojoCdmService* service) {
+  for (auto it = services_.begin(); it != services_.end(); ++it) {
+    if (it->second == service) {
+      services_.erase(it);  // This destroys the |service|.
+      return;
+    }
+  }
+
+  NOTREACHED() << "Service " << service << " not found.";
+}
+
+}  // namespace media