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

Unified Diff: services/ui/public/cpp/window.h

Issue 2387013003: Adds OnChildWindowVisibilityChanged to ui::WindowObserver (Closed)
Patch Set: fix test Created 4 years, 2 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: services/ui/public/cpp/window.h
diff --git a/services/ui/public/cpp/window.h b/services/ui/public/cpp/window.h
index f687a54b27168bb8685daf5c92db1d329b781020..bd3160052c896f9fc84a6d8f077806d3cf3a20d5 100644
--- a/services/ui/public/cpp/window.h
+++ b/services/ui/public/cpp/window.h
@@ -314,15 +314,15 @@ class Window {
void NotifyWindowStackingChanged();
// Methods implementing visibility change notifications. See WindowObserver
// for more details.
- void NotifyWindowVisibilityChanged(Window* target);
+ void NotifyWindowVisibilityChanged(Window* target, bool visible);
// Notifies this window's observers. Returns false if |this| was deleted
// during the call (by an observer), otherwise true.
- bool NotifyWindowVisibilityChangedAtReceiver(Window* target);
+ bool NotifyWindowVisibilityChangedAtReceiver(Window* target, bool visible);
// Notifies this window and its child hierarchy. Returns false if |this| was
// deleted during the call (by an observer), otherwise true.
- bool NotifyWindowVisibilityChangedDown(Window* target);
+ bool NotifyWindowVisibilityChangedDown(Window* target, bool visible);
// Notifies this window and its parent hierarchy.
- void NotifyWindowVisibilityChangedUp(Window* target);
+ void NotifyWindowVisibilityChangedUp(Window* target, bool visible);
// Returns true if embed is allowed for this node. If embedding is allowed all
// the children are removed.

Powered by Google App Engine
This is Rietveld 408576698