Allows support for persistent session types in BrowserCdm.

chromecast/media/cdm/browser_cdm_cast.cc

Index: chromecast/media/cdm/browser_cdm_cast.cc
diff --git a/chromecast/media/cdm/browser_cdm_cast.cc b/chromecast/media/cdm/browser_cdm_cast.cc
index d00a96f25ec8630241ab6d3a29ef95131772642e..9f56a9d61576f917e3208960586aeb10bb1330a1 100644
--- a/chromecast/media/cdm/browser_cdm_cast.cc
+++ b/chromecast/media/cdm/browser_cdm_cast.cc
@@ -54,16 +54,6 @@ void BrowserCdmCast::UnregisterPlayer(int registration_id) {
   player_tracker_impl_->UnregisterPlayer(registration_id);
 }
 

[ddorwin, 2015/05/28 23:51:50]

Why are you deleting LoadSession()? This won't compile will it? (There is no
definition of the MediaKeys interface.)

[gunsch, 2015/05/28 23:59:12]

We have internal implementations that subclass BrowserCdmCast. BrowserCdmCast
also contains its own pure virtual methods.

[ddorwin, 2015/05/29 02:13:59]

Oh, I thought it was possible to build Cast, albeit stubbed, using only Chromium
code.

[gunsch, 2015/05/29 16:40:10]

It is. We don't instantiate any BrowserCdmCast instances in our public code
today.

If you want to test EME today on Chromecast, you have to either:
a) Use a Cast Android build (since the DRM plugins are already in the Android
platform)
b) Do a full Chromecast build with internal code and test on Chromecast hardware

Ideally we'd have a BrowserCdm-based ClearKey support to test Chromecast's EME
flow in the public code. crbug.com/441957 is tracking but I don't think it's
actively being worked on.

-void BrowserCdmCast::LoadSession(
-    ::media::MediaKeys::SessionType session_type,
-    const std::string& session_id,
-    scoped_ptr<::media::NewSessionCdmPromise> promise) {
-  NOTREACHED() << "LoadSession not supported";
-  legacy_session_error_cb_.Run(
-      session_id, ::media::MediaKeys::Exception::NOT_SUPPORTED_ERROR, 0,
-      std::string());
-}
-
 ::media::CdmContext* BrowserCdmCast::GetCdmContext() {
   NOTREACHED();
   return nullptr;