11 files changed, 241 insertions, 181 deletions
diff --git a/Swift/Controllers/MainController.cpp b/Swift/Controllers/MainController.cpp
index f678c0d..91140c2 100644
--- a/Swift/Controllers/MainController.cpp
+++ b/Swift/Controllers/MainController.cpp
@@ -175,32 +175,2 @@ MainController::MainController(
 
-    std::string selectedLoginJID = settings_->getSetting(SettingConstants::LAST_LOGIN_JID);
-    bool loginAutomatically = settings_->getSetting(SettingConstants::LOGIN_AUTOMATICALLY);
-    std::string cachedPassword;
-    std::string cachedCertificate;
-    ClientOptions cachedOptions;
-    bool eagle = settings_->getSetting(SettingConstants::FORGET_PASSWORDS);
-    if (!eagle) {
-        for (auto&& profile : settings->getAvailableProfiles()) {
-            ProfileSettingsProvider profileSettings(profile, settings);
-            std::string password = profileSettings.getStringSetting("pass");
-            std::string certificate = profileSettings.getStringSetting("certificate");
-            std::string jid = profileSettings.getStringSetting("jid");
-            ClientOptions clientOptions = parseClientOptions(profileSettings.getStringSetting("options"));
-
-#ifdef SWIFTEN_PLATFORM_WIN32
-            clientOptions.singleSignOn = settings_->getSetting(SettingConstants::SINGLE_SIGN_ON);
-#endif
-
-            loginWindow_->addAvailableAccount(jid, password, certificate, clientOptions);
-            if (jid == selectedLoginJID) {
-                cachedPassword = password;
-                cachedCertificate = certificate;
-                cachedOptions = clientOptions;
-            }
-        }
-        loginWindow_->selectUser(selectedLoginJID);
-        loginWindow_->setLoginAutomatically(loginAutomatically);
-    }
-
-
     loginWindow_->onLoginRequest.connect(boost::bind(&MainController::handleLoginRequest, this, _1, _2, _3, _4, _5, _6, _7));
@@ -219,9 +189,2 @@ MainController::MainController(
 
-    if (loginAutomatically) {
-        profileSettings_ = new ProfileSettingsProvider(selectedLoginJID, settings_);
-        /* FIXME: deal with autologin with a cert*/
-        handleLoginRequest(selectedLoginJID, cachedPassword, cachedCertificate, CertificateWithKey::ref(), cachedOptions, true, true);
-    } else {
-        profileSettings_ = nullptr;
-    }
 }
@@ -561,4 +524,3 @@ void MainController::handleLoginRequest(const std::string &username, const std::
             profileSettings_->storeString("options", optionString);
-            settings_->storeSetting(SettingConstants::LAST_LOGIN_JID, username);
-            settings_->storeSetting(SettingConstants::LOGIN_AUTOMATICALLY, loginAutomatically);
+            profileSettings_->storeInt("enabled", 1);
             loginWindow_->addAvailableAccount(profileSettings_->getStringSetting("jid"), profileSettings_->getStringSetting("pass"), profileSettings_->getStringSetting("certificate"), options);
@@ -762,2 +724,3 @@ void MainController::signOut() {
     }
+    profileSettings_->storeInt("enabled", 0);
     eventController_->clear();
@@ -866,2 +829,4 @@ void MainController::handleQuitRequest() {
 
+//FIXME: Switch all this to boost::serialise
+
 #define SERIALIZE_BOOL(option) result += options.option ? "1" : "0"; result += ",";
@@ -900,51 +865,3 @@ std::string MainController::serializeClientOptions(const ClientOptions& options)
     SERIALIZE_BOOL(tlsOptions.schannelTLS1_0Workaround);
-    return result;
-}
-
-#define CHECK_PARSE_LENGTH if (i >= segments.size()) {return result;}
-#define PARSE_INT_RAW(defaultValue) CHECK_PARSE_LENGTH intVal = defaultValue; try {intVal = boost::lexical_cast<int>(segments[i]);} catch(const boost::bad_lexical_cast&) {};i++;
-#define PARSE_STRING_RAW CHECK_PARSE_LENGTH stringVal = byteArrayToString(Base64::decode(segments[i]));i++;
-
-#define PARSE_BOOL(option, defaultValue) PARSE_INT_RAW(defaultValue); result.option = (intVal == 1);
-#define PARSE_INT(option, defaultValue) PARSE_INT_RAW(defaultValue); result.option = intVal;
-#define PARSE_STRING(option) PARSE_STRING_RAW; result.option = stringVal;
-#define PARSE_SAFE_STRING(option) PARSE_STRING_RAW; result.option = SafeString(createSafeByteArray(stringVal));
-#define PARSE_URL(option) {PARSE_STRING_RAW; result.option = URL::fromString(stringVal);}
-
-
-ClientOptions MainController::parseClientOptions(const std::string& optionString) {
-    ClientOptions result;
-    size_t i = 0;
-    int intVal = 0;
-    std::string stringVal;
-    std::vector<std::string> segments = String::split(optionString, ',');
-
-    PARSE_BOOL(useStreamCompression, 1);
-    PARSE_INT_RAW(-1);
-    switch (intVal) {
-        case 1: result.useTLS = ClientOptions::NeverUseTLS;break;
-        case 2: result.useTLS = ClientOptions::UseTLSWhenAvailable;break;
-        case 3: result.useTLS = ClientOptions::RequireTLS;break;
-        default:;
-    }
-    PARSE_BOOL(allowPLAINWithoutTLS, 0);
-    PARSE_BOOL(useStreamResumption, 0);
-    PARSE_BOOL(useAcks, 1);
-    PARSE_STRING(manualHostname);
-    PARSE_INT(manualPort, -1);
-    PARSE_INT_RAW(-1);
-    switch (intVal) {
-        case 1: result.proxyType = ClientOptions::NoProxy;break;
-        case 2: result.proxyType = ClientOptions::SystemConfiguredProxy;break;
-        case 3: result.proxyType = ClientOptions::SOCKS5Proxy;break;
-        case 4: result.proxyType = ClientOptions::HTTPConnectProxy;break;
-    }
-    PARSE_STRING(manualProxyHostname);
-    PARSE_INT(manualProxyPort, -1);
-    PARSE_URL(boshURL);
-    PARSE_URL(boshHTTPConnectProxyURL);
-    PARSE_SAFE_STRING(boshHTTPConnectProxyAuthID);
-    PARSE_SAFE_STRING(boshHTTPConnectProxyAuthPassword);
-    PARSE_BOOL(tlsOptions.schannelTLS1_0Workaround, false);
-
+    SERIALIZE_BOOL(singleSignOn);
     return result;
diff --git a/Swift/Controllers/MainController.h b/Swift/Controllers/MainController.h
index 8b62415..7a06a0b 100644
--- a/Swift/Controllers/MainController.h
+++ b/Swift/Controllers/MainController.h
@@ -134,3 +134,2 @@ namespace Swift {
             std::string serializeClientOptions(const ClientOptions& options);
-            ClientOptions parseClientOptions(const std::string& optionString);
 
@@ -148,3 +147,3 @@ namespace Swift {
             SettingsProvider *settings_;
-            ProfileSettingsProvider* profileSettings_;
+            ProfileSettingsProvider* profileSettings_ = nullptr;
             Dock* dock_;
diff --git a/Swift/Controllers/SettingConstants.cpp b/Swift/Controllers/SettingConstants.cpp
index 1191c28..8336200 100644
--- a/Swift/Controllers/SettingConstants.cpp
+++ b/Swift/Controllers/SettingConstants.cpp
@@ -16,4 +16,2 @@ const SettingsProvider::Setting<bool> SettingConstants::FORGET_PASSWORDS = Setti
 const SettingsProvider::Setting<bool> SettingConstants::REMEMBER_RECENT_CHATS = SettingsProvider::Setting<bool>("rememberRecentChats", true);
-const SettingsProvider::Setting<std::string> SettingConstants::LAST_LOGIN_JID = SettingsProvider::Setting<std::string>("lastLoginJID", "");
-const SettingsProvider::Setting<bool> SettingConstants::LOGIN_AUTOMATICALLY = SettingsProvider::Setting<bool>("loginAutomatically", false);
 const SettingsProvider::Setting<bool> SettingConstants::SHOW_OFFLINE("showOffline", false);
@@ -25,3 +23,2 @@ const SettingsProvider::Setting<std::string> SettingConstants::INVITE_AUTO_ACCEP
 const SettingsProvider::Setting<bool> SettingConstants::DISCONNECT_ON_CARD_REMOVAL("disconnectOnCardRemoval", true);
-const SettingsProvider::Setting<bool> SettingConstants::SINGLE_SIGN_ON("singleSignOn", false);
 const SettingsProvider::Setting<bool> SettingConstants::MUC_MARKING_ELISION("mucMarkingElision", true);
diff --git a/Swift/Controllers/SettingConstants.h b/Swift/Controllers/SettingConstants.h
index f82dfb5..68c22b7 100644
--- a/Swift/Controllers/SettingConstants.h
+++ b/Swift/Controllers/SettingConstants.h
@@ -36,4 +36,2 @@ namespace Swift {
             static const SettingsProvider::Setting<bool> REMEMBER_RECENT_CHATS;
-            static const SettingsProvider::Setting<std::string> LAST_LOGIN_JID;
-            static const SettingsProvider::Setting<bool> LOGIN_AUTOMATICALLY;
             /**
@@ -87,11 +85,2 @@ namespace Swift {
             /**
-             * The #SINGLE_SIGN_ON setting
-             * specifies whether to log in using Single Sign On.
-             * This is currently supported on Windows.
-             *
-             * If set true Swift will use GSSAPI authentication to
-             * log in the user; else not.
-             */
-            static const SettingsProvider::Setting<bool> SINGLE_SIGN_ON;
-            /**
              * The #MUC_MARKING_ELISION setting
diff --git a/Swift/QtUI/QtLoginWindow.h b/Swift/QtUI/QtLoginWindow.h
index 6920903..d3c2601 100644
--- a/Swift/QtUI/QtLoginWindow.h
+++ b/Swift/QtUI/QtLoginWindow.h
@@ -61,4 +61,6 @@ namespace Swift {
 
-        private slots:
+        public slots:
             void loginClicked();
+
+        private slots:
             void handleCertficateChecked(bool);
diff --git a/Swift/QtUI/QtSingleWindow.cpp b/Swift/QtUI/QtSingleWindow.cpp
index d53f247..6881c4f 100644
--- a/Swift/QtUI/QtSingleWindow.cpp
+++ b/Swift/QtUI/QtSingleWindow.cpp
@@ -8,2 +8,5 @@
 
+#include <QPushButton>
+#include <QVBoxLayout>
+
 #include <Swiften/Base/Platform.h>
@@ -11,2 +14,3 @@
 #include <Swift/QtUI/QtChatTabs.h>
+#include <Swift/QtUI/QtLoginWindow.h>
 #include <Swift/QtUI/QtSettingsProvider.h>
@@ -26,2 +30,21 @@ QtSingleWindow::QtSingleWindow(QtSettingsProvider* settings) : QSplitter() {
     setChildrenCollapsible(false);
+
+    auto left = new QWidget(this);
+    list_ = new QListWidget(left);
+    auto addButton = new QPushButton("+", left);
+    QVBoxLayout* leftLayout = new QVBoxLayout();
+    leftLayout->addWidget(list_);
+    leftLayout->addWidget(addButton);
+    left->setLayout(leftLayout);
+    QSplitter::addWidget(left);
+    loginWindows_ = new QStackedWidget(this);
+    QSplitter::addWidget(loginWindows_);
+    tabs_ = new QStackedWidget(this);
+    QSplitter::addWidget(tabs_);
+    restoreSplitters();
+    setStretchFactor(0, 0);
+    setStretchFactor(1, 0);
+    setStretchFactor(2, 1);
+    connect(list_, SIGNAL(itemClicked(QListWidgetItem*)), this, SLOT(handleListItemClicked(QListWidgetItem*)));
+    connect(addButton, SIGNAL(clicked()), this, SIGNAL(wantsToAddAccount()));
 #ifdef SWIFTEN_PLATFORM_MACOSX
@@ -35,10 +58,2 @@ QtSingleWindow::~QtSingleWindow() {
 
-void QtSingleWindow::addWidget(QWidget* widget) {
-    QtChatTabs* tabs = dynamic_cast<QtChatTabs*>(widget);
-    if (tabs) {
-        connect(tabs, SIGNAL(onTitleChanged(const QString&)), this, SLOT(handleTabsTitleChanged(const QString&)));
-    }
-    QSplitter::addWidget(widget);
-}
-
 void QtSingleWindow::handleTabsTitleChanged(const QString& title) {
@@ -57,15 +72,10 @@ void QtSingleWindow::handleSplitterMoved() {
 void QtSingleWindow::restoreSplitters() {
-    QList<QVariant> variantValues = settings_->getQSettings()->value(SINGLE_WINDOW_SPLITS).toList();
-    QList<int> intValues;
-    for (auto&& value : variantValues) {
-        intValues.append(value.toInt());
-    }
-    setSizes(intValues);
-}
-
-void QtSingleWindow::insertAtFront(QWidget* widget) {
-    insertWidget(0, widget);
     auto splitsVariant = settings_->getQSettings()->value(SINGLE_WINDOW_SPLITS);
     if (splitsVariant.isValid()) {
-        restoreSplitters();
+        auto variantValues = splitsVariant.toList();
+        QList<int> intValues;
+        for (auto&& value : variantValues) {
+            intValues.append(value.toInt());
+        }
+        setSizes(intValues);
     }
@@ -74,4 +84,2 @@ void QtSingleWindow::insertAtFront(QWidget* widget) {
     }
-    setStretchFactor(0, 0);
-    setStretchFactor(1, 1);
 }
@@ -91,2 +99,23 @@ void QtSingleWindow::moveEvent(QMoveEvent*) {
 
+void QtSingleWindow::addAccount(QtLoginWindow* loginWindow, QtChatTabs* tabs) {
+    if (!loginWindows_->count()) {
+        connect(tabs, SIGNAL(onTitleChanged(const QString&)), this, SLOT(handleTabsTitleChanged(const QString&)));
+    }
+    loginWindows_->addWidget(loginWindow);
+    tabs_->addWidget(tabs);
+    list_->addItem(QString("Account %1").arg(loginWindows_->count()));
+}
+
+void QtSingleWindow::handleListItemClicked(QListWidgetItem* /*item*/) {
+    //FIXME: Should use a full model/view and do this properly (and render pretty things ourselves too)
+    auto currentTabs = tabs_->widget(tabs_->currentIndex());
+    disconnect(currentTabs, SIGNAL(onTitleChanged(const QString&)), this, SLOT(handleTabsTitleChanged(const QString&)));
+    loginWindows_->setCurrentIndex(list_->currentRow());
+    tabs_->setCurrentIndex(list_->currentRow());
+    currentTabs = tabs_->widget(tabs_->currentIndex());
+    connect(currentTabs, SIGNAL(onTitleChanged(const QString&)), this, SLOT(handleTabsTitleChanged(const QString&)));
+    //TODO change the title of the window.
+    handleTabsTitleChanged(QString("Swift"));
+}
+
 }
diff --git a/Swift/QtUI/QtSingleWindow.h b/Swift/QtUI/QtSingleWindow.h
index c6f22cf..9a7e475 100644
--- a/Swift/QtUI/QtSingleWindow.h
+++ b/Swift/QtUI/QtSingleWindow.h
@@ -8,5 +8,10 @@
 
+#include <QListWidget>
 #include <QSplitter>
+#include <QStackedWidget>
+
 
 namespace Swift {
+    class QtChatTabs;
+    class QtLoginWindow;
     class QtSettingsProvider;
@@ -18,4 +23,7 @@ namespace Swift {
             virtual ~QtSingleWindow();
-            void insertAtFront(QWidget* widget);
-            void addWidget(QWidget* widget);
+            void addAccount(QtLoginWindow* widget, QtChatTabs* tabs);
+
+        signals:
+            void wantsToAddAccount();
+
         protected:
@@ -26,2 +34,3 @@ namespace Swift {
             void handleTabsTitleChanged(const QString& title);
+            void handleListItemClicked(QListWidgetItem*);
         private:
@@ -33,2 +42,5 @@ namespace Swift {
             QtSettingsProvider* settings_;
+            QListWidget* list_;
+            QStackedWidget* loginWindows_;
+            QStackedWidget* tabs_;
     };
diff --git a/Swift/QtUI/QtSwift.cpp b/Swift/QtUI/QtSwift.cpp
index 89dfa01..28c7044 100644
--- a/Swift/QtUI/QtSwift.cpp
+++ b/Swift/QtUI/QtSwift.cpp
@@ -24,4 +24,6 @@
 #include <Swiften/Base/Platform.h>
+#include <Swiften/Base/String.h>
 #include <Swiften/Client/Client.h>
 #include <Swiften/Elements/Presence.h>
+#include <Swiften/StringCodecs/Base64.h>
 #include <Swiften/TLS/TLSContextFactory.h>
@@ -94,4 +96,2 @@ po::options_description QtSwift::getOptionsDescription() {
         ("latency-debug", "Use latency debugging (unsupported)")
-        ("multi-account", po::value<int>()->default_value(1), "Number of accounts to open windows for (unsupported)")
-        ("start-minimized", "Don't show the login/roster window at startup")
         ("enable-jid-adhocs", "Enable AdHoc commands to custom JIDs.")
@@ -169,15 +169,7 @@ QtSwift::QtSwift(const po::variables_map& options) : networkFactories_(&clientMa
 
-    std::map<std::string, std::string> emoticons;
-    loadEmoticonsFile(":/emoticons/emoticons.txt", emoticons);
-    loadEmoticonsFile(P2QSTRING(pathToString(Paths::getExecutablePath() / "emoticons.txt")), emoticons);
+    loadEmoticonsFile(":/emoticons/emoticons.txt", emoticons_);
+    loadEmoticonsFile(P2QSTRING(pathToString(Paths::getExecutablePath() / "emoticons.txt")), emoticons_);
 
     splitter_ = new QtSingleWindow(qtSettings_);
-
-    int numberOfAccounts = 1;
-    try {
-        numberOfAccounts = options["multi-account"].as<int>();
-    } catch (...) {
-        /* This seems to fail on a Mac when the .app is launched directly (the usual path).*/
-        numberOfAccounts = 1;
-    }
+    connect(splitter_, SIGNAL(wantsToAddAccount()), this, SLOT(handleWantsToAddAccount()));
 
@@ -204,2 +196,4 @@ QtSwift::QtSwift(const po::variables_map& options) : networkFactories_(&clientMa
     }
+    //TODO this old option can be purged
+    useDelayForLatency_ = options.count("latency-debug") > 0;
 
@@ -242,5 +236,3 @@ QtSwift::QtSwift(const po::variables_map& options) : networkFactories_(&clientMa
 #endif
-    bool enableAdHocCommandOnJID = options.count("enable-jid-adhocs") > 0;
-    tabs_ = new QtChatTabs(settingsHierarchy_, true);
-    bool startMinimized = options.count("start-minimized") > 0;
+    enableAdHocCommandOnJID_ = options.count("enable-jid-adhocs") > 0;
     applicationPathProvider_ = new PlatformApplicationPathProvider(SWIFT_APPLICATION_NAME);
@@ -248,3 +240,2 @@ QtSwift::QtSwift(const po::variables_map& options) : networkFactories_(&clientMa
     certificateStorageFactory_ = new CertificateFileStorageFactory(applicationPathProvider_->getDataDir(), tlsFactories_.getCertificateFactory(), networkFactories_.getCryptoProvider());
-    chatWindowFactory_ = new QtChatWindowFactory(splitter_, settingsHierarchy_, qtSettings_, tabs_, ":/themes/Default/", emoticons);
     soundPlayer_ = new QtSoundPlayer(applicationPathProvider_);
@@ -300,28 +291,4 @@ QtSwift::QtSwift(const po::variables_map& options) : networkFactories_(&clientMa
     }
-
-    for (int i = 0; i < numberOfAccounts; i++) {
-        if (i > 0) {
-            // Don't add the first tray (see note above)
-            systemTrays_.push_back(new QtSystemTray());
-        }
-        QtUIFactory* uiFactory = new QtUIFactory(settingsHierarchy_, qtSettings_, tabs_, splitter_, systemTrays_[i], chatWindowFactory_, networkFactories_.getTimerFactory(), statusCache_, autoUpdater_, startMinimized, !emoticons.empty(), enableAdHocCommandOnJID);
-        uiFactories_.push_back(uiFactory);
-        MainController* mainController = new MainController(
-                &clientMainThreadCaller_,
-                &networkFactories_,
-                uiFactory,
-                settingsHierarchy_,
-                systemTrays_[i],
-                soundPlayer_,
-                storagesFactory_,
-                certificateStorageFactory_,
-                dock_,
-                notifier_,
-                uriHandler_,
-                &idleDetector_,
-                emoticons,
-                options.count("latency-debug") > 0);
-        mainControllers_.push_back(mainController);
-    }
-
+    migrateLastLoginAccount();
+    restoreAccounts();
     connect(qApp, SIGNAL(aboutToQuit()), this, SLOT(handleAboutToQuit()));
@@ -341,4 +308,2 @@ QtSwift::~QtSwift() {
     }
-    delete tabs_;
-    delete chatWindowFactory_;
     delete splitter_;
@@ -381,2 +346,137 @@ void QtSwift::handleAutoUpdaterStateChanged(AutoUpdater::State updatedState) {
 
+void QtSwift::handleWantsToAddAccount() {
+    auto loginWindow = addAccount();
+    if (!settingsHierarchy_->getSetting(SettingConstants::FORGET_PASSWORDS)) {
+        for (const auto& profile : settingsHierarchy_->getAvailableProfiles()) {
+            ProfileSettingsProvider profileSettings(profile, settingsHierarchy_);
+            if (profileSettings.getIntSetting("enabled", 0)) {
+                // No point showing accounts that're already logged in
+                continue;
+            }
+            const auto& password = profileSettings.getStringSetting("pass");
+            const auto& certificate = profileSettings.getStringSetting("certificate");
+            const auto& jid = profileSettings.getStringSetting("jid");
+            const auto& clientOptions = parseClientOptions(profileSettings.getStringSetting("options"));
+            loginWindow->addAvailableAccount(jid, password, certificate, clientOptions);
+        }
+    }
+}
+
+void QtSwift::restoreAccounts() {
+    if (!settingsHierarchy_->getSetting(SettingConstants::FORGET_PASSWORDS)) {
+        for (const auto& profile : settingsHierarchy_->getAvailableProfiles()) {
+            ProfileSettingsProvider profileSettings(profile, settingsHierarchy_);
+            if (!profileSettings.getIntSetting("enabled", 0)) {
+                continue;
+            }
+            const auto& jid = profileSettings.getStringSetting("jid");
+            const auto& password = profileSettings.getStringSetting("pass");
+            const auto& certificate = profileSettings.getStringSetting("certificate");
+            const auto& clientOptions = parseClientOptions(profileSettings.getStringSetting("options"));
+            auto loginWindow = addAccount();
+            loginWindow->addAvailableAccount(jid, password, certificate, clientOptions);
+            loginWindow->loginClicked();
+        }
+    }
+}
+
+void QtSwift::migrateLastLoginAccount() {
+    const SettingsProvider::Setting<bool> loginAutomatically = SettingsProvider::Setting<bool>("loginAutomatically", false);
+    if (settingsHierarchy_->getSetting(loginAutomatically)) {
+        auto selectedLoginJID = settingsHierarchy_->getSetting(SettingsProvider::Setting<std::string>("lastLoginJID", ""));
+        for (const auto& profile : settingsHierarchy_->getAvailableProfiles()) {
+            ProfileSettingsProvider profileSettings(profile, settingsHierarchy_);
+            if (profileSettings.getStringSetting("jid") == selectedLoginJID) {
+                profileSettings.storeInt("enabled", 1);
+                break;
+            }
+        }
+        settingsHierarchy_->storeSetting(loginAutomatically, false);
+    }
+}
+
+QtLoginWindow* QtSwift::addAccount() {
+    if (uiFactories_.size() > 0) {
+        // Don't add the first tray (see note above)
+        systemTrays_.push_back(new QtSystemTray());
+    }
+    auto tabs = new QtChatTabs(settingsHierarchy_, true);
+    QtUIFactory* uiFactory = new QtUIFactory(settingsHierarchy_, qtSettings_, tabs, splitter_, systemTrays_[systemTrays_.size() - 1], networkFactories_.getTimerFactory(), statusCache_, autoUpdater_, emoticons_, enableAdHocCommandOnJID_);
+    uiFactories_.push_back(uiFactory);
+    MainController* mainController = new MainController(
+        &clientMainThreadCaller_,
+        &networkFactories_,
+        uiFactory,
+        settingsHierarchy_,
+        systemTrays_[systemTrays_.size() - 1],
+        soundPlayer_,
+        storagesFactory_,
+        certificateStorageFactory_,
+        dock_,
+        notifier_,
+        uriHandler_,
+        &idleDetector_,
+        emoticons_,
+        useDelayForLatency_);
+    mainControllers_.push_back(mainController);
+
+    //FIXME - mainController has already created the window, so we can pass null here and get the old one
+    auto loginWindow = uiFactory->createLoginWindow(nullptr);
+
+    return dynamic_cast<QtLoginWindow*>(loginWindow);
+}
+
+//FIXME: Switch all this to boost::serialise
+
+#define CHECK_PARSE_LENGTH if (i >= segments.size()) {return result;}
+#define PARSE_INT_RAW(defaultValue) CHECK_PARSE_LENGTH intVal = defaultValue; try {intVal = boost::lexical_cast<int>(segments[i]);} catch(const boost::bad_lexical_cast&) {};i++;
+#define PARSE_STRING_RAW CHECK_PARSE_LENGTH stringVal = byteArrayToString(Base64::decode(segments[i]));i++;
+
+#define PARSE_BOOL(option, defaultValue) PARSE_INT_RAW(defaultValue); result.option = (intVal == 1);
+#define PARSE_INT(option, defaultValue) PARSE_INT_RAW(defaultValue); result.option = intVal;
+#define PARSE_STRING(option) PARSE_STRING_RAW; result.option = stringVal;
+#define PARSE_SAFE_STRING(option) PARSE_STRING_RAW; result.option = SafeString(createSafeByteArray(stringVal));
+#define PARSE_URL(option) {PARSE_STRING_RAW; result.option = URL::fromString(stringVal);}
+
+
+ClientOptions QtSwift::parseClientOptions(const std::string& optionString) {
+    ClientOptions result;
+    size_t i = 0;
+    int intVal = 0;
+    std::string stringVal;
+    std::vector<std::string> segments = String::split(optionString, ',');
+
+    PARSE_BOOL(useStreamCompression, 1);
+    PARSE_INT_RAW(-1);
+    switch (intVal) {
+    case 1: result.useTLS = ClientOptions::NeverUseTLS; break;
+    case 2: result.useTLS = ClientOptions::UseTLSWhenAvailable; break;
+    case 3: result.useTLS = ClientOptions::RequireTLS; break;
+    default:;
+    }
+    PARSE_BOOL(allowPLAINWithoutTLS, 0);
+    PARSE_BOOL(useStreamResumption, 0);
+    PARSE_BOOL(useAcks, 1);
+    PARSE_STRING(manualHostname);
+    PARSE_INT(manualPort, -1);
+    PARSE_INT_RAW(-1);
+    switch (intVal) {
+    case 1: result.proxyType = ClientOptions::NoProxy; break;
+    case 2: result.proxyType = ClientOptions::SystemConfiguredProxy; break;
+    case 3: result.proxyType = ClientOptions::SOCKS5Proxy; break;
+    case 4: result.proxyType = ClientOptions::HTTPConnectProxy; break;
+    }
+    PARSE_STRING(manualProxyHostname);
+    PARSE_INT(manualProxyPort, -1);
+    PARSE_URL(boshURL);
+    PARSE_URL(boshHTTPConnectProxyURL);
+    PARSE_SAFE_STRING(boshHTTPConnectProxyAuthID);
+    PARSE_SAFE_STRING(boshHTTPConnectProxyAuthPassword);
+    PARSE_BOOL(tlsOptions.schannelTLS1_0Workaround, false);
+    PARSE_BOOL(singleSignOn, false);
+
+    return result;
+}
+
+
 }
diff --git a/Swift/QtUI/QtSwift.h b/Swift/QtUI/QtSwift.h
index f35d2cc..d876cd8 100644
--- a/Swift/QtUI/QtSwift.h
+++ b/Swift/QtUI/QtSwift.h
@@ -8,2 +8,3 @@
 
+#include <map>
 #include <string>
@@ -14,2 +15,3 @@
 #include <Swiften/Base/Platform.h>
+#include <Swiften/Client/ClientOptions.h>
 #include <Swiften/EventLoop/Qt/QtEventLoop.h>
@@ -70,2 +72,3 @@ namespace Swift {
             void handleAutoUpdaterStateChanged(AutoUpdater::State updatedState);
+            void handleWantsToAddAccount();
 
@@ -75,2 +78,9 @@ namespace Swift {
             static const std::string& updateChannelToFeed(const std::string& channel);
+            QtLoginWindow* addAccount();
+            ClientOptions parseClientOptions(const std::string& optionString);
+            void restoreAccounts();
+            /**
+            * Upgrades the config from pre-multi-account to post-multi-account format (added in 5.0).
+            */
+            void migrateLastLoginAccount();
 
@@ -91,3 +101,2 @@ namespace Swift {
             URIHandler* uriHandler_;
-            QtChatTabs* tabs_;
             ApplicationPathProvider* applicationPathProvider_;
@@ -100,2 +109,5 @@ namespace Swift {
             ActualIdleDetector idleDetector_;
+            std::map<std::string, std::string> emoticons_;
+            bool enableAdHocCommandOnJID_ = false;
+            bool useDelayForLatency_;
 #if defined(SWIFTEN_PLATFORM_MACOSX)
diff --git a/Swift/QtUI/QtUIFactory.cpp b/Swift/QtUI/QtUIFactory.cpp
index 0b19f63..2762d68 100644
--- a/Swift/QtUI/QtUIFactory.cpp
+++ b/Swift/QtUI/QtUIFactory.cpp
@@ -42,3 +42,5 @@ namespace Swift {
 
-QtUIFactory::QtUIFactory(SettingsProviderHierachy* settings, QtSettingsProvider* qtOnlySettings, QtChatTabs* tabs, QtSingleWindow* netbookSplitter, QtSystemTray* systemTray, QtChatWindowFactory* chatWindowFactory, TimerFactory* timerFactory, StatusCache* statusCache, AutoUpdater* autoUpdater, bool startMinimized, bool emoticonsExist, bool enableAdHocCommandOnJID) : settings_(settings), qtOnlySettings_(qtOnlySettings), tabs_(tabs), netbookSplitter_(netbookSplitter), systemTray_(systemTray), chatWindowFactory_(chatWindowFactory), timerFactory_(timerFactory), lastMainWindow_(nullptr), loginWindow_(nullptr), statusCache_(statusCache), autoUpdater_(autoUpdater), startMinimized_(startMinimized), emoticonsExist_(emoticonsExist), enableAdHocCommandOnJID_(enableAdHocCommandOnJID) {
+QtUIFactory::QtUIFactory(SettingsProviderHierachy* settings, QtSettingsProvider* qtOnlySettings, QtChatTabs* tabs, QtSingleWindow* netbookSplitter, QtSystemTray* systemTray, TimerFactory* timerFactory, StatusCache* statusCache, AutoUpdater* autoUpdater, std::map<std::string, std::string>& emoticons, bool enableAdHocCommandOnJID) : settings_(settings), qtOnlySettings_(qtOnlySettings), tabs_(tabs), netbookSplitter_(netbookSplitter), systemTray_(systemTray), timerFactory_(timerFactory), lastMainWindow_(nullptr), loginWindow_(nullptr), statusCache_(statusCache), autoUpdater_(autoUpdater), emoticons_(emoticons), enableAdHocCommandOnJID_(enableAdHocCommandOnJID) {
+    emoticonsExist_ = !emoticons_.empty();
+    chatWindowFactory_ = new QtChatWindowFactory(netbookSplitter_, settings, qtOnlySettings, tabs_, ":/themes/Default/", emoticons_);
     chatFontSize_ = settings_->getSetting(QtUISettingConstants::CHATWINDOW_FONT_SIZE);
@@ -52,2 +54,3 @@ QtUIFactory::~QtUIFactory() {
     }
+    delete chatWindowFactory_;
 }
@@ -93,8 +96,8 @@ MainWindow* QtUIFactory::createMainWindow(Chattables& chattables, UIEventStream*
 LoginWindow* QtUIFactory::createLoginWindow(UIEventStream* eventStream) {
+    if (loginWindow_) {
+        return loginWindow_;
+    }
     loginWindow_ = new QtLoginWindow(eventStream, settings_, timerFactory_, autoUpdater_);
-    netbookSplitter_->insertAtFront(loginWindow_);
+    netbookSplitter_->addAccount(loginWindow_, tabs_);
     connect(systemTray_, SIGNAL(clicked()), loginWindow_, SLOT(toggleBringToFront()));
-    if (startMinimized_) {
-        loginWindow_->hide();
-    }
     return loginWindow_;
diff --git a/Swift/QtUI/QtUIFactory.h b/Swift/QtUI/QtUIFactory.h
index 4013668..ad8dc1f 100644
--- a/Swift/QtUI/QtUIFactory.h
+++ b/Swift/QtUI/QtUIFactory.h
@@ -37,3 +37,3 @@ namespace Swift {
         public:
-            QtUIFactory(SettingsProviderHierachy* settings, QtSettingsProvider* qtOnlySettings, QtChatTabs* tabs, QtSingleWindow* netbookSplitter, QtSystemTray* systemTray, QtChatWindowFactory* chatWindowFactory, TimerFactory* timerFactory, StatusCache* statusCache, AutoUpdater* autoUpdater, bool startMinimized, bool emoticonsExist, bool enableAdHocCommandOnJID);
+            QtUIFactory(SettingsProviderHierachy* settings, QtSettingsProvider* qtOnlySettings, QtChatTabs* tabs, QtSingleWindow* netbookSplitter, QtSystemTray* systemTray, TimerFactory* timerFactory, StatusCache* statusCache, AutoUpdater* autoUpdater, std::map<std::string, std::string>& emoticons, bool enableAdHocCommandOnJID);
             ~QtUIFactory();
@@ -77,3 +77,2 @@ namespace Swift {
             std::vector<QPointer<QtChatWindow> > chatWindows_;
-            bool startMinimized_;
             int chatFontSize_;
@@ -81,2 +80,3 @@ namespace Swift {
             bool emoticonsExist_;
+            std::map<std::string, std::string>& emoticons_;
             bool enableAdHocCommandOnJID_;