Don't pass in WS_EX_TRANSPARENT in the window style in classic mode in AURA as that causes the wind…

ui/views/widget/widget_hwnd_utils.cc

Index: ui/views/widget/widget_hwnd_utils.cc
===================================================================
--- ui/views/widget/widget_hwnd_utils.cc	(revision 221492)
+++ ui/views/widget/widget_hwnd_utils.cc	(working copy)
@@ -63,8 +63,13 @@
   //    DwmExtendFrameIntoClientArea passing -1 as the margins.
   if (params.opacity == Widget::InitParams::TRANSLUCENT_WINDOW) {
 #if defined(USE_AURA)
-    if (ui::win::IsAeroGlassEnabled())
+    if (ui::win::IsAeroGlassEnabled()) {
       *ex_style |= WS_EX_COMPOSITED;
+    } else {
+      // In classic mode, the WS_EX_TRANSPARENT style causes the window to not
+      // paint correctly.
+      *ex_style &= ~WS_EX_TRANSPARENT;

[sky, 2013/09/06 21:03:47]

Won't this cause bad things to happen where we are using WS_EX_TRANSPARENT?

[ananta, 2013/09/06 21:23:26]

The HWNDMessageHandler::SetInitialFocus function is the only place we need to
worry about, I think. It does not set focus if the window has either of the
WS_EX_TRANSPARENT or WS_EX_NOACTIVATE.
In any case as this style brings in bugs, we should not be using it.

[ananta, 2013/09/06 22:08:50]

As per our discussion we pass in accept_events = true when creating windows in
desktop AURA. I will look into the painting issues with transparent separately.

+    }
 #else
     *ex_style |= WS_EX_LAYERED;
 #endif