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

Unified Diff: services/window_manager/focus_rules.h

Issue 747163002: Port more focus controller unit tests and fix our focus rules. (Closed) Base URL: https://github.com/domokit/mojo.git@master
Patch Set: Remove O(n) solution with O(1). Created 6 years 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 | « services/window_manager/focus_controller_unittest.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: services/window_manager/focus_rules.h
diff --git a/services/window_manager/focus_rules.h b/services/window_manager/focus_rules.h
index 3d97f0c1e0341a971f34feb91f34ed7556732553..83fd1d59bdb58f13b2ad5104da2fb17fe20c910d 100644
--- a/services/window_manager/focus_rules.h
+++ b/services/window_manager/focus_rules.h
@@ -16,6 +16,9 @@ class FocusRules {
public:
virtual ~FocusRules() {}
+ // Returns true if the children of |window| can be activated.
+ virtual bool SupportsChildActivation(View* window) const = 0;
+
// Returns true if |view| is a toplevel view. Whether or not a view
// is considered toplevel is determined by a similar set of rules that
// govern activation and focus. Not all toplevel views are activatable,
« no previous file with comments | « services/window_manager/focus_controller_unittest.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698