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

Unified Diff: media/mojo/clients/mojo_renderer.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/clients/mojo_renderer.cc
diff --git a/media/mojo/clients/mojo_renderer.cc b/media/mojo/clients/mojo_renderer.cc
index 94b2a57602336995df035ef4be2c1abfe81aa472..6e362fbf855e77b5146dda733d023401d335c7ce 100644
--- a/media/mojo/clients/mojo_renderer.cc
+++ b/media/mojo/clients/mojo_renderer.cc
@@ -11,9 +11,11 @@
#include "base/location.h"
#include "base/single_thread_task_runner.h"
#include "media/base/demuxer_stream_provider.h"
+#include "media/base/pipeline_status.h"
#include "media/base/renderer_client.h"
#include "media/base/video_renderer_sink.h"
#include "media/mojo/clients/mojo_demuxer_stream_impl.h"
+#include "media/mojo/common/media_type_converters.h"
#include "media/renderers/video_overlay_factory.h"
namespace media {
@@ -239,6 +241,18 @@ void MojoRenderer::OnVideoOpacityChange(bool opaque) {
client_->OnVideoOpacityChange(opaque);
}
+void MojoRenderer::OnStatisticsUpdate(mojom::PipelineStatisticsPtr stats) {
+ DVLOG(2) << __FUNCTION__;
alokp 2016/06/27 21:50:48 This fires too often. I would use a level 3 here.
slan 2016/06/27 22:19:39 Done.
+ DCHECK(task_runner_->BelongsToCurrentThread());
+ client_->OnStatisticsUpdate(stats.To<PipelineStatistics>());
+}
+
+void MojoRenderer::OnWaitingForDecryptionKey() {
+ DVLOG(2) << __FUNCTION__;
alokp 2016/06/27 21:50:48 level 1 here.
slan 2016/06/27 22:19:39 Done.
+ DCHECK(task_runner_->BelongsToCurrentThread());
+ client_->OnWaitingForDecryptionKey();
+}
+
void MojoRenderer::OnConnectionError() {
DVLOG(1) << __FUNCTION__;
DCHECK(task_runner_->BelongsToCurrentThread());

Powered by Google App Engine
This is Rietveld 408576698