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

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

Issue 2548513002: Update bool WindowManager::OnWmSetBounds() to match with its desirable behavior. (Closed)
Patch Set: Change gfx::Rect* to const gfx::Rect&. Created 4 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
Index: services/ui/public/cpp/window_manager_delegate.h
diff --git a/services/ui/public/cpp/window_manager_delegate.h b/services/ui/public/cpp/window_manager_delegate.h
index 51de4329f176f13a6af0d69432e57ddea6ce668a..e65b1dd32495924ae47c3e21f5fec80a18211e06 100644
--- a/services/ui/public/cpp/window_manager_delegate.h
+++ b/services/ui/public/cpp/window_manager_delegate.h
@@ -75,7 +75,7 @@ class WindowManagerDelegate {
// indicates the change is not allowed.
// NOTE: This should not change the bounds of |window|. Instead return the
// bounds the window should be in |bounds|.
- virtual bool OnWmSetBounds(Window* window, gfx::Rect* bounds) = 0;
+ virtual void OnWmSetBounds(Window* window, const gfx::Rect& bounds) = 0;
// A client requested the shared property named |name| to change to
// |new_data|. Return true to allow the change to |new_data|, false

Powered by Google App Engine
This is Rietveld 408576698