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

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 bool to void. 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 2f724fc877fe9646be6ea52adc7f75ce30ea7b8d..3b0f6301f0cd0162d99f7d48b741731b4bce22f0 100644
--- a/services/ui/public/cpp/window_manager_delegate.h
+++ b/services/ui/public/cpp/window_manager_delegate.h
@@ -76,7 +76,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, 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