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

Unified Diff: chrome/renderer/resources/extensions/media_galleries_custom_bindings.js

Issue 250143002: Media Galleries API: Audio/Video attached pictures support. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: fix linux_chromium_chromeos_rel argument evaluation order bug Created 6 years, 7 months 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: chrome/renderer/resources/extensions/media_galleries_custom_bindings.js
diff --git a/chrome/renderer/resources/extensions/media_galleries_custom_bindings.js b/chrome/renderer/resources/extensions/media_galleries_custom_bindings.js
index 9b51fcc271685b3f9afbc0011f1aa4decbeb0cdd..f838086848c1b555f6a5e490288202d191f99a08 100644
--- a/chrome/renderer/resources/extensions/media_galleries_custom_bindings.js
+++ b/chrome/renderer/resources/extensions/media_galleries_custom_bindings.js
@@ -103,6 +103,22 @@ binding.registerCustomHook(function(bindingsAPI, extensionId) {
var blobUuid = blobNatives.GetBlobUuid(mediaFile)
return [blobUuid, options, callback];
});
+
+ apiFunctions.setCustomCallback('getMetadata',
+ function(name, request, response) {
+ if (response.attachedImagesBlobInfo) {
+ for (var i = 0; i < response.attachedImagesBlobInfo.length; i++) {
+ var blobInfo = response.attachedImagesBlobInfo[i];
+ var blob = blobNatives.TakeBrowserProcessBlob(
+ blobInfo.blobUUID, blobInfo.type, blobInfo.size);
+ response.metadata.attachedImages.push(blob);
+ }
+ }
+
+ if (request.callback)
+ request.callback(response.metadata);
+ request.callback = null;
+ });
});
exports.binding = binding.generate();

Powered by Google App Engine
This is Rietveld 408576698