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

Unified Diff: chrome/utility/media_galleries/media_metadata_parser.h

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/media_galleries/media_metadata_parser.h
diff --git a/chrome/utility/media_galleries/media_metadata_parser.h b/chrome/utility/media_galleries/media_metadata_parser.h
index 980c07a58c1c8df81891b860da8d746aa9a5c285..d9b139685cb7484189d7ade7b2dbb8ad7f1f29ee 100644
--- a/chrome/utility/media_galleries/media_metadata_parser.h
+++ b/chrome/utility/media_galleries/media_metadata_parser.h
@@ -5,9 +5,13 @@
#ifndef CHROME_UTILITY_MEDIA_GALLERIES_MEDIA_METADATA_PARSER_H_
#define CHROME_UTILITY_MEDIA_GALLERIES_MEDIA_METADATA_PARSER_H_
+#include <string>
+#include <vector>
+
#include "base/callback.h"
#include "base/memory/scoped_ptr.h"
#include "chrome/common/extensions/api/media_galleries.h"
+#include "chrome/common/media_galleries/metadata_types.h"
namespace base {
class Thread;
@@ -27,11 +31,15 @@ namespace metadata {
class MediaMetadataParser {
public:
typedef extensions::api::media_galleries::MediaMetadata MediaMetadata;
- typedef base::Callback<void(scoped_ptr<MediaMetadata>)> MetadataCallback;
+ typedef base::Callback<
+ void(scoped_ptr<MediaMetadata> metadata,
+ const std::vector<AttachedImage>& attached_images)>
+ MetadataCallback;
// Does not take ownership of |source|. Caller is responsible for ensuring
// that |source| outlives this object.
- MediaMetadataParser(media::DataSource* source, const std::string& mime_type);
+ MediaMetadataParser(media::DataSource* source, const std::string& mime_type,
+ bool get_attached_images);
~MediaMetadataParser();
@@ -44,6 +52,8 @@ class MediaMetadataParser {
const std::string mime_type_;
+ bool get_attached_images_;
+
// Thread that blocking media parsing operations run on while the main thread
// handles messages from the browser process.
// TODO(tommycli): Replace with a reference to a WorkerPool if we ever use

Powered by Google App Engine
This is Rietveld 408576698