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

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

Issue 513923004: More viewmanager renaming: (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: sim30 Created 6 years, 4 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
« no previous file with comments | « mojo/services/view_manager/BUILD.gn ('k') | mojo/services/view_manager/access_policy_delegate.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: mojo/services/view_manager/access_policy.h
diff --git a/mojo/services/view_manager/access_policy.h b/mojo/services/view_manager/access_policy.h
index 8c4e84d4026445c31add0f2d332ca4710514de82..66167ca720f1c944c8588776c7f79b40e8ad5115 100644
--- a/mojo/services/view_manager/access_policy.h
+++ b/mojo/services/view_manager/access_policy.h
@@ -11,8 +11,7 @@
namespace mojo {
namespace service {
-class Node;
-class View;
+class ServerView;
// AccessPolicy is used by ViewManagerServiceImpl to determine what a connection
// is allowed to do.
@@ -21,32 +20,31 @@ class AccessPolicy {
virtual ~AccessPolicy() {}
// Unless otherwise mentioned all arguments have been validated. That is the
- // |node| and/or |view| arguments are non-null unless otherwise stated (eg
- // CanSetView() is allowed to take a NULL view).
- virtual bool CanRemoveNodeFromParent(const Node* node) const = 0;
- virtual bool CanAddNode(const Node* parent, const Node* child) const = 0;
- virtual bool CanReorderNode(const Node* node,
- const Node* relative_node,
+ // |view| arguments are non-null unless otherwise stated (eg CanSetView() is
+ // allowed to take a NULL view).
+ virtual bool CanRemoveViewFromParent(const ServerView* view) const = 0;
+ virtual bool CanAddView(const ServerView* parent,
+ const ServerView* child) const = 0;
+ virtual bool CanReorderView(const ServerView* view,
+ const ServerView* relative_view,
OrderDirection direction) const = 0;
- virtual bool CanDeleteNode(const Node* node) const = 0;
- virtual bool CanGetNodeTree(const Node* node) const = 0;
- // Used when building a node tree (GetNodeTree()) to decide if we should
- // descend into |node|.
- virtual bool CanDescendIntoNodeForNodeTree(const Node* node) const = 0;
- virtual bool CanEmbed(const Node* node) const = 0;
- virtual bool CanChangeNodeVisibility(const Node* node) const = 0;
- virtual bool CanSetNodeContents(const Node* node) const = 0;
- virtual bool CanSetNodeBounds(const Node* node) const = 0;
+ virtual bool CanDeleteView(const ServerView* view) const = 0;
+ virtual bool CanGetViewTree(const ServerView* view) const = 0;
+ // Used when building a view tree (GetViewTree()) to decide if we should
+ // descend into |view|.
+ virtual bool CanDescendIntoViewForViewTree(const ServerView* view) const = 0;
+ virtual bool CanEmbed(const ServerView* view) const = 0;
+ virtual bool CanChangeViewVisibility(const ServerView* view) const = 0;
+ virtual bool CanSetViewContents(const ServerView* view) const = 0;
+ virtual bool CanSetViewBounds(const ServerView* view) const = 0;
// Returns whether the connection should notify on a hierarchy change.
// |new_parent| and |old_parent| are initially set to the new and old parents
- // but may be altered so that the client only sees a certain set of nodes.
+ // but may be altered so that the client only sees a certain set of views.
virtual bool ShouldNotifyOnHierarchyChange(
- const Node* node,
- const Node** new_parent,
- const Node** old_parent) const = 0;
-
- virtual bool ShouldSendViewDeleted(const ViewId& view_id) const = 0;
+ const ServerView* view,
+ const ServerView** new_parent,
+ const ServerView** old_parent) const = 0;
};
} // namespace service
« no previous file with comments | « mojo/services/view_manager/BUILD.gn ('k') | mojo/services/view_manager/access_policy_delegate.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698