Add SetImageData api to chrome.clipboard.

extensions/browser/api/clipboard/clipboard_api.cc

Index: extensions/browser/api/clipboard/clipboard_api.cc
diff --git a/extensions/browser/api/clipboard/clipboard_api.cc b/extensions/browser/api/clipboard/clipboard_api.cc
index 9b996e460181bf8a7ea8080ed474c0dab65c36b5..db063d6cb76ab1927e82a9b4d4c6772f74a9a08a 100644
--- a/extensions/browser/api/clipboard/clipboard_api.cc
+++ b/extensions/browser/api/clipboard/clipboard_api.cc
@@ -10,6 +10,7 @@
 #include "base/memory/ptr_util.h"
 #include "base/values.h"
 #include "extensions/browser/event_router.h"
+#include "extensions/browser/extensions_browser_client.h"
 #include "extensions/common/api/clipboard.h"
 #include "ui/base/clipboard/clipboard_monitor.h"
 
@@ -47,4 +48,31 @@ void ClipboardAPI::OnClipboardDataChanged() {
   }
 }
 
+ClipboardSetImageDataFunction::~ClipboardSetImageDataFunction() {}
+
+ExtensionFunction::ResponseAction ClipboardSetImageDataFunction::Run() {
+  std::unique_ptr<clipboard::SetImageData::Params> params(
+      clipboard::SetImageData::Params::Create(*args_));
+  EXTENSION_FUNCTION_VALIDATE(params.get());
+  SaveImageData(params->image_data, params->type);
+  return RespondLater();
+}
+
+void ClipboardSetImageDataFunction::SaveImageData(
+    const std::vector<char>& image_data,
+    const std::string& type) {
+  ExtensionsBrowserClient::Get()->SaveImageDataToClipboard(

[dcheng, 2016/10/04 05:13:27]

I might be missing something, but can the ExtensionFunction just use the helper
directly instead of going through ContentBrowserClient? From a layering
perspective, it would be nice not to have to expose extension-specific
functionality there.

[jennyz, 2016/10/04 23:53:54]

The image decoding code we used is under chrome/browser, which
extensions/browser/api code can not depends on, so we have to access it via
ExtensionsBrowserClient interface.

+      image_data, type,
+      base::Bind(&ClipboardSetImageDataFunction::OnSaveImageDataSuccess, this),
+      base::Bind(&ClipboardSetImageDataFunction::OnSaveImageDataError, this));
+}
+
+void ClipboardSetImageDataFunction::OnSaveImageDataSuccess() {
+  Respond(OneArgument(base::MakeUnique<base::FundamentalValue>(true)));
+}
+
+void ClipboardSetImageDataFunction::OnSaveImageDataError() {
+  Respond(OneArgument(base::MakeUnique<base::FundamentalValue>(false)));
+}
+
 }  // namespace extensions