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

Unified Diff: media/mojo/services/mojo_renderer_service.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/services/mojo_renderer_service.cc
diff --git a/media/mojo/services/mojo_renderer_service.cc b/media/mojo/services/mojo_renderer_service.cc
index 5d75037717ddbf1c1c740f2bd094fb6db27daa56..34df9b984edd1595d35df9622b64209571889654 100644
--- a/media/mojo/services/mojo_renderer_service.cc
+++ b/media/mojo/services/mojo_renderer_service.cc
@@ -9,6 +9,7 @@
#include "base/bind.h"
#include "media/base/media_keys.h"
#include "media/base/renderer.h"
+#include "media/mojo/common/media_type_converters.h"
#include "media/mojo/services/demuxer_stream_provider_shim.h"
#include "media/mojo/services/mojo_cdm_service_context.h"
@@ -115,7 +116,8 @@ void MojoRendererService::OnEnded() {
}
void MojoRendererService::OnStatisticsUpdate(const PipelineStatistics& stats) {
- // TODO(alokp): Plumb the event to mojom::RendererClient. crbug/585287
+ DVLOG(2) << __FUNCTION__;
alokp 2016/06/27 21:50:49 make the log levels consistent with MojoRenderer.
slan 2016/06/27 22:19:39 Done.
+ client_->OnStatisticsUpdate(mojom::PipelineStatistics::From(stats));
}
void MojoRendererService::OnBufferingStateChange(BufferingState state) {
@@ -124,7 +126,8 @@ void MojoRendererService::OnBufferingStateChange(BufferingState state) {
}
void MojoRendererService::OnWaitingForDecryptionKey() {
- // TODO(alokp): Plumb the event to mojom::RendererClient. crbug/585287
+ DVLOG(2) << __FUNCTION__;
+ client_->OnWaitingForDecryptionKey();
}
void MojoRendererService::OnVideoNaturalSizeChange(const gfx::Size& size) {
« media/mojo/interfaces/renderer.mojom ('K') | « media/mojo/interfaces/renderer.mojom ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698