Add SetImageData api to chrome.clipboard.

chrome/test/data/extensions/api_test/clipboard/set_image_data/test.js

+// 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.
+
+// Test clipboard extension api chrome.clipboard.onClipboardDataChanged event.
+
+var testSuccessCount = 0;
+
+function testSetImageDataClipboard(imageUrl, imageType, expectSucceed) {
+  var oReq = new XMLHttpRequest();
+  oReq.open("GET", imageUrl, true);
+  oReq.responseType = "arraybuffer";
+
+  oReq.onload = function (oEvent) {
+    var arrayBuffer = oReq.response;
+    var binaryString = '';
+
+    if (arrayBuffer) {
+      chrome.clipboard.setImageData(arrayBuffer, imageType, function() {
+        if (!chrome.runtime.lastError) {

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

Can't this big chunk just be
chrome.test.assertEq(expectSucceed, !!chrome.runtime.lastError);
?

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

Done.

+          if (expectSucceed)
+            chrome.test.succeed();
+          else
+            chrome.test.fail();
+        } else {
+          if (expectSucceed)
+            chrome.test.fail();
+          else
+            chrome.test.succeed();
+        }
+        testSuccessCount++;
+        chrome.test.sendMessage('test success ' + testSuccessCount);

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

why do we need these messages?  The test system will fail if any tests fail

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

Done.

+      });
+    } else {
+      chrome.test.fail('Failed to load the png image file');
+    }
+  };
+
+  oReq.send(null);
+}
+
+function testSavePngImageToClipboard(baseUrl) {
+  testSetImageDataClipboard(baseUrl + '/penguin.png', 'png', true);

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

Rietveld won't let me comment on the pictures, so commenting here.

Are you sure that these images are not copyrighted/licenses, etc?  Are free to
use and distribute with the open source chromium license?

Is there a reason we can't just use one of the pngs we already have in chrome?

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

Good point. Changed to use the existing image files in chrome. New image files
used:
chrome/test/data/extensions/icon1.png: existing file
chrome/test/data/extensions/test.jpg: copied from
chrome/test/data/extensions/api_test/wallpaper/test.jpg
chrome/test/data/extensions/redirect_target.gif: copied from
chrome/test/data/extensions/api_test/webrequest/cors/redirect_target.gif

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

I see we're still adding the other images, though?

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

The other images from copied from existing image files from other directory so
that they are all sitting in the same directory. If you prefer to refer directly
to existing file from other directly, I can make the change.

[Devlin, 2016/12/19 23:08:22]

Latest patch set:
https://codereview.chromium.org/2379573008/diff/280001/chrome/test/data/exten...

That's a new image being added to the source tree.  We shouldn't need to do that
anymore, right?

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

I have removed the previously added image files.

+}
+
+function testSaveJpegImageToClipboard(baseUrl) {
+  testSetImageDataClipboard(baseUrl + '/panda.jpg', 'jpeg', true);
+}
+
+function testSaveBadImageData(baseUrl) {
+  testSetImageDataClipboard(baseUrl + '/star.gif', 'jpeg', false);
+}
+
+function bindTest(test, param) {
+  var result = test.bind(null, param);
+  result.generatedName = test.name;
+  return result;
+}
+
+chrome.test.getConfig(function(config) {
+  var baseUrl = 'http://localhost:' + config.testServer.port + '/extensions';
+  chrome.test.runTests([
+    bindTest(testSavePngImageToClipboard, baseUrl),
+    bindTest(testSaveJpegImageToClipboard, baseUrl),
+    bindTest(testSaveBadImageData, baseUrl)
+  ]);
+})