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

Unified Diff: services/prediction/prediction_service_impl.cc

Issue 2015643002: Add (optional) options to Run[Main]Application(). (Closed) Base URL: https://github.com/domokit/mojo.git@master
Patch Set: doh 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
Index: services/prediction/prediction_service_impl.cc
diff --git a/services/prediction/prediction_service_impl.cc b/services/prediction/prediction_service_impl.cc
index 42bf597a130f62525b5f1ec67a1ccf63d80f8b90..ce26d6e0f8a9e70c0e1eb627262c870f5bcfe6c9 100644
--- a/services/prediction/prediction_service_impl.cc
+++ b/services/prediction/prediction_service_impl.cc
@@ -4,8 +4,9 @@
#include "services/prediction/prediction_service_impl.h"
-#include "mojo/application/application_runner_chromium.h"
#include "mojo/public/c/system/main.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"
namespace prediction {
@@ -30,24 +31,27 @@ void PredictionServiceImpl::GetPredictionList(
callback.Run(prediction_list.Pass());
}
-PredictionServiceDelegate::PredictionServiceDelegate() {}
-
-PredictionServiceDelegate::~PredictionServiceDelegate() {}
-
-bool PredictionServiceDelegate::ConfigureIncomingConnection(
- mojo::ServiceProviderImpl* service_provider_impl) {
- service_provider_impl->AddService<PredictionService>(
- [](const mojo::ConnectionContext& connection_context,
- mojo::InterfaceRequest<PredictionService> prediction_service_request) {
- new PredictionServiceImpl(prediction_service_request.Pass());
- });
- return true;
-}
+// TODO(vtl): Maybe this (and MojoMain()) should go in a different file?
+class PredictionServiceApp : public mojo::ApplicationImplBase {
+ public:
+ PredictionServiceApp() {}
+ ~PredictionServiceApp() override {}
+
+ // mojo::ApplicationImplBase override:
+ bool OnAcceptConnection(
+ mojo::ServiceProviderImpl* service_provider_impl) override {
+ service_provider_impl->AddService<PredictionService>([](
+ const mojo::ConnectionContext& connection_context,
+ mojo::InterfaceRequest<PredictionService> prediction_service_request) {
+ new PredictionServiceImpl(prediction_service_request.Pass());
+ });
+ return true;
+ }
+};
} // namespace prediction
MojoResult MojoMain(MojoHandle application_request) {
- mojo::ApplicationRunnerChromium runner(
- new prediction::PredictionServiceDelegate());
- return runner.Run(application_request);
+ prediction::PredictionServiceApp prediction_service_app;
+ return mojo::RunMainApplication(application_request, &prediction_service_app);
}
« services/media/audio/audio_server_app.cc ('K') | « services/prediction/prediction_service_impl.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698