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

Unified Diff: chrome/utility/chrome_content_utility_client.cc

Issue 250143002: Media Galleries API: Audio/Video attached pictures support. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Create Blobs on browser-process, eliminating two IPC copies. Created 6 years, 8 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/utility/chrome_content_utility_client.cc
diff --git a/chrome/utility/chrome_content_utility_client.cc b/chrome/utility/chrome_content_utility_client.cc
index 7390adb3e50d85838ae4d1e3f79037af7f2f7723..291725f18b54a72a2684ef62bd0c86b314b39d66 100644
--- a/chrome/utility/chrome_content_utility_client.cc
+++ b/chrome/utility/chrome_content_utility_client.cc
@@ -68,6 +68,7 @@
#endif // defined(OS_WIN) || defined(OS_MACOSX)
#if !defined(OS_ANDROID) && !defined(OS_IOS)
+#include "chrome/common/media_galleries/metadata_types.h"
#include "chrome/utility/media_galleries/image_metadata_extractor.h"
#include "chrome/utility/media_galleries/ipc_data_source.h"
#include "chrome/utility/media_galleries/media_metadata_parser.h"
@@ -303,9 +304,10 @@ typedef PdfFunctionsBase PdfFunctions;
#if !defined(OS_ANDROID) && !defined(OS_IOS)
void FinishParseMediaMetadata(
metadata::MediaMetadataParser* parser,
- scoped_ptr<extensions::api::media_galleries::MediaMetadata> metadata) {
+ scoped_ptr<extensions::api::media_galleries::MediaMetadata> metadata,
+ const std::vector<metadata::AttachedImage>& attached_images) {
Send(new ChromeUtilityHostMsg_ParseMediaMetadata_Finished(
- true, *(metadata->ToValue().get())));
+ true, *(metadata->ToValue().get()), attached_images));
ReleaseProcessIfNeeded();
}
#endif // !defined(OS_ANDROID) && !defined(OS_IOS)
@@ -919,14 +921,13 @@ void ChromeContentUtilityClient::OnCheckMediaFile(
}
void ChromeContentUtilityClient::OnParseMediaMetadata(
- const std::string& mime_type,
- int64 total_size) {
+ const std::string& mime_type, int64 total_size, bool get_attached_images) {
// Only one IPCDataSource may be created and added to the list of handlers.
metadata::IPCDataSource* source = new metadata::IPCDataSource(total_size);
handlers_.push_back(source);
- metadata::MediaMetadataParser* parser =
- new metadata::MediaMetadataParser(source, mime_type);
+ metadata::MediaMetadataParser* parser = new metadata::MediaMetadataParser(
+ source, mime_type, get_attached_images);
parser->Start(base::Bind(&FinishParseMediaMetadata, base::Owned(parser)));
}
#endif // !defined(OS_ANDROID) && !defined(OS_IOS)

Powered by Google App Engine
This is Rietveld 408576698