Thanks for fixing this. However, shouldn't this have been pushed into the 4.12 branch first since it is a bug fix?


On Wed, Mar 5, 2014 at 4:50 AM, David Faure <faure@kde.org> wrote:
Git commit fa77b7702fb698ee0bc74e4818eb4060ca636848 by David Faure.
Committed on 05/03/2014 at 09:49.
Pushed by dfaure into branch 'KDE/4.13'.

Handle "Cancel" in the file dialog correctly.

It used to loop in the while(true) and keep popping up the dialog
over and over again.

No time to look up for bug reports on bugs.kde.org :/

CCMAIL: adawit@kde.org

M  +1    -1    kdewebkit/kwebpage.cpp

http://commits.kde.org/kdelibs/fa77b7702fb698ee0bc74e4818eb4060ca636848

diff --git a/kdewebkit/kwebpage.cpp b/kdewebkit/kwebpage.cpp
index 63fd293..04f5081 100644
--- a/kdewebkit/kwebpage.cpp
+++ b/kdewebkit/kwebpage.cpp
@@ -582,7 +582,7 @@ bool KWebPage::handleReply(QNetworkReply* reply, QString* contentType, KIO::Meta
                             return true;
                     }
                     if (!downloadResource(replyUrl, suggestedFileName, d->windowWidget()))
-                        break;
+                        return true; // file dialog was cancelled, stop here
                 }
                 return true;
             case KParts::BrowserOpenOrSaveQuestion::Cancel: