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

Unified Diff: media/mojo/common/media_type_converters.cc

Issue 2101043003: Plumb callbacks from mojo media service to client. (Closed) Base URL: https://chromium.googlesource.com/chromium/src@master
Patch Set: Created 4 years, 6 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: media/mojo/common/media_type_converters.cc
diff --git a/media/mojo/common/media_type_converters.cc b/media/mojo/common/media_type_converters.cc
index 17e3a8fd04227929a4599524227d1d4a6b7cbb5c..16924b670b0ba1b31063fd5d1106cdc91c7e38ad 100644
--- a/media/mojo/common/media_type_converters.cc
+++ b/media/mojo/common/media_type_converters.cc
@@ -21,6 +21,7 @@
#include "media/base/demuxer_stream.h"
#include "media/base/encryption_scheme.h"
#include "media/base/media_keys.h"
+#include "media/base/pipeline_status.h"
#include "media/base/video_decoder_config.h"
#include "media/base/video_frame.h"
#include "media/mojo/common/mojo_shared_buffer_video_frame.h"
@@ -727,4 +728,33 @@ TypeConverter<scoped_refptr<media::VideoFrame>, media::mojom::VideoFramePtr>::
base::TimeDelta::FromMicroseconds(input->timestamp_usec));
}
+// static
+media::mojom::PipelineStatisticsPtr
alokp 2016/06/27 21:50:48 Is it possible to use type mapping here? https://w
alokp 2016/06/27 22:09:57 or may be just [Native]
slan 2016/06/27 22:22:55 After reviewing this feature, this approach seems
+TypeConverter<media::mojom::PipelineStatisticsPtr, media::PipelineStatistics>::
+ Convert(const media::PipelineStatistics& input) {
+ media::mojom::PipelineStatisticsPtr result(
+ media::mojom::PipelineStatistics::New());
+ result->audio_bytes_decoded = input.audio_bytes_decoded;
+ result->video_bytes_decoded = input.video_bytes_decoded;
+ result->video_frames_decoded = input.video_frames_decoded;
+ result->video_frames_dropped = input.video_frames_dropped;
+ result->audio_memory_usage = input.audio_memory_usage;
+ result->video_memory_usage = input.video_memory_usage;
+ return result;
+}
+
+// static
+media::PipelineStatistics
+TypeConverter<media::PipelineStatistics, media::mojom::PipelineStatisticsPtr>::
+ Convert(const media::mojom::PipelineStatisticsPtr& input) {
+ media::PipelineStatistics result;
+ result.audio_bytes_decoded = input->audio_bytes_decoded;
+ result.video_bytes_decoded = input->video_bytes_decoded;
+ result.video_frames_decoded = input->video_frames_decoded;
+ result.video_frames_dropped = input->video_frames_dropped;
+ result.audio_memory_usage = input->audio_memory_usage;
+ result.video_memory_usage = input->video_memory_usage;
+ return result;
+}
+
} // namespace mojo

Powered by Google App Engine
This is Rietveld 408576698