Add SetImageData api to chrome.clipboard.

chrome/browser/extensions/api/chrome_extensions_api_client.cc

 // Copyright 2014 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 "chrome/browser/extensions/api/chrome_extensions_api_client.h"
 
 #include "base/bind.h"
 #include "base/files/file_path.h"
 #include "base/memory/ptr_util.h"
 #include "build/build_config.h"
@@ skipping 18 matching lines @@
 #include "components/pdf/browser/pdf_web_contents_helper.h"
 #include "content/public/browser/browser_context.h"
 #include "content/public/browser/browser_thread.h"
 #include "extensions/browser/api/virtual_keyboard_private/virtual_keyboard_delegate.h"
 #include "extensions/browser/guest_view/web_view/web_view_guest.h"
 #include "extensions/browser/guest_view/web_view/web_view_permission_helper.h"
 #include "printing/features/features.h"
 
 #if defined(OS_CHROMEOS)
 #include "chrome/browser/extensions/api/virtual_keyboard_private/chrome_virtual_keyboard_delegate.h"
+#include "chrome/browser/extensions/clipboard_extension_helper_chromeos.h"
 #endif
 
 #if BUILDFLAG(ENABLE_PRINTING)
 #if BUILDFLAG(ENABLE_PRINT_PREVIEW)
 #include "chrome/browser/printing/print_preview_message_handler.h"
 #include "chrome/browser/printing/print_view_manager.h"
 #else
 #include "chrome/browser/printing/print_view_manager_basic.h"
 #endif  // BUILDFLAG(ENABLE_PRINT_PREVIEW)
 #endif  // BUILDFLAG(ENABLE_PRINTING)
 
 namespace extensions {
 
-ChromeExtensionsAPIClient::ChromeExtensionsAPIClient() {}
+ChromeExtensionsAPIClient::ChromeExtensionsAPIClient() {

[Devlin, 2016/12/21 23:58:38]

Why this change?

[jennyz, 2016/12/22 22:45:21]

I thought you suggested me to move it to ChromeExtensionsAPIClient, see your
previous comment:

"If you're convinced we can't implement this in //extensions, this should go on
the ChromeExtensionsAPIClient."

https://codereview.chromium.org/2379573008/diff/220001/chrome/browser/extensi...

Maybe I misunderstood?

[Devlin, 2016/12/27 17:08:00]

Yes.  But why do we need to put a newline here?  It seems against clang format,
is inconsistent with line 56 below, and isn't relevant to this change.

[jennyz, 2016/12/29 05:16:03]

Done.

+}
 
 ChromeExtensionsAPIClient::~ChromeExtensionsAPIClient() {}
 
 void ChromeExtensionsAPIClient::AddAdditionalValueStoreCaches(
     content::BrowserContext* context,
     const scoped_refptr<ValueStoreFactory>& factory,
     const scoped_refptr<base::ObserverListThreadSafe<SettingsObserver>>&
         observers,
     std::map<settings_namespace::Namespace, ValueStoreCache*>* caches) {
   // Add support for chrome.storage.sync.
@@ skipping 96 matching lines @@
     const {
   return new ChromeManagementAPIDelegate;
 }
 
 MetricsPrivateDelegate* ChromeExtensionsAPIClient::GetMetricsPrivateDelegate() {
   if (!metrics_private_delegate_)
     metrics_private_delegate_.reset(new ChromeMetricsPrivateDelegate());
   return metrics_private_delegate_.get();
 }
 
+#if defined(OS_CHROMEOS)
+void ChromeExtensionsAPIClient::SaveImageDataToClipboard(
+    const std::vector<char>& image_data,
+    api::clipboard::ImageType type,
+    const base::Closure& success_callback,
+    const base::Callback<void(const std::string&)>& error_callback) {
+  if (!clipboard_extension_helper_)
+    clipboard_extension_helper_.reset(new ClipboardExtensionHelper());

[Devlin, 2016/12/21 23:58:38]

prefer MakeUnique

[jennyz, 2016/12/22 22:45:21]

Done.

+  clipboard_extension_helper_->DecodeAndSaveImageData(
+      image_data, type, success_callback, error_callback);
+}
+#endif
+
 }  // namespace extensions