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

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

Issue 2349003002: Makes WindowTree::DeleteWindow() unembed for roots (Closed)
Patch Set: grammar Created 4 years, 3 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
Index: services/ui/ws/window_tree.h
diff --git a/services/ui/ws/window_tree.h b/services/ui/ws/window_tree.h
index cc54eb85536a13dbb9cf40ae9f6345b4161413d5..2a59479ba98d19e6bb548eab2812d3b74b2c578b 100644
--- a/services/ui/ws/window_tree.h
+++ b/services/ui/ws/window_tree.h
@@ -226,7 +226,7 @@ class WindowTree : public mojom::WindowTree,
const ServerWindow* relative_window,
mojom::OrderDirection direction,
bool originated_change);
- void ProcessWindowDeleted(const ServerWindow* window, bool originated_change);
+ void ProcessWindowDeleted(ServerWindow* window, bool originated_change);
void ProcessWillChangeWindowVisibility(const ServerWindow* window,
bool originated_change);
void ProcessWindowOpacityChanged(const ServerWindow* window,
@@ -276,6 +276,9 @@ class WindowTree : public mojom::WindowTree,
// Another client is being embedded in the window.
EMBED,
+
+ // The embedded client explicitly asked to be unembedded.
+ UNEMBED,
};
bool ShouldRouteToWindowManager(const ServerWindow* window) const;
@@ -314,7 +317,7 @@ class WindowTree : public mojom::WindowTree,
// Removes a root from set of roots of this tree. This does not remove
// the window from the window tree, only from the set of roots.
- void RemoveRoot(const ServerWindow* window, RemoveRootReason reason);
+ void RemoveRoot(ServerWindow* window, RemoveRootReason reason);
// Converts Window(s) to WindowData(s) for transport. This assumes all the
// windows are valid for the client. The parent of windows the client is not

Powered by Google App Engine
This is Rietveld 408576698