Connection::assumeIdentity: Set UserId and DeviceId and completeSetup…

… before connecting to the server

This allows libQuotient-based clients to operate with the cached state (and in the future, room events)
when there is no network connection. It's losing the check whether userid and deviceid correspond to the given token,
but I don't see a strong need for that.

Quotient/accountregistry.cpp

@@ -144,6 +144,7 @@ void AccountRegistry::invokeLogin()
                             });
                     connection->assumeIdentity(
                         account.userId(),
+                        account.deviceId(),
                         QString::fromUtf8(accessTokenLoadingJob->binaryData()));
                     add(connection);
                 });

Quotient/connection.cpp

@@ -171,8 +171,10 @@ void Connection::loginWithToken(const QString& loginToken,
                      QString() /*password*/, loginToken, deviceId, initialDeviceName);
 }
 
-void Connection::assumeIdentity(const QString& mxId, const QString& accessToken)
+void Connection::assumeIdentity(const QString& mxId, const QString& deviceId, const QString& accessToken)
 {
+    d->data->setDeviceId(deviceId);
+    d->completeSetup(mxId);
     d->ensureHomeserver(mxId).then([this, mxId, accessToken] {
         d->data->setToken(accessToken.toLatin1());
         callApi<GetTokenOwnerJob>().onResult([this, mxId](const GetTokenOwnerJob* job) {
@@ -182,8 +184,6 @@ void Connection::assumeIdentity(const QString& mxId, const QString& accessToken)
                     qCWarning(MAIN).nospace()
                         << "The access_token owner (" << job->userId()
                         << ") is different from passed MXID (" << mxId << ")!";
-                d->data->setDeviceId(job->deviceId());
-                d->completeSetup(job->userId());
                 return;
             case BaseJob::NetworkError:
                 emit networkError(job->errorString(), job->rawDataSample(), job->maxRetries(), -1);

Quotient/connection.h

@@ -650,7 +650,7 @@ public Q_SLOTS:
     //! Similar to loginWithPassword(), this method checks that the homeserver
     //! URL is valid and tries to resolve it from the MXID in case it is not.
     //! \since 0.7.2
-    void assumeIdentity(const QString& mxId, const QString& accessToken);
+    void assumeIdentity(const QString& mxId, const QString& deviceId, const QString& accessToken);
 
     //! Explicitly request capabilities from the server
     void reloadCapabilities();