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

Unified Diff: media/mojo/clients/mojo_android_overlay.cc

Issue 2864603002: Provide callback to create mojo AndroidOverlays to AVDA. (Closed)
Patch Set: rebased Created 3 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
« no previous file with comments | « media/mojo/clients/mojo_android_overlay.h ('k') | media/mojo/clients/mojo_android_overlay_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/mojo/clients/mojo_android_overlay.cc
diff --git a/media/mojo/clients/mojo_android_overlay.cc b/media/mojo/clients/mojo_android_overlay.cc
index ab6554acb7a05799812cf74bb55b86680dd6bd85..eb03d7414ba477292c375171f882f475c5cb8cbf 100644
--- a/media/mojo/clients/mojo_android_overlay.cc
+++ b/media/mojo/clients/mojo_android_overlay.cc
@@ -6,20 +6,14 @@
#include "gpu/ipc/common/gpu_surface_lookup.h"
#include "services/service_manager/public/cpp/connect.h"
-#include "services/service_manager/public/interfaces/interface_provider.mojom.h"
namespace media {
MojoAndroidOverlay::MojoAndroidOverlay(
- service_manager::mojom::InterfaceProvider* interface_provider,
+ mojom::AndroidOverlayProviderPtr provider_ptr,
AndroidOverlayConfig config,
const base::UnguessableToken& routing_token)
- : interface_provider_(interface_provider), config_(std::move(config)) {
- // Connect to the provider service.
- mojom::AndroidOverlayProviderPtr provider_ptr;
- service_manager::GetInterface<mojom::AndroidOverlayProvider>(
- interface_provider_, &provider_ptr_);
-
+ : config_(std::move(config)) {
// Fill in details of |config| into |mojo_config|. Our caller could do this
// too, but since we want to retain |config_| anyway, we do it here.
mojom::AndroidOverlayConfigPtr mojo_config =
@@ -32,8 +26,8 @@ MojoAndroidOverlay::MojoAndroidOverlay(
binding_ = base::MakeUnique<mojo::Binding<mojom::AndroidOverlayClient>>(
this, mojo::MakeRequest(&ptr));
- provider_ptr_->CreateOverlay(mojo::MakeRequest(&overlay_ptr_), std::move(ptr),
- std::move(mojo_config));
+ provider_ptr->CreateOverlay(mojo::MakeRequest(&overlay_ptr_), std::move(ptr),
+ std::move(mojo_config));
}
MojoAndroidOverlay::~MojoAndroidOverlay() {
« no previous file with comments | « media/mojo/clients/mojo_android_overlay.h ('k') | media/mojo/clients/mojo_android_overlay_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698