2009-05-21 Xan Lopez <xlopez@igalia.com>
authorxan@webkit.org <xan@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 22 May 2009 19:06:07 +0000 (19:06 +0000)
committerxan@webkit.org <xan@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 22 May 2009 19:06:07 +0000 (19:06 +0000)
        Reviewed by Maciej Stachowiak.

        https://bugs.webkit.org/show_bug.cgi?id=24818
        [GTK] http auth dialog pops up twice after a cancelled atempt

        Workaround libsoup bug
        http://bugzilla.gnome.org/show_bug.cgi?id=583462, which made
        libsoup emit the 'authenticate' signal more often than it was
        expected.

        We'll be able to remove this when we depend on libsoup > 2.26.2.

        * webkit/webkitsoupauthdialog.c:
        (session_authenticate):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@44052 268f45cc-cd09-0410-ab3c-d52691b4dbfc

WebKit/gtk/ChangeLog
WebKit/gtk/webkit/webkitsoupauthdialog.c

index c9ebe85dff16a4426261a5d115289cc6e0f0f3ee..154cee3e859b6b4ade242dfcb3587617b8e6c5b4 100644 (file)
@@ -1,3 +1,20 @@
+2009-05-21  Xan Lopez  <xlopez@igalia.com>
+
+        Reviewed by Maciej Stachowiak.
+
+        https://bugs.webkit.org/show_bug.cgi?id=24818
+        [GTK] http auth dialog pops up twice after a cancelled atempt
+
+        Workaround libsoup bug
+        http://bugzilla.gnome.org/show_bug.cgi?id=583462, which made
+        libsoup emit the 'authenticate' signal more often than it was
+        expected.
+
+        We'll be able to remove this when we depend on libsoup > 2.26.2.
+
+        * webkit/webkitsoupauthdialog.c:
+        (session_authenticate):
+
 2009-05-21  Darin Fisher  <darin@chromium.org>
 
         Fix build bustage.
 2009-05-21  Darin Fisher  <darin@chromium.org>
 
         Fix build bustage.
index 335f5367f66a4ee60965158b4cd86fe493759efa..d7043b8c10b84a3b82b39010ce5d32782e9b292f 100644 (file)
@@ -315,6 +315,13 @@ static void session_authenticate(SoupSession* session, SoupMessage* msg, SoupAut
     WebKitAuthData* authData;
     SoupSessionFeature* manager = (SoupSessionFeature*)user_data;
 
     WebKitAuthData* authData;
     SoupSessionFeature* manager = (SoupSessionFeature*)user_data;
 
+    /* 
+     * Workaround for http://bugzilla.gnome.org/show_bug.cgi?id=583462
+     * FIXME: we can remove this once we depend on a libsoup newer than 2.26.2
+     */
+    if (msg->status_code == 0)
+        return;
+
     soup_session_pause_message(session, msg);
     /* We need to make sure the message sticks around when pausing it */
     g_object_ref(msg);
     soup_session_pause_message(session, msg);
     /* We need to make sure the message sticks around when pausing it */
     g_object_ref(msg);