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

Unified Diff: ash/mus/workspace/workspace_layout_manager_unittest.cc

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: ash/mus/workspace/workspace_layout_manager_unittest.cc
diff --git a/ash/mus/workspace/workspace_layout_manager_unittest.cc b/ash/mus/workspace/workspace_layout_manager_unittest.cc
index 9f58d2e050c36df01118efefdabcc13e6d55d81e..f5e1799e6e63602a730a43962325e4a24306dd18 100644
--- a/ash/mus/workspace/workspace_layout_manager_unittest.cc
+++ b/ash/mus/workspace/workspace_layout_manager_unittest.cc
@@ -524,8 +524,8 @@ class FocusObserver : public ui::WindowObserver {
~FocusObserver() override { window_->RemoveObserver(this); }
// aura::test::TestWindowDelegate overrides:
- void OnWindowVisibilityChanged(ui::Window* window) override {
- if (window_->visible())
+ void OnWindowVisibilityChanged(ui::Window* window, bool visible) override {
+ if (visible)
window_->SetFocus();
show_state_ = WmWindowMus::Get(window_)->GetShowState();
}

Powered by Google App Engine
This is Rietveld 408576698