Add SetImageData api to chrome.clipboard.

chrome/browser/extensions/clipboard_extension_apitest_chromeos.cc

Index: chrome/browser/extensions/clipboard_extension_apitest_chromeos.cc
diff --git a/chrome/browser/extensions/clipboard_extension_apitest_chromeos.cc b/chrome/browser/extensions/clipboard_extension_apitest_chromeos.cc
index 37fa8118e31b2bab20f0d8ca8c09f6f08a612aaa..71bce83f5a6e4c58af1409ff7846133b62584d07 100644
--- a/chrome/browser/extensions/clipboard_extension_apitest_chromeos.cc
+++ b/chrome/browser/extensions/clipboard_extension_apitest_chromeos.cc
@@ -17,3 +17,14 @@ IN_PROC_BROWSER_TEST_F(ClipboardExtensionApiTest, ClipboardDataChanged) {
       << 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());
+}