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

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

Issue 2500973002: Converts test_wm to use aura (Closed)
Patch Set: oopsie Created 4 years, 1 month 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/ws/window_tree.cc
diff --git a/services/ui/ws/window_tree.cc b/services/ui/ws/window_tree.cc
index 9b6459540c5dd71d5ca6405db13a846b15f02e70..1803d2f0d42902e381e695697c50603c065aca56 100644
--- a/services/ui/ws/window_tree.cc
+++ b/services/ui/ws/window_tree.cc
@@ -307,6 +307,9 @@ bool WindowTree::AddTransientWindow(const ClientWindowId& window_id,
bool WindowTree::DeleteWindow(const ClientWindowId& window_id) {
ServerWindow* window = GetWindowByClientId(window_id);
+ DVLOG(3) << "removing window from parent client=" << id_
+ << " client window_id= " << window_id.id << " global window_id="
+ << (window ? WindowIdToTransportId(window->id()) : 0);
if (!window)
return false;
@@ -1195,8 +1198,16 @@ void WindowTree::AddWindow(uint32_t change_id, Id parent_id, Id child_id) {
void WindowTree::RemoveWindowFromParent(uint32_t change_id, Id window_id) {
bool success = false;
ServerWindow* window = GetWindowByClientId(ClientWindowId(window_id));
- if (window && window->parent() &&
- access_policy_->CanRemoveWindowFromParent(window)) {
+ DVLOG(3) << "removing window from parent client=" << id_
+ << " client window_id= " << window_id << " global window_id="
+ << (window ? WindowIdToTransportId(window->id()) : 0);
+ if (!window) {
+ DVLOG(1) << "remove failing, invalid window id=" << change_id;
+ } else if (!window->parent()) {
+ DVLOG(1) << "remove failing, no parent id=" << change_id;
+ } else if (access_policy_->CanRemoveWindowFromParent(window)) {
+ DVLOG(1) << "remove failing, access policy disallowed id=" << change_id;
+ } else {
success = true;
Operation op(this, window_server_,
OperationType::REMOVE_WINDOW_FROM_PARENT);

Powered by Google App Engine
This is Rietveld 408576698