Modified the flash cipboard interface to add html clipboard support.

ppapi/api/private/ppb_flash_clipboard.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 private <code>PPB_Flash_Clipboard</code> API used by
  * Pepper Flash for reading and writing to the clipboard.
  */
 
@@ skipping 47 matching lines @@
    * Returns true if the given format is available from the given clipboard.
    */
   PP_Bool IsFormatAvailable(
       [in] PP_Instance instance_id,
       [in] PP_Flash_Clipboard_Type clipboard_type,
       [in] PP_Flash_Clipboard_Format format);
 
   /**
    * Reads plain text data from the clipboard.
    */
   PP_Var ReadPlainText(

[viettrungluu, 2012/01/25 00:01:55]

Maybe we should have:

PP_Var ReadData([in] PP_Instance instance_id,
                [in] PP_Flash_Clipboard_Type clipboard_type,
                [in] PP_Flash_Clipboard_Format format);

and

int32_t AddData([in] PP_Instance instance_id,
                [in] PP_Flash_Clipboard_Type clipboard_type,
                [in] PP_Var data);

(see below for "Add" versus "Write").

[raymes1, 2012/01/25 01:33:12]

Sounds good, I was debating whether or not to do this.

       [in] PP_Instance instance_id,
       [in] PP_Flash_Clipboard_Type clipboard_type);
 
   /**
-   * Writes plain text data to the clipboard. If <code>text</code> is too large,
-   * it will return <code>PP_ERROR_NOSPACE</code> and not write to the
-   * clipboard.
+   * Writes plain text data to the clipboard buffer. If <code>text</code> is too
+   * large, it will return <code>PP_ERROR_NOSPACE</code> and not write to the
+   * clipboard buffer.
    */
   int32_t WritePlainText(

[viettrungluu, 2012/01/25 00:01:55]

Since "write" no longer parallels "read" directly (since the former now requires
"commit"), I think we should call the former "add" or maybe "set".

[raymes1, 2012/01/25 01:33:12]

Done.

       [in] PP_Instance instance_id,
       [in] PP_Flash_Clipboard_Type clipboard_type,
       [in] PP_Var text);
+  
+  /**
+   * Reads HTML data from the clipboard.
+   */
+  PP_Var ReadHTML(
+      [in] PP_Instance instance_id,
+      [in] PP_Flash_Clipboard_Type clipboard_type);
+
+  /**
+   * Writes HTML data to the clipboard buffer. If <code>html</code> is too 
+   * large, it will return <code>PP_ERROR_NOSPACE</code> and not write to the
+   * clipboard buffer.
+   */
+  int32_t WriteHTML(
+      [in] PP_Instance instance_id,
+      [in] PP_Flash_Clipboard_Type clipboard_type,
+      [in] PP_Var html);
+  
+  /**
+   * Clear the contents of the clipboard buffer.
+   */
+  void Clear(

[viettrungluu, 2012/01/25 00:01:55]

Is this instantaneous? Or does it also require commit?

[raymes1, 2012/01/25 01:33:12]

This requires a commit. I updated the docs to make that clear.

+      [in] PP_Instance instance_id,
+      [in] PP_Flash_Clipboard_Type clipboard_type); 
+  
+  /**
+    * Commit the contents of the clipboard buffer to the clipboard. Any existing
+    * contents on the clipboard are cleared before the contents of the clipboard
+    * buffer are written.
+    */
+   void Commit(
+       [in] PP_Instance instance_id,
+       [in] PP_Flash_Clipboard_Type clipboard_type);
+ 
 };