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

Unified Diff: components/exo/wayland/server.cc

Issue 2778733004: Add WindowPinType property on arua::Window (Closed)
Patch Set: Address review issues Created 3 years, 8 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 side-by-side diff with in-line comments
Download patch
« components/exo/shell_surface.h ('K') | « components/exo/shell_surface_unittest.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/exo/wayland/server.cc
diff --git a/components/exo/wayland/server.cc b/components/exo/wayland/server.cc
index af6732c0b8b2f3e164ff5715c203495d02cf0d17..c62e8143c744d0abdd7806bab8cb6cf104be2fb3 100644
--- a/components/exo/wayland/server.cc
+++ b/components/exo/wayland/server.cc
@@ -1951,11 +1951,15 @@ void remote_surface_unfullscreen(wl_client* client, wl_resource* resource) {
void remote_surface_pin(wl_client* client,
wl_resource* resource,
int32_t trusted) {
- GetUserDataAs<ShellSurface>(resource)->SetPinned(true, trusted);
+ ash::mojom::WindowPinType type =
reveman 2017/04/04 18:26:16 optional nit: no need for this temporary variable
Peng 2017/04/04 19:43:49 Done.
+ trusted ? ash::mojom::WindowPinType::TRUSTED_PINNED
+ : ash::mojom::WindowPinType::PINNED;
+ GetUserDataAs<ShellSurface>(resource)->SetPinned(type);
}
void remote_surface_unpin(wl_client* client, wl_resource* resource) {
- GetUserDataAs<ShellSurface>(resource)->SetPinned(false, /* trusted */ false);
+ GetUserDataAs<ShellSurface>(resource)->SetPinned(
+ ash::mojom::WindowPinType::NONE);
}
void remote_surface_set_system_modal(wl_client* client, wl_resource* resource) {
« components/exo/shell_surface.h ('K') | « components/exo/shell_surface_unittest.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698