Modified the flash cipboard interface to add html clipboard support.

ppapi/cpp/private/flash_clipboard.h

 // 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.
 
 #ifndef PPAPI_CPP_PRIVATE_FLASH_CLIPBOARD_H_
 #define PPAPI_CPP_PRIVATE_FLASH_CLIPBOARD_H_
 
 #include <string>
+#include <vector>
 
 #include "ppapi/c/private/ppb_flash_clipboard.h"
 
 namespace pp {
 
 class Instance;
 
 namespace flash {
 
 class Clipboard {
  public:
   // Returns true if the required interface is available.
   static bool IsAvailable();
 
   // Returns true if the given format is available from the given clipboard.
   static bool IsFormatAvailable(Instance* instance,
                                 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, 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 (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 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
 }  // namespace pp
 
 #endif  // PPAPI_CPP_PRIVATE_FLASH_CLIPBOARD_H_