Add missing return statement.

cletnick · Commit Bot · commit 914ffe03702a · 2020-08-11T05:07:55.000Z
Bug: None Test: CQ Change-Id: Ic9122141316852794b1024d578eec9e6e41a63e1 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2342077 Reviewed-by: Ken Rockot <rockot@google.com> Commit-Queue: Chris Letnick <cletnick@google.com> Auto-Submit: Chris Letnick <cletnick@google.com> Cr-Commit-Position: refs/heads/master@{#796705}
diff --git a/services/service_manager/service_instance.cc b/services/service_manager/service_instance.cc
@@ -473,6 +473,7 @@ void ServiceInstance::RegisterServiceInstance(
   if (!service_manager_->RegisterService(identity, std::move(service_remote),
                                          std::move(metadata_receiver))) {
     std::move(callback).Run(mojom::ConnectResult::ACCESS_DENIED);
+    return;
   }
 
   std::move(callback).Run(mojom::ConnectResult::SUCCEEDED);

Original file line number	Diff line number	Diff line change
`@@ -473,6 +473,7 @@ void ServiceInstance::RegisterServiceInstance(`
`473`	`473`	`if (!service_manager_->RegisterService(identity, std::move(service_remote),`
`474`	`474`	`std::move(metadata_receiver))) {`
`475`	`475`	`std::move(callback).Run(mojom::ConnectResult::ACCESS_DENIED);`
	`476`	`+ return;`
`476`	`477`	`}`
`477`	`478`
`478`	`479`	`std::move(callback).Run(mojom::ConnectResult::SUCCEEDED);`