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

Unified Diff: components/mus/public/cpp/window_manager_delegate.h

Issue 1567623003: Adds remainder of support for multiple roots to WS (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fix comment Created 4 years, 11 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: components/mus/public/cpp/window_manager_delegate.h
diff --git a/components/mus/public/cpp/window_manager_delegate.h b/components/mus/public/cpp/window_manager_delegate.h
index 7751623fb8e0c0cda4c5f03be5179f14c71401c9..d27729e7e4898ff703dacfc683c08dcce002fb1f 100644
--- a/components/mus/public/cpp/window_manager_delegate.h
+++ b/components/mus/public/cpp/window_manager_delegate.h
@@ -7,6 +7,7 @@
#include <stdint.h>
+#include <map>
#include <vector>
#include "base/memory/scoped_ptr.h"
@@ -38,6 +39,15 @@ class WindowManagerDelegate {
const std::string& name,
scoped_ptr<std::vector<uint8_t>>* new_data) = 0;
+ // A client has requested a new top level window. The delegate should create
+ // and parent the window appropriately and return it. |properties| is the
+ // supplied properties from the client requesting the new window. The
+ // delegate may modify |properties| before calling NewWindow(), but the
+ // delegate does *not* own |properties|, they are valid only for the life
+ // of OnWmCreateTopLevelWindow().
+ virtual Window* OnWmCreateTopLevelWindow(
+ std::map<std::string, std::vector<uint8_t>>* properties) = 0;
+
protected:
virtual ~WindowManagerDelegate() {}
};

Powered by Google App Engine
This is Rietveld 408576698