Don't use double QUrl::fromUserInput in QupZilla constructor.

Closes #762

CHANGELOG

@@ -21,6 +21,7 @@ Version 1.4.0
     * don't steal Ctrl+B/U/I shortcuts from page
     * disabled by default opacity effect on tab previews - see BUILDING
     * improved showing navigation toolbar in fullscreen
+    * fixed opening browser with url with ampersand (%26) as command line argument
     * fixed scrolling to anchor in background tabs
     * fixed parsing UTF-8 filenames in Content-Disposition header
     * fixed crash with context menu in websearchbar and locationbar

src/lib/app/mainapplication.cpp

@@ -564,12 +564,12 @@ void MainApplication::receiveAppMessage(QString message)
     QUrl actUrl;
 
     if (message.startsWith(QLatin1String("URL:"))) {
-        QUrl url = QUrl::fromUserInput(message.mid(4));
+        const QUrl &url = QUrl::fromUserInput(message.mid(4));
         addNewTab(url);
         actWin = getWindow();
     }
     else if (message.startsWith(QLatin1String("ACTION:"))) {
-        QString text = message.mid(7);
+        const QString &text = message.mid(7);
         if (text == QLatin1String("NewTab")) {
             addNewTab();
         }
@@ -589,8 +589,11 @@ void MainApplication::receiveAppMessage(QString message)
         }
     }
 
-    if (!actWin && !isClosing()) { // It can only occur if download manager window was still open
-        makeNewWindow(Qz::BW_NewWindow, actUrl);
+    if (!actWin) {
+        if (!isClosing()) {
+            // It can only occur if download manager window was still opened
+            makeNewWindow(Qz::BW_NewWindow, actUrl);
+        }
         return;
     }
 

src/lib/app/qupzilla.cpp

@@ -204,7 +204,7 @@ void QupZilla::postLaunch()
     show();
 
     if (!m_startingUrl.isEmpty()) {
-        startUrl = QUrl::fromUserInput(m_startingUrl.toString());
+        startUrl = m_startingUrl;
         addTab = true;
     }