Microphone support for Pepper Flash.

ppapi/api/trusted/ppb_audio_input_trusted_dev.idl

+/* Copyright (c) 2011 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 trusted audio input interface.
+ */
+
+label Chrome {
+  M17 = 0.1
+};
+
+/**
+ * This interface is to be used by proxy implementations.  All
+ * functions should be called from the main thread only.  The
+ * resource returned is an Audio input esource; most of the PPB_Audio

[dmichael (off chromium), 2011/11/17 04:05:21]

nit: esource -> resource

[viettrungluu, 2011/11/21 21:26:13]

Done.

+ * interface is also usable on this resource.
+ */
+[version=0.1, macro="PPB_AUDIO_INPUT_TRUSTED_DEV_INTERFACE"]

[dmichael (off chromium), 2011/11/17 04:05:21]

nit: I don't think you need this line.

+interface PPB_AudioInputTrusted_Dev {
+  /** Returns an audio input resource. */
+  PP_Resource CreateTrusted(
+      [in] PP_Instance instance);

[dmichael (off chromium), 2011/11/17 04:05:21]

Why do we need a separate 'Create' function? Can't we use the same
PPB_AudioInput_Dev.Create function, and allow passing a PPB_AudioInput_Dev
resource to the functions below?

[viettrungluu, 2011/11/21 21:26:13]

I'd have to check whether anything special is done before |Open()| is called,
but in any case this is consistent with the PPB_AudioTrusted interface (FWIW).

+
+  /**
+   * Opens a paused audio interface, used by trusted side of proxy.
+   * Returns PP_ERROR_WOULD_BLOCK on success, and invokes
+   * the |create_callback| asynchronously to complete.
+   * As this function should always be invoked from the main thread,
+   * do not use the blocking variant of PP_CompletionCallback.
+   */
+  int32_t Open(
+      [in] PP_Resource audio_input,
+      [in] PP_Resource config,
+      [in] PP_CompletionCallback create_callback);

[dmichael (off chromium), 2011/11/17 04:05:21]

nit: it seems like the params could align right after the 'Open(', with the
first param on the same line as Open. (Same for functions below)

[viettrungluu, 2011/11/21 21:26:13]

Also currently the style in all the .idl files.

+
+  /**
+   * Get the sync socket.  Use once Open has completed.
+   * Returns PP_OK on success.
+   */
+  int32_t GetSyncSocket(
+      [in] PP_Resource audio_input,
+      [out] handle_t sync_socket);
+
+  /**
+   * Get the shared memory interface.  Use once Open has completed.
+   * Returns PP_OK on success.
+   */
+  int32_t GetSharedMemory(
+      [in] PP_Resource audio_input,
+      [out] handle_t shm_handle,
+      [out] uint32_t shm_size);
+};