Fixing opacity for browser frame window on ASH/Windows.

chrome/browser/ui/views/chrome_views_delegate.cc

Index: chrome/browser/ui/views/chrome_views_delegate.cc
diff --git a/chrome/browser/ui/views/chrome_views_delegate.cc b/chrome/browser/ui/views/chrome_views_delegate.cc
index 65f44b5e6f12b8281697b7ee3d25648f4a9326c2..dcda4e401077b63c40ebdf1a04a3b21ee46b8428 100644
--- a/chrome/browser/ui/views/chrome_views_delegate.cc
+++ b/chrome/browser/ui/views/chrome_views_delegate.cc
@@ -175,21 +175,6 @@ views::NonClientFrameView* ChromeViewsDelegate::CreateDefaultNonClientFrameView(
   return NULL;
 }
 
-bool ChromeViewsDelegate::UseTransparentWindows() const {
-#if defined(USE_ASH)
-  // TODO(scottmg): http://crbug.com/133312. This needs context to determine
-  // if it's desktop or ash.
-#if defined(OS_CHROMEOS)
-  return true;
-#else
-  NOTIMPLEMENTED();
-  return false;
-#endif
-#else
-  return false;
-#endif
-}
-
 void ChromeViewsDelegate::AddRef() {
   g_browser_process->AddRefModule();
 }
@@ -207,6 +192,11 @@ content::WebContents* ChromeViewsDelegate::CreateWebContents(
 void ChromeViewsDelegate::OnBeforeWidgetInit(
     views::Widget::InitParams* params,
     views::internal::NativeWidgetDelegate* delegate) {
+  // We would need to determine opacity if it's not already specified.
+  if (params->opacity == views::Widget::InitParams::INFER_OPACITY) {

[sky, 2013/12/17 15:59:00]

nit: no {}

[Shrikant Kelkar, 2013/12/17 18:01:00]

Done.

+    params->opacity = InferOpacity(params);
+  }
+
   // If we already have a native_widget, we don't have to try to come
   // up with one.
   if (params->native_widget)
@@ -311,3 +301,10 @@ base::TimeDelta
 ChromeViewsDelegate::GetDefaultTextfieldObscuredRevealDuration() {
   return base::TimeDelta();
 }
+
+#if !defined(USE_AURA) && !defined(USE_CHROMEOS)
+views::Widget::InitParams::WindowOpacity
+ChromeViewsDelegate::InferOpacity(views::Widget::InitParams* params) {
+  return views::Widget::InitParams::OPAQUE_WINDOW;
+}
+#endif