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

Unified Diff: services/device_info/device_info.cc

Issue 2019743002: Yet more ApplicationDelegate/ApplicationRunner conversion. (Closed) Base URL: https://github.com/domokit/mojo.git@master
Patch Set: Created 4 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 | « examples/indirect_service/integer_service.cc ('k') | services/nacl/nonsfi/pnacl_compile.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: services/device_info/device_info.cc
diff --git a/services/device_info/device_info.cc b/services/device_info/device_info.cc
index 749f9099eecaf44dc552ccb9a742eb5316255da7..76d4def8a762035a0f6473677f3d8dec3c0b132f 100644
--- a/services/device_info/device_info.cc
+++ b/services/device_info/device_info.cc
@@ -2,13 +2,12 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
-#include <cstdlib>
-#include <memory>
+#include <stdlib.h>
#include "mojo/common/binding_set.h"
#include "mojo/public/c/system/main.h"
-#include "mojo/public/cpp/application/application_delegate.h"
-#include "mojo/public/cpp/application/application_runner.h"
+#include "mojo/public/cpp/application/application_impl_base.h"
+#include "mojo/public/cpp/application/run_application.h"
#include "mojo/public/cpp/application/service_provider_impl.h"
#include "mojo/services/device_info/interfaces/device_info.mojom.h"
@@ -18,7 +17,7 @@ namespace device_info {
// This is a native Mojo application which implements |DeviceInfo| interface for
// Linux.
-class DeviceInfo : public ApplicationDelegate, public mojo::DeviceInfo {
+class DeviceInfoApp : public ApplicationImplBase, public mojo::DeviceInfo {
public:
// We look for the 'DISPLAY' environment variable. If present, then we assume
// it to be a desktop, else we assume it to be a commandline
@@ -27,9 +26,8 @@ class DeviceInfo : public ApplicationDelegate, public mojo::DeviceInfo {
: DeviceInfo::DeviceType::HEADLESS);
}
- // |ApplicationDelegate| override.
- bool ConfigureIncomingConnection(
- ServiceProviderImpl* service_provider_impl) override {
+ // |ApplicationImplBase| override.
+ bool OnAcceptConnection(ServiceProviderImpl* service_provider_impl) override {
service_provider_impl->AddService<mojo::DeviceInfo>(
[this](const ConnectionContext& connection_context,
InterfaceRequest<mojo::DeviceInfo> device_info_request) {
@@ -47,8 +45,6 @@ class DeviceInfo : public ApplicationDelegate, public mojo::DeviceInfo {
} // namespace mojo
MojoResult MojoMain(MojoHandle application_request) {
- mojo::ApplicationRunner runner(
- std::unique_ptr<mojo::services::device_info::DeviceInfo>(
- new mojo::services::device_info::DeviceInfo()));
- return runner.Run(application_request);
+ mojo::services::device_info::DeviceInfoApp device_info_app;
+ return mojo::RunMainApplication(application_request, &device_info_app);
}
« no previous file with comments | « examples/indirect_service/integer_service.cc ('k') | services/nacl/nonsfi/pnacl_compile.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698