Modified the flash cipboard interface to add html clipboard support.

ppapi/cpp/private/flash_clipboard.h

Index: ppapi/cpp/private/flash_clipboard.h
diff --git a/ppapi/cpp/private/flash_clipboard.h b/ppapi/cpp/private/flash_clipboard.h
index db13ce67c8d1952e883d35330eed45576bfc9d9e..af3eef89d75afc66843f431593612aed7e229dcc 100644
--- a/ppapi/cpp/private/flash_clipboard.h
+++ b/ppapi/cpp/private/flash_clipboard.h
@@ -6,6 +6,7 @@
 #define PPAPI_CPP_PRIVATE_FLASH_CLIPBOARD_H_
 
 #include <string>
+#include <vector>
 
 #include "ppapi/c/private/ppb_flash_clipboard.h"
 
@@ -25,16 +26,18 @@ class Clipboard {
                                 PP_Flash_Clipboard_Type clipboard_type,
                                 PP_Flash_Clipboard_Format format);
 
-  // Returns true on success, in which case |text_out| will be filled with plain
-  // text read from the given clipboard.
-  static bool ReadPlainText(Instance* instance,
-                            PP_Flash_Clipboard_Type clipboard_type,
-                            std::string* text_out);
-
-  // Returns true on success (it may fail if |text| is too big).
-  static bool WritePlainText(Instance* instance,
-                             PP_Flash_Clipboard_Type clipboard_type,
-                             const std::string& text);
+  // Returns true on success, in which case |out| will be filled with
+  // data read from the given clipboard in the given format.
+  static bool ReadData(Instance* instance,
+                       PP_Flash_Clipboard_Type clipboard_type,
+                       PP_Flash_Clipboard_Format clipboard_format,
+                       PP_Var* out);

[dmichael (off chromium), 2012/02/23 18:13:25]

why not use pp::Var?

[raymes, 2012/02/24 07:28:28]

Done.

+
+  // Returns true on success in which case all of |data| will be written to
+  // the clipboard. Otherwise nothing will be written.
+  static bool WriteData(Instance* instance,
+                        PP_Flash_Clipboard_Type clipboard_type,
+                        const std::vector<PP_Flash_Clipboard_Data_Item>& data);

[dmichael (off chromium), 2012/02/23 18:13:25]

Similarly, it might be better to use pp::Var so the reference counting is harder
to screw up. That might require separating PP_Flash_Clipboard_Data_Item up in to
two parameters instead of using the struct. It's probably not worth the trouble
to try to wrap PP_Flash_Clipboard_Data_Item in a C++ class.

[raymes, 2012/02/24 07:28:28]

Thanks, I understand the ref counting better now. Based on this, I decided that
it simplified things a lot to just have 2 arrays everywhere (one for formats and
one for data) so I've updated the CL with this approach.

Done. 

 };
 
 }  // namespace flash