qt: use persisted profile, not off-the-record

Author: benma

frontends/qt/main.cpp

@@ -66,6 +66,7 @@
 // their CSP response headers.
 static const char* scheme = "bitboxapp";
 
+static QWebEngineProfile* profile;
 static QWebEngineView* view;
 static QWebEnginePage* mainPage;
 static QWebEnginePage* externalPage;
@@ -92,7 +93,7 @@ class BitBoxApp : public SingleApplication
 
 class WebEnginePage : public QWebEnginePage {
 public:
-    WebEnginePage(QObject* parent) : QWebEnginePage(parent) {}
+    WebEnginePage(QWebEngineProfile* profile) : QWebEnginePage(profile) {}
 
     QWebEnginePage* createWindow(QWebEnginePage::WebWindowType type) {
         Q_UNUSED(type);
@@ -354,7 +355,11 @@ int main(int argc, char *argv[])
     }
 
     externalPage = new QWebEnginePage(view);
-    mainPage = new WebEnginePage(view);
+
+    profile = new QWebEngineProfile("BitBoxApp");
+    // Set parent to profile, making sure mainPage is deleted before the profile. This is a requirement for
+    // the profile to be able to flush to disk properly.
+    mainPage = new WebEnginePage(profile);
     view->setPage(mainPage);
 
     pageLoaded = false;
@@ -464,6 +469,10 @@ int main(int argc, char *argv[])
         webClass = nullptr;
         delete view;
         view = nullptr;
+        delete mainPage;
+        mainPage = nullptr;
+        delete profile;
+        profile = nullptr;
         webClassMutex.unlock();
         backendShutdown();
     });