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

Unified Diff: media/mojo/services/mojo_renderer_service.cc

Issue 1978973002: Moves video frame callbacks from VideoFrameCompositor to Renderer. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fixes media_bit_reader_fuzzer link error Created 4 years, 7 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 33480282b483dfdbf72652909dc19269fe7de680..652ebc5aa5701205a155c23cdb84cfccae461e32 100644
--- a/media/mojo/services/mojo_renderer_service.cc
+++ b/media/mojo/services/mojo_renderer_service.cc
@@ -11,6 +11,7 @@
#include "media/base/renderer.h"
#include "media/mojo/services/demuxer_stream_provider_shim.h"
#include "media/mojo/services/mojo_cdm_service_context.h"
+#include "mojo/converters/geometry/geometry_type_converters.h"
namespace media {
@@ -129,6 +130,16 @@ void MojoRendererService::OnWaitingForDecryptionKey() {
// TODO(alokp): Plumb the event to interfaces::RendererClient. crbug/585287
}
+void MojoRendererService::OnVideoNaturalSizeChange(const gfx::Size& size) {
+ DVLOG(2) << __FUNCTION__ << "(" << size.ToString() << ")";
+ client_->OnVideoNaturalSizeChange(mojo::Size::From(size));
+}
+
+void MojoRendererService::OnVideoOpacityChange(bool opaque) {
+ DVLOG(2) << __FUNCTION__ << "(" << opaque << ")";
+ client_->OnVideoOpacityChange(opaque);
+}
+
void MojoRendererService::OnStreamReady(
const mojo::Callback<void(bool)>& callback) {
DCHECK_EQ(state_, STATE_INITIALIZING);

Powered by Google App Engine
This is Rietveld 408576698