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

Unified Diff: mojo/services/ui/views/interfaces/view_provider.mojom

Issue 1679023006: Reify view ownership as a message pipe. (Closed) Base URL: git@github.com:domokit/mojo.git@master
Patch Set: Created 4 years, 10 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: mojo/services/ui/views/interfaces/view_provider.mojom
diff --git a/mojo/services/ui/views/interfaces/view_provider.mojom b/mojo/services/ui/views/interfaces/view_provider.mojom
index a1062dec2a8f8d31f0d98dd66ee51c7579529b0c..d719f010d679363a07a70bc3162c0b2fab0ab3a8 100644
--- a/mojo/services/ui/views/interfaces/view_provider.mojom
+++ b/mojo/services/ui/views/interfaces/view_provider.mojom
@@ -15,17 +15,19 @@ import "mojo/services/ui/views/interfaces/views.mojom";
[ServiceName="mojo::ui::ViewProvider"]
interface ViewProvider {
// Creates and registers a view with the view manager and returns its
- // view token (as provided by |ViewManager.RegisterView()|).
+ // view owner which may subsequently be passed to |ViewHost.AddChild()|
+ // to attach the view to a view hierarchy.
//
- // Having received the view token, the caller should attach the view to
- // a view tree and lay it out.
+ // Implementors of this interface are responsible for creating the view
+ // and forwarding the |view_owner| interface request to
+ // |ViewManager.RegisterView()|.
//
// The caller may provide services to the view via the |services|
// service provider.
//
// The caller may receive services from the view via the |exposed_services|
// service provider.
- CreateView(mojo.ServiceProvider&? services,
- mojo.ServiceProvider? exposed_services) =>
- (ViewToken view_token);
+ CreateView(ViewOwner& view_owner,
+ mojo.ServiceProvider&? services,
+ mojo.ServiceProvider? exposed_services);
};
« no previous file with comments | « mojo/services/ui/views/interfaces/view_manager.mojom ('k') | mojo/services/ui/views/interfaces/view_trees.mojom » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698