Resume playback of EME stream only on SessionKeysChange event.

content/renderer/media/crypto/ppapi_decryptor.h

Index: content/renderer/media/crypto/ppapi_decryptor.h
diff --git a/content/renderer/media/crypto/ppapi_decryptor.h b/content/renderer/media/crypto/ppapi_decryptor.h
index 0c4c0ddc4634fc841cab8c0bfeb93dec2a1cde5f..d63dd0bca9e142ddfd8736a02282476dc1f2dc27 100644
--- a/content/renderer/media/crypto/ppapi_decryptor.h
+++ b/content/renderer/media/crypto/ppapi_decryptor.h
@@ -116,7 +116,7 @@ class PpapiDecryptor : public media::MediaKeys, public media::Decryptor {
                       uint32 system_code,
                       const std::string& error_description);
 
-  // On a successful Update() or SessionReady event, trigger playback to resume.
+  // On an OnSessionKeysChange event, trigger playback to resume.

[sandersd (OOO until July 31), 2014/11/03 21:54:16]

"On a SessionKeysChange event..."

[ddorwin, 2014/11/03 22:33:30]

Why does this function comment need to explain when it is called?

Not for this CL, but "AttemptToResumePlayback" is probably a better name. We can
tackle that with some of the related bugs.

[jrummell, 2014/11/03 23:19:29]

Removed comment and renamed function.

   void ResumePlayback();
 
   // Callback to notify that a fatal error happened in |plugin_cdm_delegate_|.