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 "ppapi/c/private/ppb_flash_clipboard.h"
 
 namespace pp {
 
 class Instance;
 
 namespace flash {
 
+class ClipboardData {

[viettrungluu, 2012/02/06 21:29:29]

This seems like overkill to me. I'd just use an
std::vector<PP_Flash_Clipboard_Data_Item>.

[raymes, 2012/02/07 00:02:25]

Done.

+ public:
+  static const uint32_t kMaxFormats = 3;
+
+  void SetPlainText(const std::string& text);
+  void SetHTML(const std::string& html);
+  void Clear();
+  PP_Flash_Clipboard_Data_Item const*
+      GetDataItems(uint32_t* data_item_count_out) const;
+
+ private:
+  uint32_t GetFormatIndex(PP_Flash_Clipboard_Format format);
+  void SetTextVar(const std::string& text, PP_Flash_Clipboard_Format format);
+
+  PP_Flash_Clipboard_Data_Item data_items_[kMaxFormats];
+  uint32_t data_item_count_;
+};
+
 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 (it may fail if |text| is too big).
-  static bool WritePlainText(Instance* instance,

[viettrungluu, 2012/02/06 21:29:29]

You should keep this (and also ReadPlainText()) temporarily for backwards
compatibility, and indicate that it's deprecated.

[raymes, 2012/02/07 00:02:25]

Done.

-                             PP_Flash_Clipboard_Type clipboard_type,
-                             const std::string& text);
+  // Returns true on success, in which case |html_out| will be filled with html

[viettrungluu, 2012/02/06 21:29:29]

I'd just implement a ReadData() that parallels the C interface.

[raymes, 2012/02/07 00:02:25]

Done.

+  // text read from the given clipboard.
+  static bool ReadHTML(Instance* instance,
+                       PP_Flash_Clipboard_Type clipboard_type,
+                       std::string* html_out);
+
+
+  // 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 ClipboardData& data);
+
+ private:
+  // Helper method for writing strings to the clipboard.
+  static bool ReadTextVar(Instance*,
+                          PP_Flash_Clipboard_Type clipboard_type,
+                          PP_Flash_Clipboard_Format clipboard_format,
+                          std::string* text_out);
 };
 
 }  // namespace flash
 }  // namespace pp
 
 #endif  // PPAPI_CPP_PRIVATE_FLASH_CLIPBOARD_H_