zhangqd / chromiumembedded

Automatically exported from code.google.com/p/chromiumembedded
0 stars 1 forks source link

Incorrect default background color #1252

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. Browse to about:blank with cefclient.

What is the expected output? What do you see instead?
The background should be the default white color instead of black. The attached 
image shows the black background.

What version of the product are you using? On what operating system?
This happens with VS2012 in 1750 revision 1651 on Windows 8.1.

Please provide any additional information below.
This appears to be related to the fix for issue 1161. The background color is 
set to 0 by default, but switched to white in browser_host_impl_win.cc, line 
784. However, the same switch is not done in browser_settings.cc, line 131.

Modifying the default to white in browser_settings.cc does fix the issue for 
me, but it may be more useful to actually default the setting to white earlier 
instead of waiting until it's used.

Original issue reported on code.google.com by alexar...@gmail.com on 15 Apr 2014 at 2:47

Attachments:

GoogleCodeExporter commented 9 years ago

Original comment by magreenb...@gmail.com on 15 Apr 2014 at 7:00