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

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

Issue 2535753009: Couple of changes for client area (Closed)
Patch Set: nuke todo 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
« no previous file with comments | « services/ui/ws/operation.h ('k') | ui/views/mus/desktop_window_tree_host_mus.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: services/ui/ws/window_tree.cc
diff --git a/services/ui/ws/window_tree.cc b/services/ui/ws/window_tree.cc
index b75af49bd5a53f81d357dec13b367f902cab3220..830e8babe17179d65f8cb57c76384e4a888e43f9 100644
--- a/services/ui/ws/window_tree.cc
+++ b/services/ui/ws/window_tree.cc
@@ -1462,9 +1462,21 @@ void WindowTree::SetClientArea(Id transport_window_id,
transport_additional_client_areas) {
ServerWindow* window =
GetWindowByClientId(ClientWindowId(transport_window_id));
- if (!window || !access_policy_->CanSetClientArea(window))
+ DVLOG(3) << "SetClientArea client window_id=" << transport_window_id
+ << " global window_id="
+ << (window ? WindowIdToTransportId(window->id()) : 0)
+ << " insets=" << insets.top() << " " << insets.left() << " "
+ << insets.bottom() << " " << insets.right();
+ if (!window) {
+ DVLOG(1) << "SetClientArea failed, no window";
return;
+ }
+ if (!access_policy_->CanSetClientArea(window)) {
+ DVLOG(1) << "SetClientArea failed, access denied";
+ return;
+ }
+ Operation op(this, window_server_, OperationType::SET_CLIENT_AREA);
window->SetClientArea(insets, transport_additional_client_areas.value_or(
std::vector<gfx::Rect>()));
}
« no previous file with comments | « services/ui/ws/operation.h ('k') | ui/views/mus/desktop_window_tree_host_mus.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698