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

Side by Side Diff: ui/aura/mus/window_manager_delegate.h

Issue 2778823002: Simplify WindowManager::OnWmSetBounds (Closed)
Patch Set: 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 unified diff | Download patch
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef UI_AURA_MUS_WINDOW_MANAGER_DELEGATE_H_ 5 #ifndef UI_AURA_MUS_WINDOW_MANAGER_DELEGATE_H_
6 #define UI_AURA_MUS_WINDOW_MANAGER_DELEGATE_H_ 6 #define UI_AURA_MUS_WINDOW_MANAGER_DELEGATE_H_
7 7
8 #include <stdint.h> 8 #include <stdint.h>
9 9
10 #include <map> 10 #include <map>
(...skipping 63 matching lines...) Expand 10 before | Expand all | Expand 10 after
74 public: 74 public:
75 // Called once to give the delegate access to functions only exposed to 75 // Called once to give the delegate access to functions only exposed to
76 // the WindowManager. 76 // the WindowManager.
77 virtual void SetWindowManagerClient(WindowManagerClient* client) = 0; 77 virtual void SetWindowManagerClient(WindowManagerClient* client) = 0;
78 78
79 // A client requested the bounds of |window| to change to |bounds|. Return 79 // A client requested the bounds of |window| to change to |bounds|. Return
80 // true if the bounds are allowed to change. A return value of false 80 // true if the bounds are allowed to change. A return value of false
81 // indicates the change is not allowed. 81 // indicates the change is not allowed.
82 // NOTE: This should not change the bounds of |window|. Instead return the 82 // NOTE: This should not change the bounds of |window|. Instead return the
83 // bounds the window should be in |bounds|. 83 // bounds the window should be in |bounds|.
84 virtual bool OnWmSetBounds(Window* window, gfx::Rect* bounds) = 0; 84 virtual void OnWmSetBounds(Window* window, gfx::Rect* bounds) = 0;
85 85
86 // A client requested the shared property named |name| to change to 86 // A client requested the shared property named |name| to change to
87 // |new_data|. Return true to allow the change to |new_data|, false 87 // |new_data|. Return true to allow the change to |new_data|, false
88 // otherwise. If true is returned the property is set via 88 // otherwise. If true is returned the property is set via
89 // PropertyConverter::SetPropertyFromTransportValue(). 89 // PropertyConverter::SetPropertyFromTransportValue().
90 virtual bool OnWmSetProperty( 90 virtual bool OnWmSetProperty(
91 Window* window, 91 Window* window,
92 const std::string& name, 92 const std::string& name,
93 std::unique_ptr<std::vector<uint8_t>>* new_data) = 0; 93 std::unique_ptr<std::vector<uint8_t>>* new_data) = 0;
94 94
(...skipping 88 matching lines...) Expand 10 before | Expand all | Expand 10 after
183 // window. 183 // window.
184 virtual void OnWmDeactivateWindow(Window* window) = 0; 184 virtual void OnWmDeactivateWindow(Window* window) = 0;
185 185
186 protected: 186 protected:
187 virtual ~WindowManagerDelegate() {} 187 virtual ~WindowManagerDelegate() {}
188 }; 188 };
189 189
190 } // namespace ui 190 } // namespace ui
191 191
192 #endif // UI_AURA_MUS_WINDOW_MANAGER_DELEGATE_H_ 192 #endif // UI_AURA_MUS_WINDOW_MANAGER_DELEGATE_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698