Modify SetImageData to add additional items along with image data to save on clipboard.

extensions/common/api/clipboard.idl

 // Copyright 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.
 
 // The <code>chrome.clipboard</code> API is provided to allow users to
 // access data of the clipboard. This is a temporary solution for
 // chromeos platform apps until open-web alternative is available. It will be
 // deprecated once open-web solution is available, which could be in 2017 Q2.

[Devlin, 2017/04/26 18:20:15]

Any update on this?

[jennyz, 2017/05/16 18:22:03]

I will ask dcheng@ for update, he is OOO until May 2nd.

[Devlin, 2017/05/17 16:08:43]

Past May 2 :)

[dcheng, 2017/05/19 13:05:09]

garykac@ is working on landing the IDL changes for it now and it's in the
implement phase. It will still have to go through the shipping intent. garykac@
will have the full details =)

[jennyz, 2017/05/19 22:53:12]

I will sync with garykac@, thanks!

[jennyz, 2017/05/22 23:59:09]

Done.

[jennyz, 2017/05/22 23:59:09]

Done.

 [platforms=("chromeos"),
  implemented_in="extensions/browser/api/clipboard/clipboard_api.h"]
 namespace clipboard {
   // Supported image types.
   enum ImageType {png, jpeg};
+
+  // Additional data item to be added along with the |image_data| to describe
+  // the |image_data|.
+  dictionary AdditionalDataItem {
+    // MIME type of the additional data item, supported types are:
+    // "text/plain" and "text/html".

[Devlin, 2017/04/26 18:20:15]

if these are the only allowed types, why not an enum?

[jennyz, 2017/05/16 18:22:03]

Done.

+    DOMString type;
+
+    // Content of the additional data item. Either the plain text string if
+    // |type| is "text/plain" or markup string if |type| is "text/html".
+    DOMString data;

[Devlin, 2017/04/26 18:20:15]

From the implementation, I see that this has a limit, but it's not described
here in the API?

[jennyz, 2017/05/16 18:22:03]

Done.

+  };
   
   interface Events {
     // Fired when clipboard data changes.
     // Requires clipboard and clipboardRead permissions for adding listener to
     // chrome.clipboard.onClipboardDataChanged event.
     // After this event fires, the clipboard data is available by calling
     // document.execCommand('paste').
     static void onClipboardDataChanged();
   };
 
   callback SetImageDataCallback = void();
 
   interface Functions {
     // Sets image data to clipboard.
     //
     // |image_data|: The encoded image data.
     // |type|: The type of image being passed.
+    // |additional_items|: Additional data items for describing image data.
     // The callback is called with <code>chrome.runtime.lastError</code>
     // set to error code if there is an error.
     // Requires clipboard and clipboardWrite permissions.
     static void setImageData(ArrayBuffer image_data,
                              ImageType type,
+                             AdditionalDataItem[] additional_items,

[Devlin, 2017/04/26 18:20:15]

This is not a backwards compatible change, and will break all existing clients.

[jennyz, 2017/05/16 18:22:03]

Mark the new parameter as optional.

                              SetImageDataCallback callback);
   };
 };