Keep reference to CDM to avoid it's premature destruction

media/blink/cdm_session_adapter.h

Index: media/blink/cdm_session_adapter.h
diff --git a/media/blink/cdm_session_adapter.h b/media/blink/cdm_session_adapter.h
index a17a6426b9078a259c468c469518abe0038dd9b9..a3089662a671578837ab37eb7f2b2838f0ad2378 100644
--- a/media/blink/cdm_session_adapter.h
+++ b/media/blink/cdm_session_adapter.h
@@ -90,10 +90,8 @@ class CdmSessionAdapter : public base::RefCounted<CdmSessionAdapter> {
   void RemoveSession(const std::string& session_id,
                      std::unique_ptr<SimpleCdmPromise> promise);
 
-  // Returns the CdmContext associated with |media_keys_|.
-  // TODO(jrummell): Figure out lifetimes, as WMPI may still use the decryptor
-  // after WebContentDecryptionModule is freed. http://crbug.com/330324
-  CdmContext* GetCdmContext();
+  // Returns a reference to the CDM.
+  scoped_refptr<MediaKeys> GetCdmReference();

[xhwang, 2016/10/07 20:44:31]

CDM (MediaKeys) is already ref-counted so it seems "Reference" is not necessary.
In other words, the caller won't do anything different whether this is called
GetCdmReference() or GetCdm().

[jrummell, 2016/10/07 23:46:45]

Done.

 
   // Returns the key system name.
   const std::string& GetKeySystem() const;