Update Pepper interface for EME

ppapi/api/private/ppb_content_decryptor_private.idl

 /* Copyright (c) 2012 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.
  */
 
 /**
  * This file defines the <code>PPB_ContentDecryptor_Private</code>
  * interface. Note: This is a special interface, only to be used for Content
  * Decryption Modules, not normal plugins.
  */
@@ skipping 28 matching lines @@
    *
    * @param[in] web_session_id A <code>PP_Var</code> of type
    * <code>PP_VARTYPE_STRING</code> containing the session's ID attribute.
    */
   void PromiseResolvedWithSession(
       [in] PP_Instance instance,
       [in] uint32_t promise_id,
       [in] PP_Var web_session_id);
 
   /**
+   * A promise has been resolved by the CDM.

[ddorwin, 2014/08/22 20:49:21]

These versions that have specific purposes should probably have a
differentiating part of the comment.

[jrummell, 2014/08/25 21:54:37]

Done.

+   *
+   * @param[in] promise_id Identifies the promise that the CDM resolved.
+   *
+   * @param[in] key_ids A <code>PP_Var</code> of type
+   * <code>PP_VARTYPE_ARRAY</code> containing the session's usable key IDs.

[ddorwin, 2014/08/22 20:49:21]

Do we need to define the element type?

[jrummell, 2014/08/25 21:54:37]

Not sure. I can't find any other IDL files returning PP_VARTYPE_ARRAY. Updated
comment.

+   */
+  void PromiseResolvedWithKeyIds(
+      [in] PP_Instance instance,
+      [in] uint32_t promise_id,
+      [in] PP_Var key_ids);

[ddorwin, 2014/08/22 20:49:21]

key_ids_array?

[jrummell, 2014/08/25 21:54:37]

Done.

+
+  /**
    * A promise has been rejected by the CDM due to an error.
    *
    * @param[in] promise_id Identifies the promise that the CDM rejected.
    *
    * @param[in] exception_code A <code>PP_CdmExceptionCode</code> containing
    * the exception code.
    *
    * @param[in] system_code A system error code.
    *
    * @param[in] error_description A <code>PP_Var</code> of type
@@ skipping 14 matching lines @@
    * on the <code>PPP_ContentDecryptor_Private</code> interface, the plugin
    * must send a message containing the license request.
    *
    * Note that <code>SessionMessage()</code> can be used for purposes other than
    * responses to <code>CreateSession()</code> calls. See also the text
    * in the comment for <code>SessionReady()</code>, which describes a sequence
    * of <code>UpdateSession()</code> and <code>SessionMessage()</code> calls
    * required to prepare for decryption.
    *
    * @param[in] web_session_id A <code>PP_Var</code> of type
    * <code>PP_VARTYPE_STRING</code> containing the session's ID attribute for

[ddorwin, 2014/08/22 20:49:21]

This sentence structure/description is appropriate for
PromiseResolvedWithSession, which is delivering the ID for the attribute, but
probably not these methods.

[jrummell, 2014/08/25 21:54:37]

Done.

    * which the message is intended.
    *
    * @param[in] message A <code>PP_Var</code> of type
    * <code>PP_VARTYPE_ARRAY_BUFFER</code> that contains the message.
    *
    * @param[in] destination_url A <code>PP_Var</code> of type
    * <code>PP_VARTYPE_STRING</code> containing the destination URL for the
    * message.
    */
   void SessionMessage(
       [in] PP_Instance instance,
       [in] PP_Var web_session_id,
       [in] PP_Var message,
       [in] PP_Var destination_url);
 
   /**
+   * The keys for a session have changed.
+   *
+   * @param[in] web_session_id A <code>PP_Var</code> of type
+   * <code>PP_VARTYPE_STRING</code> containing the session's ID attribute of

[ddorwin, 2014/08/22 20:49:21]

ditto here and below.

[jrummell, 2014/08/25 21:54:37]

Done.

+   * the session that has a change in keys.
+   *
+   * @param[in] has_additional_usable_key A <code>PP_Bool</code> indicating if
+   * a new usable key has been added.
+   */
+  void SessionKeysChange(
+      [in] PP_Instance instance,
+      [in] PP_Var web_session_id,
+      [in] PP_Bool has_additional_usable_key);
+
+  /**
+   * The expiration time for a session has changed.
+   *
+   * @param[in] web_session_id A <code>PP_Var</code> of type
+   * <code>PP_VARTYPE_STRING</code> containing the session's ID attribute of
+   * the session that has a new expiration time.
+   *
+   * @param[in] new_expiry_time A <code>PP_Time</code> indicating the new
+   * expiry time of the session. The value is defined as the number of seconds
+   * since the Epoch (00:00:00 UTC, January 1, 1970).
+   */
+  void SessionExpirationChange(
+      [in] PP_Instance instance,
+      [in] PP_Var web_session_id,
+      [in] PP_Time new_expiry_time);
+
+  /**
    * The session is now ready to decrypt the media stream.
    *
    * Note: The above describes the most simple case. Depending on the key
    * system, a series of <code>SessionMessage()</code> calls from the CDM will
    * be sent to the browser, and then on to the web application. The web
    * application must then provide more data to the CDM by directing the browser
    * to pass the data to the CDM via calls to <code>UpdateSession()</code> on
    * the <code>PPP_ContentDecryptor_Private</code> interface.
    * The CDM must call <code>SessionReady()</code> when the sequence is
    * completed, and, in response, the browser must notify the web application.
@@ skipping 172 matching lines @@
    *
    * @param[in] decrypted_sample_info A <code>PP_DecryptedSampleInfo</code> that
    * contains the tracking info and result code associated with the decrypted
    * samples.
    */
   void DeliverSamples(
       [in] PP_Instance instance,
       [in] PP_Resource audio_frames,
       [in] PP_DecryptedSampleInfo decrypted_sample_info);
 };