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

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. +#include 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 9b778029530627e63ca792ea57f11021be4dd65d..a4d35e8dd8730778f9082c718e99ede4f2fa7b7e 100644
--- a/chrome/browser/media/router/mojo/media_router_mojo_impl.cc
+++ b/chrome/browser/media/router/mojo/media_router_mojo_impl.cc
@@ -127,8 +127,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
@@ -152,7 +151,7 @@ void MediaRouterMojoImpl::RegisterMediaRouteProvider(
media_route_provider_ = std::move(media_route_provider_ptr);
media_route_provider_.set_connection_error_handler(base::Bind(
&MediaRouterMojoImpl::OnConnectionError, base::Unretained(this)));
- callback.Run(instance_id_);
+ std::move(callback).Run(instance_id_);
ExecutePendingRequests();
SyncStateToMediaRouteProvider();

Powered by Google App Engine
This is Rietveld 408576698