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

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

Issue 2639503002: Convert utility process CheckMediaFile IPC to mojo (Closed)
Patch Set: Removed unused #include files, add TODO. 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..cc700c5acc0b171df1515bc1316ba8bb39c67785 100644
--- a/chrome/common/extensions/media_parser.mojom
+++ b/chrome/common/extensions/media_parser.mojom
@@ -7,6 +7,8 @@
module extensions.mojom;
+import "mojo/common/file.mojom";
+import "mojo/common/time.mojom";
import "mojo/common/values.mojom";
struct AttachedImage {
@@ -21,4 +23,13 @@ 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 at most |decode_time| wall clock time. Returns |success| true if
+ // |file| appears to be a well-formed media file, false otherwise.
+ // Note: it is still not safe to decode the file in the browser process
+ // after this check.
+ CheckMediaFile(mojo.common.mojom.TimeDelta decode_time,
+ mojo.common.mojom.File file)
+ => (bool success);
};
« no previous file with comments | « chrome/common/extensions/chrome_utility_extensions_messages.h ('k') | chrome/utility/extensions/extensions_handler.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698