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

Unified Diff: ash/common/wm/system_modal_container_layout_manager.h

Issue 2374613004: Use WmLayoutManager::OnChildWindowVisibilityChanged to update the system modal state. (Closed)
Patch Set: Observe WmLayoutManager::OnChildWindowVisibilityChanged to update the systemo modal state. Created 4 years, 3 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 | « no previous file | ash/common/wm/system_modal_container_layout_manager.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ash/common/wm/system_modal_container_layout_manager.h
diff --git a/ash/common/wm/system_modal_container_layout_manager.h b/ash/common/wm/system_modal_container_layout_manager.h
index 18fe75db8a35990fc0b0a375d0110cf62a85077c..7bf81c85c77cabdb79da9b819273c9ae0f032a2d 100644
--- a/ash/common/wm/system_modal_container_layout_manager.h
+++ b/ash/common/wm/system_modal_container_layout_manager.h
@@ -36,6 +36,7 @@ class ASH_EXPORT SystemModalContainerLayoutManager
bool has_window_dimmer() const { return window_dimmer_ != nullptr; }
// Overridden from WmSnapToPixelLayoutManager:
+ void OnChildWindowVisibilityChanged(WmWindow* child, bool visible) override;
void OnWindowResized() override;
void OnWindowAddedToLayout(WmWindow* child) override;
void OnWillRemoveWindowFromLayout(WmWindow* child) override;
@@ -45,7 +46,6 @@ class ASH_EXPORT SystemModalContainerLayoutManager
// Overridden from WmWindowObserver:
void OnWindowPropertyChanged(WmWindow* window,
WmWindowProperty property) override;
- void OnWindowVisibilityChanged(WmWindow* window, bool visible) override;
// Overridden from keyboard::KeyboardControllerObserver:
void OnKeyboardBoundsChanging(const gfx::Rect& new_bounds) override;
« no previous file with comments | « no previous file | ash/common/wm/system_modal_container_layout_manager.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698