PpapiDecryptor: Call NewKeyCB on OnSessionReady().

content/renderer/media/crypto/ppapi_decryptor.cc

Index: content/renderer/media/crypto/ppapi_decryptor.cc
diff --git a/content/renderer/media/crypto/ppapi_decryptor.cc b/content/renderer/media/crypto/ppapi_decryptor.cc
index 350bfa199f71c0fd0230991cc94372b2fe425234..e07c913434901b946e68c9b964c8cb6865737913 100644
--- a/content/renderer/media/crypto/ppapi_decryptor.cc
+++ b/content/renderer/media/crypto/ppapi_decryptor.cc
@@ -137,12 +137,6 @@ void PpapiDecryptor::UpdateSession(uint32 session_id,
     ReportFailureToCallPlugin(session_id);
     return;
   }
-
-  if (!new_audio_key_cb_.is_null())
-    new_audio_key_cb_.Run();
-
-  if (!new_video_key_cb_.is_null())
-    new_video_key_cb_.Run();
 }
 
 void PpapiDecryptor::ReleaseSession(uint32 session_id) {
@@ -352,6 +346,13 @@ void PpapiDecryptor::OnSessionMessage(uint32 session_id,
 
 void PpapiDecryptor::OnSessionReady(uint32 session_id) {
   DCHECK(render_loop_proxy_->BelongsToCurrentThread());
+
+  if (!new_audio_key_cb_.is_null())

[ddorwin, 2014/02/19 00:04:09]

Step 3.5 of the update() algorithm is:
If the associated media element(s) are waiting for a key, queue a task to
attempt to resume playback.

Fortunately, it also says:
The user agent may choose to skip this step if it knows resuming will fail (i.e.
no usable key was added).

We should add some type of comment about what we're doing AND a TODO to rename
this callback (something to the effect of should try decrypting
again/[re]new[ed] key available) when "ready" is eliminated.

[xhwang, 2014/02/19 00:24:23]

Done.

+    new_audio_key_cb_.Run();
+
+  if (!new_video_key_cb_.is_null())
+    new_video_key_cb_.Run();
+
   session_ready_cb_.Run(session_id);
 }