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

Unified Diff: mojo/services/view_manager/access_policy_delegate.h

Issue 699173003: Makes ViewManagerServiceImpl track a single root (Closed) Base URL: https://github.com/domokit/mojo.git@master
Patch Set: comment Created 6 years, 1 month 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 | « no previous file | mojo/services/view_manager/connection_manager.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: mojo/services/view_manager/access_policy_delegate.h
diff --git a/mojo/services/view_manager/access_policy_delegate.h b/mojo/services/view_manager/access_policy_delegate.h
index 12c7479387c8df074cda20545d62277879b7b1e9..e32c79dee70416a35c12fe904f6585e2778d3de6 100644
--- a/mojo/services/view_manager/access_policy_delegate.h
+++ b/mojo/services/view_manager/access_policy_delegate.h
@@ -18,9 +18,8 @@ class ServerView;
// Delegate used by the AccessPolicy implementations to get state.
class AccessPolicyDelegate {
public:
- // Returns the ids of the roots views for this connection. That is, this is
- // the set of views the connection was embedded at.
- virtual const base::hash_set<Id>& GetRootsForAccessPolicy() const = 0;
+ // Returns true if |id| is the root of the connection.
+ virtual bool IsRootForAccessPolicy(const ViewId& id) const = 0;
// Returns true if |view| has been exposed to the client.
virtual bool IsViewKnownForAccessPolicy(const ServerView* view) const = 0;
« no previous file with comments | « no previous file | mojo/services/view_manager/connection_manager.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698