Windows: modal dialog boxes smear on main window

[GoogleCodeExporter]

What steps will reproduce the problem?
1. Run the cefclient sample app.
2. From the Tests menu, choose Begin Tracing and then End Tracing.
3. When the Save As dialog box appears, drag it around.

What is the expected output? What do you see instead?

Expected: Main window updates properly under dragged dialog box.
Observed: Dialog box smears its image on the main window, and the main window 
does not refresh until the dialog box is dismissed.

What version of the product are you using? On what operating system?

3.2171.1949 (32 bit)
Windows 7 Enterprise (for both build and run)

Please provide any additional information below.

The same smearing appears to happen with any modal dialog box over the main 
window, including a MessageBox() shown in the ClientHandler::OnJSDialog() 
handler, and the open file dialog box in the ClientHandler::OnFileDialog() 
handler.

However, it doesn't happen with a "confirm quit" MessageBox() that I show in 
response to the WM_CLOSE message, so it appears to be something specific to 
showing a dialog box during a cef callback.

I didn't have this problem using 3.1650.1539.

This bug is currently blocking me from a release.

Original issue reported on code.google.com by makdaddy98102@gmail.com on 10 Dec 2014 at 11:09

• Merged into: #1347

Attachments:

• smear.png

[GoogleCodeExporter]

Related forum thread: http://magpcss.org/ceforum/viewtopic.php?f=6&t=12389

Original comment by magreenb...@gmail.com on 11 Dec 2014 at 9:39

• Changed title: Windows: modal dialog boxes smear on main window

[GoogleCodeExporter]

The old Chromium implementation of file dialogs used a separate dialog thread 
and the EnableWindow function to disable the parent window while the dialog was 
visible: 
https://chromium.googlesource.com/chromium/src/+/530f43f302f1eef9d3530767015891f
1db1f1272/chrome/browser/views/shell_dialogs_win.cc

Original comment by magreenb...@gmail.com on 15 Jan 2015 at 6:27

[GoogleCodeExporter]

Original comment by magreenb...@gmail.com on 16 Jan 2015 at 9:45

• Changed state: Duplicate