Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(11)

Unified Diff: extensions/browser/api/clipboard/clipboard_api.h

Issue 2379573008: Add SetImageData api to chrome.clipboard. (Closed)
Patch Set: Address code review comments and add test cases. Created 4 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: extensions/browser/api/clipboard/clipboard_api.h
diff --git a/extensions/browser/api/clipboard/clipboard_api.h b/extensions/browser/api/clipboard/clipboard_api.h
index 57d2a4c8cbb05209ab1498464d2bc468ad422d54..8d0d759e7b074ac15f1366476f471d6af3391167 100644
--- a/extensions/browser/api/clipboard/clipboard_api.h
+++ b/extensions/browser/api/clipboard/clipboard_api.h
@@ -7,10 +7,13 @@
#include "extensions/browser/browser_context_keyed_api_factory.h"
#include "extensions/browser/extension_function.h"
+#include "extensions/common/api/clipboard.h"
#include "ui/base/clipboard/clipboard_observer.h"
namespace extensions {
+namespace clipboard = api::clipboard;
Devlin 2016/12/09 15:23:53 See comment here. https://codereview.chromium.org
jennyz 2016/12/14 01:15:36 Done.
+
class ClipboardAPI : public BrowserContextKeyedAPI,
public ui::ClipboardObserver {
public:
@@ -32,6 +35,20 @@ class ClipboardAPI : public BrowserContextKeyedAPI,
content::BrowserContext* const browser_context_;
};
+class ClipboardSetImageDataFunction : public UIThreadExtensionFunction {
+ public:
+ DECLARE_EXTENSION_FUNCTION("clipboard.setImageData", CLIPBOARD_SETIMAGEDATA);
+
+ protected:
+ ~ClipboardSetImageDataFunction() override;
+ ResponseAction Run() override;
+
+ void SaveImageData(const std::vector<char>& image_data,
+ clipboard::ImageType type);
+ void OnSaveImageDataSuccess();
+ void OnSaveImageDataError();
+};
+
} // namespace extensions
#endif // EXTENSIONS_BROWSER_API_CLIPBOARD_CLIPBOARD_API_H_

Powered by Google App Engine
This is Rietveld 408576698