media/mojo/interfaces/media_types.mojom - Issue 2101043003: Plumb callbacks from mojo media service to client.

Side by Side Diff

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

Keyboard Shortcuts

	File
u :	up to issue
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line

	Issue
u :	up to list of issues
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue

Side by Side Diff: media/mojo/interfaces/media_types.mojom

Issue 2101043003: Plumb callbacks from mojo media service to client. (Closed) Base URL: https://chromium.googlesource.com/chromium/src@master

Patch Set: Remove unneeded deps from typemap file. Created 4 years, 5 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 unified diff | Download patch

OLD	NEW
1 // Copyright 2014 The Chromium Authors. All rights reserved.	1 // Copyright 2014 The Chromium Authors. All rights reserved.

2 // Use of this source code is governed by a BSD-style license that can be	2 // Use of this source code is governed by a BSD-style license that can be

3 // found in the LICENSE file.	3 // found in the LICENSE file.

4	4

5 module media.mojom;	5 module media.mojom;

6	6

7 import "ui/gfx/geometry/mojo/geometry.mojom";	7 import "ui/gfx/geometry/mojo/geometry.mojom";

8	8

9 // See media/base/buffering_state.h for descriptions.	9 // See media/base/buffering_state.h for descriptions.

10 // Kept in sync with media::BufferingState via static_asserts.	10 // Kept in sync with media::BufferingState via static_asserts.

(...skipping 318 matching lines...) Expand 10 before \| Expand all \| Expand 10 after Loading...
329	329

330 // Stride and offsets for each plane. Offsets are relative to the start	330 // Stride and offsets for each plane. Offsets are relative to the start

331 // of \|frame_data\|.	331 // of \|frame_data\|.

332 int32 y_stride;	332 int32 y_stride;

333 int32 u_stride;	333 int32 u_stride;

334 int32 v_stride;	334 int32 v_stride;

335 uint64 y_offset;	335 uint64 y_offset;

336 uint64 u_offset;	336 uint64 u_offset;

337 uint64 v_offset;	337 uint64 v_offset;

338 };	338 };

	339

	340 struct PipelineStatistics {

	341 uint64 audio_bytes_decoded;

	342 uint64 video_bytes_decoded;

	343 uint32 video_frames_decoded;

	344 uint32 video_frames_dropped;

	345 int64 audio_memory_usage;
	dcheng 2016/07/07 04:19:37 Is there a reason for this to be signed (other tha Is there a reason for this to be signed (other than the fact that the existing one in C++ is signed)? Can memory usage be reported as negative for some reason? slan 2016/07/07 16:41:45 Original justification: https://codereview.chromiu Show quoted text On 2016/07/07 04:19:37, dcheng wrote: > Is there a reason for this to be signed (other than the fact that the existing > one in C++ is signed)? Can memory usage be reported as negative for some reason? Original justification: https://codereview.chromium.org/1409123005/diff/1/media/filters/chunk_demuxer.cc This is now used to calculate a delta in WebMediaPlayer, which must be expressed as int64_t, so it avoids a cast there: https://cs.chromium.org/chromium/src/media/blink/webmediaplayer_impl.cc?rcl=0...
	346 int64 video_memory_usage;

	347 };

OLD	NEW

« no previous file with comments | « media/mojo/common/pipeline_statistics_struct_traits.h ('k') | media/mojo/interfaces/pipeline_statistics.typemap » ('j') | no next file with comments »