Make BrowserCdmCast::Initialize() a virtual function so that a subclass can override the initializa…

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 07c3c3e5500e9d5e94d08ffc2ced5aa611543336..0eeeab01470db21142de32cc4828dee6361bd54b 100644
--- a/chromecast/media/cdm/browser_cdm_cast.cc
+++ b/chromecast/media/cdm/browser_cdm_cast.cc
@@ -39,6 +39,8 @@ void BrowserCdmCast::Initialize(
   legacy_session_error_cb_ = legacy_session_error_cb;
   session_keys_change_cb_ = session_keys_change_cb;
   session_expiration_update_cb_ = session_expiration_update_cb;
+
+  InitializeInternal();
 }
 
 int BrowserCdmCast::RegisterPlayer(const base::Closure& new_key_cb,
@@ -198,5 +200,10 @@ void BrowserCdmCastUi::RemoveSession(
   return nullptr;
 }
 
+// A default empty implementation for subclasses that don't need to any

[gunsch-google, 2015/05/15 17:48:26]

missing a word: "don't need to any"

[lcwu1, 2015/05/15 18:01:20]

Done.

+// key system specific initialization.

[gunsch-google, 2015/05/15 17:48:27]

style nit: method comments go with declaration, not definition

[lcwu1, 2015/05/15 18:01:20]

There is already a function comment in the declaration. This is a comment
specifically for the implementation.

+void BrowserCdmCast::InitializeInternal() {
+}
+
 }  // namespace media
 }  // namespace chromecast