From 45a6619e445cc1b0a1a86811b6ec1e1bbb56b90b Mon Sep 17 00:00:00 2001
From: Jacek Caban <jacek@codeweavers.com>
Date: Wed, 30 May 2012 16:53:32 +0200
Subject: [PATCH] urlmon: Correctly handle BINDF_NO_UI in handle_http_error.

---
 dlls/urlmon/http.c | 28 ++++++++++++----------------
 1 file changed, 12 insertions(+), 16 deletions(-)

diff --git a/dlls/urlmon/http.c b/dlls/urlmon/http.c
index e1ac49c9e90..7ec93fc8691 100644
--- a/dlls/urlmon/http.c
+++ b/dlls/urlmon/http.c
@@ -119,6 +119,9 @@ static HRESULT handle_http_error(HttpProtocol *This, DWORD error)
     IWindowForBindingUI *wfb_ui;
     IHttpSecurity *http_security;
     BOOL security_problem;
+    DWORD dlg_flags;
+    HWND hwnd;
+    DWORD res;
     HRESULT hres;
 
     switch(error) {
@@ -183,7 +186,6 @@ static HRESULT handle_http_error(HttpProtocol *This, DWORD error)
     hres = IServiceProvider_QueryService(serv_prov, &IID_IWindowForBindingUI, &IID_IWindowForBindingUI,
                                          (void**)&wfb_ui);
     if(SUCCEEDED(hres)) {
-        HWND hwnd;
         const IID *iid_reason;
 
         if(security_problem)
@@ -194,26 +196,20 @@ static HRESULT handle_http_error(HttpProtocol *This, DWORD error)
             iid_reason = &IID_IWindowForBindingUI;
 
         hres = IWindowForBindingUI_GetWindow(wfb_ui, iid_reason, &hwnd);
-        if(SUCCEEDED(hres) && hwnd)
-        {
-            DWORD res;
-
-            res = InternetErrorDlg(hwnd, This->base.request, error,
-                                   FLAGS_ERROR_UI_FLAGS_CHANGE_OPTIONS | FLAGS_ERROR_UI_FLAGS_GENERATE_DATA,
-                                   NULL);
-
-            if(res == ERROR_INTERNET_FORCE_RETRY || res == ERROR_SUCCESS)
-                hres = RPC_E_RETRY;
-            else
-                hres = E_FAIL;
-        }
         IWindowForBindingUI_Release(wfb_ui);
+        if(FAILED(hres))
+            hwnd = NULL;
     }
 
     IServiceProvider_Release(serv_prov);
 
-    if(hres == RPC_E_RETRY)
-        return hres;
+    dlg_flags = FLAGS_ERROR_UI_FLAGS_CHANGE_OPTIONS | FLAGS_ERROR_UI_FLAGS_GENERATE_DATA;
+    if(This->base.bindf & BINDF_NO_UI)
+        dlg_flags |= FLAGS_ERROR_UI_FLAGS_NO_UI;
+
+    res = InternetErrorDlg(hwnd, This->base.request, error, dlg_flags, NULL);
+    if(res == ERROR_INTERNET_FORCE_RETRY || res == ERROR_SUCCESS)
+        return RPC_E_RETRY;
 
     return internet_error_to_hres(error);
 }
-- 
2.11.4.GIT