Fix deleting cookies

Closes #2137

bin/themes/linux/main.css

@@ -117,7 +117,7 @@ IconProvider
 {
     margin-left: 5px;
     border-image: url(images/transp.png);
-    qproperty-fixedsize: 22px 16px;
+    qproperty-fixedsize: 21px 16px;
 }
 
 #locationbar-goicon

src/lib/cookies/cookiejar.cpp

@@ -57,6 +57,11 @@ void CookieJar::setAllowCookies(bool allow)
     m_allowCookies = allow;
 }
 
+void CookieJar::deleteCookie(const QNetworkCookie &cookie)
+{
+    m_client->deleteCookie(cookie);
+}
+
 QVector<QNetworkCookie> CookieJar::getAllCookies() const
 {
     return m_cookies;

src/lib/cookies/cookiejar.h

@@ -37,6 +37,8 @@ public:
 
     void setAllowCookies(bool allow);
 
+    void deleteCookie(const QNetworkCookie &cookie);
+
     QVector<QNetworkCookie> getAllCookies() const;
     void deleteAllCookies();
 

src/lib/cookies/cookiemanager.cpp

@@ -116,16 +116,21 @@ void CookieManager::remove()
         return;
     }
 
+    QList<QNetworkCookie> cookies;
+
     if (current->childCount()) {
         for (int i = 0; i < current->childCount(); ++i) {
             QTreeWidgetItem *item = current->child(i);
-            if (item && m_itemHash.contains(item))
-                removeCookie(m_itemHash.value(item));
+            if (item && m_itemHash.contains(item)) {
+                cookies.append(m_itemHash.value(item));
+            }
         }
+    } else if (m_itemHash.contains(current)) {
+        cookies.append(m_itemHash.value(current));
     }
-    else {
-        if (m_itemHash.contains(current))
-            removeCookie(m_itemHash.value(current));
+
+    foreach (const QNetworkCookie &cookie, cookies) {
+        mApp->cookieJar()->deleteCookie(cookie);
     }
 }
 

src/lib/cookies/cookiemanager.h

@@ -40,8 +40,6 @@ public:
     explicit CookieManager();
     ~CookieManager();
 
-    void refreshTable();
-
 private slots:
     void currentItemChanged(QTreeWidgetItem* current, QTreeWidgetItem* parent);
     void remove();