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

Unified Diff: content/common/video_capture.mojom

Issue 2409893003: VideoCapture: more migration IPC-->mojo, part 5 (Closed)
Patch Set: rebase content/common/BUILD.gn Created 4 years, 2 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
« no previous file with comments | « content/common/media/video_capture_messages.h ('k') | content/renderer/media/video_capture_impl.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/common/video_capture.mojom
diff --git a/content/common/video_capture.mojom b/content/common/video_capture.mojom
index 7179248dacf494420bf73d79a3757f0a555946d5..6848ff6f25b044b7ccfc9dfd4bdbeec3a03869b7 100644
--- a/content/common/video_capture.mojom
+++ b/content/common/video_capture.mojom
@@ -5,8 +5,11 @@
module content.mojom;
import "gpu/ipc/common/sync_token.mojom";
+import "media/mojo/interfaces/media_types.mojom";
+import "mojo/common/common_custom_types.mojom";
import "services/video_capture/public/interfaces/video_capture_device_proxy.mojom";
import "services/video_capture/public/interfaces/video_capture_format.mojom";
+import "ui/gfx/geometry/mojo/geometry.mojom";
struct VideoCaptureParams {
video_capture.mojom.VideoCaptureFormat requested_format;
@@ -14,6 +17,15 @@ struct VideoCaptureParams {
video_capture.mojom.PowerLineFrequency power_line_frequency;
};
+struct VideoFrameInfo{
+ mojo.common.mojom.TimeDelta timestamp;
+ mojo.common.mojom.DictionaryValue metadata;
+ media.mojom.VideoFormat pixel_format;
+ video_capture.mojom.VideoPixelStorage storage_type;
+ gfx.mojom.Size coded_size;
+ gfx.mojom.Rect visible_rect;
+};
+
enum VideoCaptureState {
STARTED,
PAUSED,
@@ -26,10 +38,17 @@ enum VideoCaptureState {
// Interface for notifications from Browser/Host back to Renderer/Client. This
// interface is used between VideoCaptureHost.Start() and Stop().
interface VideoCaptureObserver {
- // Gets notigied about a VideoCaptureState update.
+ // Gets notified about a VideoCaptureState update.
OnStateChanged(VideoCaptureState state);
-// TODO(mcasas): Migrate the rest of the messages, https://crbug.com/651897.
+ // |buffer_id| has video capture data with |info| containing the associated
+ // VideoFrame constituent parts.
+ OnBufferReady(int32 buffer_id, VideoFrameInfo info);
+
+ // |buffer_id| has been released by VideoCaptureHost and must not be used.
+ OnBufferDestroyed(int32 buffer_id);
+
+ // TODO(mcasas): Migrate the rest of the messages, https://crbug.com/651897.
};
interface VideoCaptureHost {
« no previous file with comments | « content/common/media/video_capture_messages.h ('k') | content/renderer/media/video_capture_impl.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698