Add support for X11 workspaces

chrome/browser/ui/browser_window_state.cc

Index: chrome/browser/ui/browser_window_state.cc
diff --git a/chrome/browser/ui/browser_window_state.cc b/chrome/browser/ui/browser_window_state.cc
index e40ef26542fcec503dc6bce047ffee699f1ab764..126a3d97e893a769006777e86582d432c272eb24 100644
--- a/chrome/browser/ui/browser_window_state.cc
+++ b/chrome/browser/ui/browser_window_state.cc
@@ -136,6 +136,13 @@ void SaveWindowPlacement(const Browser* browser,
     session_service->SetWindowBounds(browser->session_id(), bounds, show_state);
 }
 
+void SaveWindowWorkspace(const Browser* browser, const std::string& workspace) {
+  SessionService* session_service =
+      SessionServiceFactory::GetForProfileIfExisting(browser->profile());
+  if (session_service)
+    session_service->SetWindowWorkspace(browser->session_id(), workspace);
+}
+
 void GetSavedWindowBoundsAndShowState(const Browser* browser,
                                       gfx::Rect* bounds,
                                       ui::WindowShowState* show_state) {
@@ -168,4 +175,12 @@ void GetSavedWindowBoundsAndShowState(const Browser* browser,
   }
 }
 
+void GetSavedWindowWorkspace(const Browser* browser, std::string* workspace) {

[sky, 2016/05/03 17:40:52]

Why do we need this function? Can't callers call browser->initial_workspace()
directly?

[Tom (Use chromium acct), 2016/05/03 19:38:39]

I completed the TODO in the comment here.  Should we keep this function now that
it actually does something useful? (It still only gets called in one place)

[sky, 2016/05/03 20:11:59]

I don't see the point in this function, so I say nuke it.

[Tom (Use chromium acct), 2016/05/03 20:27:58]

Done.

+  DCHECK(browser);
+  DCHECK(workspace);
+  *workspace = browser->initial_workspace();
+
+  // TODO(thomasanderson): add command line option for initial workspace
+}
+
 }  // namespace chrome