Add GetSequenceNumber function to PPB_Flash_Clipboard

ppapi/cpp/private/flash_clipboard.cc

 // 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.
 
 #include "ppapi/cpp/private/flash_clipboard.h"
 
 #include <vector>
 
 #include "ppapi/c/pp_bool.h"
 #include "ppapi/c/pp_errors.h"
 #include "ppapi/cpp/instance_handle.h"
 #include "ppapi/cpp/module_impl.h"
 #include "ppapi/cpp/var.h"
 
 namespace pp {
 
 namespace {
 
 template <> const char* interface_name<PPB_Flash_Clipboard_4_0>() {
   return PPB_FLASH_CLIPBOARD_INTERFACE_4_0;
 }
 
 template <> const char* interface_name<PPB_Flash_Clipboard_5_0>() {
   return PPB_FLASH_CLIPBOARD_INTERFACE_5_0;
 }
 
+template <> const char* interface_name<PPB_Flash_Clipboard_5_1>() {
+  return PPB_FLASH_CLIPBOARD_INTERFACE_5_1;
+}
+
 }  // namespace
 
 namespace flash {
 
 // static
 bool Clipboard::IsAvailable() {
-  return has_interface<PPB_Flash_Clipboard_5_0>() ||
+  return has_interface<PPB_Flash_Clipboard_5_1>() ||
+      has_interface<PPB_Flash_Clipboard_5_0>() ||
       has_interface<PPB_Flash_Clipboard_4_0>() ;
 }
 
 // static
 uint32_t Clipboard::RegisterCustomFormat(const InstanceHandle& instance,
                                          const std::string& format_name) {
   uint32_t rv = PP_FLASH_CLIPBOARD_FORMAT_INVALID;
   if (has_interface<PPB_Flash_Clipboard_5_0>()) {
     rv = get_interface<PPB_Flash_Clipboard_5_0>()->RegisterCustomFormat(
         instance.pp_instance(), format_name.c_str());
   }
   return rv;
 }
 
 // static
 bool Clipboard::IsFormatAvailable(const InstanceHandle& instance,
                                   PP_Flash_Clipboard_Type clipboard_type,
                                   uint32_t format) {
   bool rv = false;
   if (has_interface<PPB_Flash_Clipboard_5_0>()) {

[teravest, 2014/01/21 14:31:55]

For all the existing methods, I think you need to add a
has_interface/get_interface sections for PPB_Flash_Clipboard_5_1 as well.

The argument I saw for this in the past was that adding a stanza for new
versions would make it more possible for us to remove old versions in the
future.

[raymes, 2014/01/22 23:23:18]

Done.

     rv = PP_ToBool(get_interface<PPB_Flash_Clipboard_5_0>()->IsFormatAvailable(
         instance.pp_instance(), clipboard_type, format));
   } else if (has_interface<PPB_Flash_Clipboard_4_0>()) {
     rv = PP_ToBool(get_interface<PPB_Flash_Clipboard_4_0>()->IsFormatAvailable(
         instance.pp_instance(), clipboard_type,
         static_cast<PP_Flash_Clipboard_Format>(format)));
   }
   return rv;
 }
 
@@ skipping 77 matching lines @@
         instance.pp_instance(),
         clipboard_type,
         data_items.size(),
         formats_ptr,
         data_items_ptr) == PP_OK);
   }
 
   return rv;
 }
 
+// static
+bool Clipboard::GetSequenceNumber(const InstanceHandle& instance,
+                                  PP_Flash_Clipboard_Type clipboard_type,
+                                  uint64_t* sequence_number) {
+  if (has_interface<PPB_Flash_Clipboard_5_1>()) {
+    return PP_ToBool(
+        get_interface<PPB_Flash_Clipboard_5_1>()->GetSequenceNumber(
+            instance.pp_instance(), clipboard_type, sequence_number));
+  }
+  return false;
+}
+
 }  // namespace flash
 }  // namespace pp