Refactoring of SessionService to get componentized.

chrome/browser/extensions/api/sessions/sessions_api.cc

Index: chrome/browser/extensions/api/sessions/sessions_api.cc
diff --git a/chrome/browser/extensions/api/sessions/sessions_api.cc b/chrome/browser/extensions/api/sessions/sessions_api.cc
index f8dc1ff8eced06a71be2d9f7e2855eabf901bab8..d72c6b9fbe69c6bbdcdc5c3a32d04049264084db 100644
--- a/chrome/browser/extensions/api/sessions/sessions_api.cc
+++ b/chrome/browser/extensions/api/sessions/sessions_api.cc
@@ -285,10 +285,10 @@ scoped_ptr<windows::Window> SessionsGetDevicesFunction::CreateWindowModel(
 
   windows::Window::Type type = windows::Window::TYPE_NONE;
   switch (window.type) {
-    case Browser::TYPE_TABBED:
+    case SessionWindow::TYPE_TABBED:  // TODO(skuhne) Check Browser::TYPE_TABBED

[Mr4D (OOO till 08-26), 2014/10/23 20:09:10]

I did go over all uses of WindowSession::type (which was properly handled as
BrowserType) and all commands which were storing WindowType.

Everything looks now in order and we should use for commands as well as for the
SessionWindow only the WindowType - thus decoupling from Browser. Remaining is
the Sync data which should also get changed towards WindowType.

       type = windows::Window::TYPE_NORMAL;
       break;
-    case Browser::TYPE_POPUP:
+    case SessionWindow::TYPE_POPUP:  // TODO(skuhne): Check Browser::TYPE_POPUP:
       type = windows::Window::TYPE_POPUP;
       break;
   }