Modify SetImageData to add additional items along with image data to save on clipboard.

extensions/browser/api/clipboard/clipboard_api.cc

 // Copyright (c) 2016 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 "extensions/browser/api/clipboard/clipboard_api.h"
 
 #include <utility>
 
 #include "base/lazy_instance.h"
 #include "base/memory/ptr_util.h"
+#include "base/strings/string_util.h"
 #include "base/values.h"
 #include "extensions/browser/api/extensions_api_client.h"
 #include "extensions/browser/event_router.h"
 #include "ui/base/clipboard/clipboard_monitor.h"
 
 namespace extensions {
 
 namespace clipboard = api::clipboard;
 
 static base::LazyInstance<
@@ skipping 26 matching lines @@
     router->BroadcastEvent(std::move(event));
   }
 }
 
 ClipboardSetImageDataFunction::~ClipboardSetImageDataFunction() {}
 
 ExtensionFunction::ResponseAction ClipboardSetImageDataFunction::Run() {
   std::unique_ptr<clipboard::SetImageData::Params> params(
       clipboard::SetImageData::Params::Create(*args_));
   EXTENSION_FUNCTION_VALIDATE(params);
+
+  // Fill in the omitted additiona data items with empty data.

[Devlin, 2017/05/17 16:08:44]

typo: additional

[jennyz, 2017/05/18 23:33:49]

Done.

+  if (!params->additional_items.get())

[Devlin, 2017/05/17 16:08:44]

no need for .get()

[jennyz, 2017/05/18 23:33:49]

Done.

+    params->additional_items.reset(new AdditionalDataItemList());
+
+  if (!IsAdditionalItemsParamValid(*(params->additional_items))) {
+    return RespondNow(Error("Unsupported additional_items parameter data."));

[Devlin, 2017/05/17 16:08:44]

additionalItems

[jennyz, 2017/05/18 23:33:49]

Done.

+  }
+
   ExtensionsAPIClient::Get()->SaveImageDataToClipboard(
-      params->image_data, params->type,
+      params->image_data, params->type, std::move(params->additional_items),
       base::Bind(&ClipboardSetImageDataFunction::OnSaveImageDataSuccess, this),
       base::Bind(&ClipboardSetImageDataFunction::OnSaveImageDataError, this));
   return RespondLater();
 }
 
 void ClipboardSetImageDataFunction::OnSaveImageDataSuccess() {
   Respond(NoArguments());
 }
 
 void ClipboardSetImageDataFunction::OnSaveImageDataError(
     const std::string& error) {
   Respond(Error(error));
 }
 
+bool ClipboardSetImageDataFunction::IsAdditionalItemsParamValid(
+    const AdditionalDataItemList& items) {
+  // Limit the maximum text/html data length to 2MB.
+  const size_t max_item_data_bytes = 2097152;
+
+  bool has_text_plain = false;
+  bool has_text_html = false;
+  for (const clipboard::AdditionalDataItem& item : items) {
+    if (item.type == clipboard::DATA_ITEM_TYPE_TEXT_PLAIN) {

[Devlin, 2017/05/17 16:08:44]

use a switch statement

[jennyz, 2017/05/18 23:33:49]

Done.

+      if (has_text_plain)
+        return false;
+      has_text_plain = true;
+    }
+    if (item.type == clipboard::DATA_ITEM_TYPE_TEXT_HTML) {
+      if (has_text_html)
+        return false;
+      has_text_html = true;
+    }
+    // Check maximum length of the string data.
+    if (item.data.length() > max_item_data_bytes)
+      return false;
+  }
+  return true;
+}
+
 }  // namespace extensions