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

Unified Diff: chrome/browser/media/router/mojo/media_router_mojo_impl.cc

Issue 2867713002: Use OnceCallback on Mojo interfaces in //chrome/common/media_router (Closed)
Patch Set: rebase 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
Index: chrome/browser/media/router/mojo/media_router_mojo_impl.cc
diff --git a/chrome/browser/media/router/mojo/media_router_mojo_impl.cc b/chrome/browser/media/router/mojo/media_router_mojo_impl.cc
index a397a7fbcb09e5c6cb98a0ff184879c57275d3b1..aaaa4264085e37f5b23b561f1976c2d33f100a9e 100644
--- a/chrome/browser/media/router/mojo/media_router_mojo_impl.cc
+++ b/chrome/browser/media/router/mojo/media_router_mojo_impl.cc
@@ -134,8 +134,7 @@ void MediaRouterMojoImpl::OnConnectionError() {
void MediaRouterMojoImpl::RegisterMediaRouteProvider(
mojom::MediaRouteProviderPtr media_route_provider_ptr,
- const mojom::MediaRouter::RegisterMediaRouteProviderCallback&
- callback) {
+ mojom::MediaRouter::RegisterMediaRouteProviderCallback callback) {
DCHECK_CURRENTLY_ON(content::BrowserThread::UI);
#if defined(OS_WIN)
// The MRPM may have been upgraded or otherwise reload such that we could be
@@ -166,7 +165,7 @@ void MediaRouterMojoImpl::RegisterMediaRouteProvider(
// Media Router (crbug.com/687383), so we need to disable it in the provider.
config->enable_dial_discovery = !media_router::DialLocalDiscoveryEnabled();
config->enable_cast_discovery = !media_router::CastDiscoveryEnabled();
- callback.Run(instance_id_, std::move(config));
+ std::move(callback).Run(instance_id_, std::move(config));
ExecutePendingRequests();
SyncStateToMediaRouteProvider();

Powered by Google App Engine
This is Rietveld 408576698