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

Unified Diff: chrome/common/extensions/media_parser.mojom

Issue 2639503002: Convert utility process CheckMediaFile IPC to mojo (Closed)
Patch Set: Build fix: IPC::Send is now used on #ifdef WIN and OSX only. Created 3 years, 11 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/common/extensions/media_parser.mojom
diff --git a/chrome/common/extensions/media_parser.mojom b/chrome/common/extensions/media_parser.mojom
index 16cc8464ecaacdbdfbc829d6b8649765e8fea387..c1e4e05842cd6df9a48a527fc35476fd20d5c51f 100644
--- a/chrome/common/extensions/media_parser.mojom
+++ b/chrome/common/extensions/media_parser.mojom
@@ -7,6 +7,7 @@
module extensions.mojom;
+import "mojo/common/file.mojom";
import "mojo/common/values.mojom";
struct AttachedImage {
@@ -21,4 +22,12 @@ interface MediaParser {
=> (bool parse_success,
mojo.common.mojom.DictionaryValue metadata,
array<AttachedImage> attached_images);
+
+ // Validate the passed media file with sanity checks, and file decoding
+ // for up to |decode_time_ms| wall clock time. Return |success| true if
+ // |file| appears to be a well-formed media file. Note: it is still not
+ // safe to decode the file in the browser process based on the returned
+ // |success|.
+ CheckMediaFile(int64 decode_time_ms, mojo.common.mojom.File file)
Sam McNally 2017/01/18 07:09:09 Use a mojo.common.mojom.TimeDelta for |decode_time
Noel Gordon 2017/01/19 06:33:11 Done (and done).
+ => (bool success);
};

Powered by Google App Engine
This is Rietveld 408576698