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

Unified Diff: chrome/browser/chromeos/arc/gpu_arc_video_service_host.cc

Issue 2133503002: arc: Revamp the ArcBridgeService interface (Closed) Base URL: https://chromium.googlesource.com/a/chromium/src.git@master
Patch Set: Fix ui_arc_unittests 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/chromeos/arc/gpu_arc_video_service_host.cc
diff --git a/chrome/browser/chromeos/arc/gpu_arc_video_service_host.cc b/chrome/browser/chromeos/arc/gpu_arc_video_service_host.cc
index b2e9a8bed709c13fedeba4b641d59a54e9639311..25fd1224b2044611a36bf55595aec3b2fa4743d0 100644
--- a/chrome/browser/chromeos/arc/gpu_arc_video_service_host.cc
+++ b/chrome/browser/chromeos/arc/gpu_arc_video_service_host.cc
@@ -4,6 +4,8 @@
#include "chrome/browser/chromeos/arc/gpu_arc_video_service_host.h"
+#include <string>
+
#include "base/location.h"
#include "base/logging.h"
#include "base/message_loop/message_loop.h"
@@ -60,17 +62,18 @@ class VideoAcceleratorFactoryService : public mojom::VideoAcceleratorFactory {
GpuArcVideoServiceHost::GpuArcVideoServiceHost(
arc::ArcBridgeService* bridge_service)
: ArcService(bridge_service), binding_(this) {
- arc_bridge_service()->AddObserver(this);
+ arc_bridge_service()->video()->AddObserver(this);
}
GpuArcVideoServiceHost::~GpuArcVideoServiceHost() {
DCHECK(thread_checker_.CalledOnValidThread());
- arc_bridge_service()->RemoveObserver(this);
+ arc_bridge_service()->video()->RemoveObserver(this);
}
-void GpuArcVideoServiceHost::OnVideoInstanceReady() {
+void GpuArcVideoServiceHost::OnInstanceReady(
+ mojom::VideoInstance* video_instance,
+ uint32_t version) {
DCHECK(thread_checker_.CalledOnValidThread());
- auto video_instance = arc_bridge_service()->video_instance();
DCHECK(video_instance);
video_instance->Init(binding_.CreateInterfacePtrAndBind());
}

Powered by Google App Engine
This is Rietveld 408576698