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

Unified Diff: services/ui/ws/window_server_test_base.cc

Issue 2778823002: Simplify WindowManager::OnWmSetBounds (Closed)
Patch Set: Update expectations for DragTestInteractive.DragTest Created 3 years, 9 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 | « services/ui/ws/window_server_test_base.h ('k') | services/ui/ws/window_tree.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: services/ui/ws/window_server_test_base.cc
diff --git a/services/ui/ws/window_server_test_base.cc b/services/ui/ws/window_server_test_base.cc
index 334f63d026be6d509c4033c72d18951febecc693..34f099670fa0d2bfefa01ba9eb4b8a41d6e4c899 100644
--- a/services/ui/ws/window_server_test_base.cc
+++ b/services/ui/ws/window_server_test_base.cc
@@ -152,11 +152,11 @@ void WindowServerTestBase::SetWindowManagerClient(
window_manager_client_ = client;
}
-bool WindowServerTestBase::OnWmSetBounds(aura::Window* window,
- gfx::Rect* bounds) {
- return window_manager_delegate_
- ? window_manager_delegate_->OnWmSetBounds(window, bounds)
- : true;
+void WindowServerTestBase::OnWmSetBounds(aura::Window* window,
+ const gfx::Rect& bounds) {
+ if (!window_manager_delegate_)
+ return;
+ window_manager_delegate_->OnWmSetBounds(window, bounds);
}
bool WindowServerTestBase::OnWmSetProperty(
« no previous file with comments | « services/ui/ws/window_server_test_base.h ('k') | services/ui/ws/window_tree.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698