Add SetImageData api to chrome.clipboard.

chrome/browser/extensions/clipboard_extension_apitest_chromeos.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 "chrome/browser/extensions/extension_apitest.h"
 #include "extensions/test/extension_test_message_listener.h"
 
 namespace {
 
 class ClipboardExtensionApiTest : public ExtensionApiTest {};
 
 }  // namespace
 
 IN_PROC_BROWSER_TEST_F(ClipboardExtensionApiTest, ClipboardDataChanged) {
   ExtensionTestMessageListener result_listener("success 2", false);
   ASSERT_TRUE(RunPlatformAppTest("clipboard/clipboard_data_changed"))
       << message_;
   ASSERT_TRUE(result_listener.WaitUntilSatisfied());
 }
+
+IN_PROC_BROWSER_TEST_F(ClipboardExtensionApiTest, SetImageData) {

[Devlin, 2016/12/09 15:23:43]

What's the reason this can't be a unittest?

[jennyz, 2016/12/14 01:15:35]

I made ClipboardExtensionApiTest into a interactive_ui_tests because the first
clipboard API test for ClipboardDataChanged needs interactive UI action. The
SetImageData test does not need interactive ui action, but I put it here for
convenience reason. If you are really strong about it should not be here, I can
change.

[Devlin, 2016/12/16 02:26:15]

If it can, let's make it a unit test.  Unit tests are faster and more stable,
and should always be preferred, all other things equal.

[jennyz, 2016/12/19 07:01:36]

I am not familiar with writing unit test for extension api. I will do some
research and make a change later. You may review the rest of the changes first.

[jennyz, 2016/12/20 22:16:54]

I took a look at converting it to a unit test, it looks like the ImageDecoder I
used needs to run in a utility process, which is not available in unit test
infrastructure. So it is not straightforward to make it a unit test. Can we
leave this as a browser test?

+  ASSERT_TRUE(StartEmbeddedTestServer());
+  ExtensionTestMessageListener test_listener("test success 3", false);
+  ExtensionTestMessageListener clipboard_change_listener(
+      "clipboard data changed 2", false);
+  ASSERT_TRUE(RunPlatformAppTest("clipboard/set_image_data"))
+      << message_;
+  ASSERT_TRUE(test_listener.WaitUntilSatisfied());
+  ASSERT_TRUE(clipboard_change_listener.WaitUntilSatisfied());
+}