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

Side by Side Diff: mojo/public/cpp/application/connection_context.h

Issue 2034383003: Removed exposed_services from mojom definitions. (Closed) Base URL: https://github.com/domokit/mojo.git@master
Patch Set: Created 4 years, 6 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
OLDNEW
1 // Copyright 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 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 #ifndef MOJO_PUBLIC_APPLICATION_CONNECTION_CONTEXT_H_ 5 #ifndef MOJO_PUBLIC_APPLICATION_CONNECTION_CONTEXT_H_
6 #define MOJO_PUBLIC_APPLICATION_CONNECTION_CONTEXT_H_ 6 #define MOJO_PUBLIC_APPLICATION_CONNECTION_CONTEXT_H_
7 7
8 #include <string> 8 #include <string>
9 9
10 namespace mojo { 10 namespace mojo {
11 11
12 // A |ConnectionContext| is used to track context about a given message pipe 12 // A |ConnectionContext| is used to track context about a given message pipe
13 // connection. It is usually associated to the "impl" side of an interface. 13 // connection. It is usually associated to the "impl" side of an interface.
14 // 14 //
15 // |Application| (see //mojo/public/interfaces/application/application.mojom) 15 // |Application| (see //mojo/public/interfaces/application/application.mojom)
16 // accepts a message that looks like: 16 // accepts a message that looks like:
17 // 17 //
18 // AcceptConnection(string requestor_url, 18 // AcceptConnection(string requestor_url,
19 // string resolved_url, 19 // string resolved_url,
20 // ServiceProvider& services); 20 // ServiceProvider& services);
21 // 21 //
22 // (TODO(vtl): I've jumped the gun: I've pre-removed |exposed_services|, made
23 // |services| non-nullable, and reordered the parameters.)
24 //
25 // Upon handling |AcceptConnection()|, the |ServiceProvider| implementation 22 // Upon handling |AcceptConnection()|, the |ServiceProvider| implementation
26 // bound to |services| is given a |ConnectionContext| with |type = 23 // bound to |services| is given a |ConnectionContext| with |type =
27 // Type::INCOMING|, |remote_url = requestor_url|, and |connection_url = 24 // Type::INCOMING|, |remote_url = requestor_url|, and |connection_url =
28 // resolved_url|. 25 // resolved_url|.
29 // 26 //
30 // The |ConnectionContext| is meant to be propagated: If the remote side uses 27 // The |ConnectionContext| is meant to be propagated: If the remote side uses
31 // its |ServiceProviderPtr| to request a |Foo| (from the local side), then the 28 // its |ServiceProviderPtr| to request a |Foo| (from the local side), then the
32 // |Foo| implementation (again, on the local side) may be given (a copy of) the 29 // |Foo| implementation (again, on the local side) may be given (a copy of) the
33 // |ConnectionContext| described above. 30 // |ConnectionContext| described above.
34 struct ConnectionContext { 31 struct ConnectionContext {
(...skipping 15 matching lines...) Expand all
50 std::string remote_url; 47 std::string remote_url;
51 48
52 // The URL used by the remote side to establish "this" connection (or a parent 49 // The URL used by the remote side to establish "this" connection (or a parent
53 // thereof); if unknown/not applicable, this is empty. 50 // thereof); if unknown/not applicable, this is empty.
54 std::string connection_url; 51 std::string connection_url;
55 }; 52 };
56 53
57 } // namespace mojo 54 } // namespace mojo
58 55
59 #endif // MOJO_PUBLIC_APPLICATION_CONNECTION_CONTEXT_H_ 56 #endif // MOJO_PUBLIC_APPLICATION_CONNECTION_CONTEXT_H_
OLDNEW
« no previous file with comments | « mojo/public/cpp/application/connect.h ('k') | mojo/public/cpp/application/lib/application_impl_base.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698