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

Side by Side Diff: services/log/main.cc

Issue 1928613002: Simplify LogImpl a bit. (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 unified diff | Download patch
« no previous file with comments | « services/log/log_impl_unittest.cc ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include <stdio.h>
6
5 #include <utility> 7 #include <utility>
6 8
7 #include "mojo/application/application_runner_chromium.h" 9 #include "mojo/application/application_runner_chromium.h"
8 #include "mojo/public/c/system/main.h" 10 #include "mojo/public/c/system/main.h"
9 #include "mojo/public/cpp/application/application_connection.h" 11 #include "mojo/public/cpp/application/application_connection.h"
10 #include "mojo/public/cpp/application/application_delegate.h" 12 #include "mojo/public/cpp/application/application_delegate.h"
11 #include "mojo/public/cpp/application/interface_factory.h" 13 #include "mojo/public/cpp/application/interface_factory.h"
12 #include "mojo/public/cpp/system/macros.h" 14 #include "mojo/public/cpp/system/macros.h"
13 #include "mojo/services/log/interfaces/log.mojom.h" 15 #include "mojo/services/log/interfaces/log.mojom.h"
14 #include "services/log/log_impl.h" 16 #include "services/log/log_impl.h"
(...skipping 12 matching lines...) Expand all
27 // |ApplicationDelegate| override: 29 // |ApplicationDelegate| override:
28 bool ConfigureIncomingConnection(ApplicationConnection* connection) override { 30 bool ConfigureIncomingConnection(ApplicationConnection* connection) override {
29 connection->AddService<Log>(this); 31 connection->AddService<Log>(this);
30 return true; 32 return true;
31 } 33 }
32 34
33 // |InterfaceFactory<Log>| implementation: 35 // |InterfaceFactory<Log>| implementation:
34 // We maintain a separate |LogImpl| for each incoming connection. 36 // We maintain a separate |LogImpl| for each incoming connection.
35 void Create(ApplicationConnection* connection, 37 void Create(ApplicationConnection* connection,
36 InterfaceRequest<Log> request) override { 38 InterfaceRequest<Log> request) override {
37 LogImpl::Create(connection, std::move(request), stderr); 39 LogImpl::Create(connection, std::move(request),
40 [](const std::string& message) {
41 fprintf(stderr, "%s\n", message.c_str());
42 });
38 } 43 }
39 44
40 MOJO_DISALLOW_COPY_AND_ASSIGN(LogApp); 45 MOJO_DISALLOW_COPY_AND_ASSIGN(LogApp);
41 }; 46 };
42 47
43 } // namespace log 48 } // namespace log
44 } // namespace mojo 49 } // namespace mojo
45 50
46 MojoResult MojoMain(MojoHandle application_request) { 51 MojoResult MojoMain(MojoHandle application_request) {
47 mojo::ApplicationRunnerChromium runner(new mojo::log::LogApp()); 52 mojo::ApplicationRunnerChromium runner(new mojo::log::LogApp());
48 return runner.Run(application_request); 53 return runner.Run(application_request);
49 } 54 }
OLDNEW
« no previous file with comments | « services/log/log_impl_unittest.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698