Resume playback of EME stream only on SessionKeysChange event.

content/renderer/media/crypto/ppapi_decryptor.h

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CONTENT_RENDERER_MEDIA_CRYPTO_PPAPI_DECRYPTOR_H_
 #define CONTENT_RENDERER_MEDIA_CRYPTO_PPAPI_DECRYPTOR_H_
 
 #include <string>
 #include <vector>
 
@@ skipping 98 matching lines @@
                            bool has_additional_usable_key);
   void OnSessionExpirationUpdate(const std::string& web_session_id,
                                  const base::Time& new_expiry_time);
   void OnSessionReady(const std::string& web_session_id);
   void OnSessionClosed(const std::string& web_session_id);
   void OnSessionError(const std::string& web_session_id,
                       MediaKeys::Exception exception_code,
                       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_|.
   // The error is terminal and |plugin_cdm_delegate_| should not be used after
   // this call.
   void OnFatalPluginError();
 
   ContentDecryptorDelegate* CdmDelegate();
 
   // Hold a reference of the Pepper CDM wrapper to make sure the plugin lives
@@ skipping 17 matching lines @@
 
   // NOTE: Weak pointers must be invalidated before all other member variables.
   base::WeakPtrFactory<PpapiDecryptor> weak_ptr_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(PpapiDecryptor);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_RENDERER_MEDIA_CRYPTO_PPAPI_DECRYPTOR_H_