Fixing opacity for browser frame window on ASH/Windows.

chrome/browser/ui/views/chrome_views_delegate_chromeos.cc

Index: chrome/browser/ui/views/chrome_views_delegate_chromeos.cc
diff --git a/chrome/browser/ui/views/chrome_views_delegate_chromeos.cc b/chrome/browser/ui/views/chrome_views_delegate_chromeos.cc
new file mode 100644
index 0000000000000000000000000000000000000000..758ca91f0e00bf203d6ae75ca5d63fbec7be5ac7
--- /dev/null
+++ b/chrome/browser/ui/views/chrome_views_delegate_chromeos.cc
@@ -0,0 +1,17 @@
+// Copyright (c) 2013 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "chrome/browser/ui/views/chrome_views_delegate.h"
+
+#if defined(USE_CHROMEOS)

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

You shouldn't need an ifdef here.

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

Done.

+views::Widget::InitParams::WindowOpacity
+ChromeViewsDelegate::InferOpacity(views::Widget::InitParams* params) {
+  if (params->type != views::Widget::InitParams::TYPE_WINDOW &&
+      params->type != views::Widget::InitParams::TYPE_PANEL)
+    // All other windows except TYPE_WINDOW AND TYPE_PANEL should be

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

This comment isn't helpful since it just documents what the code does. Comments
should describe why the code is the way it is.

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

Done.

+    // opaque.
+    return views::Widget::InitParams::OPAQUE_WINDOW;
+  return params->opacity = views::Widget::InitParams::TRANSLUCENT_WINDOW;
+}
+#endif