Index: media/remoting/remoting_cdm_context.h |
diff --git a/media/remoting/remoting_cdm_context.h b/media/remoting/remoting_cdm_context.h |
index f7c3284d1baccaac47d7c380915b41db34e3f4bf..306dd49650bdd1f6552f0bb2a6d76f6091deb904 100644 |
--- a/media/remoting/remoting_cdm_context.h |
+++ b/media/remoting/remoting_cdm_context.h |
@@ -8,11 +8,17 @@ |
#include "media/base/cdm_context.h" |
namespace media { |
+namespace remoting { |
class RemotingCdm; |
-class RemotingSourceImpl; |
+class SharedSession; |
// TODO(xjz): Merge this with erickung's implementation. |
+// TODO(miu): This class should just be merged into RemotingCdm and implement |
+// both the CDM and CdmContext interfaces. Also, replace the GetSharedSession() |
+// accessor and move it to a new SharedSession::FromCdmContext() function. Then, |
+// neither the controller nor renderer can gain direct access to the CDM impl. |
+// See discussion in https://codereview.chromium.org/2643253003 for more info. |
class RemotingCdmContext : public CdmContext { |
public: |
explicit RemotingCdmContext(RemotingCdm* remoting_cdm); |
@@ -22,7 +28,7 @@ class RemotingCdmContext : public CdmContext { |
// pointer to it. Otherwise, return nullptr. |
static RemotingCdmContext* From(CdmContext* cdm_context); |
- RemotingSourceImpl* GetRemotingSource(); |
+ SharedSession* GetSharedSession() const; |
// CdmContext implementations. |
Decryptor* GetDecryptor() override; |
@@ -35,6 +41,7 @@ class RemotingCdmContext : public CdmContext { |
DISALLOW_COPY_AND_ASSIGN(RemotingCdmContext); |
}; |
+} // namespace remoting |
} // namespace media |
#endif // MEDIA_REMOTING_REMOTING_CDM_CONTEXT_H_ |