MacViews: Fix CompositingWidgetTest.Transparency_DesktopWidgetTranslucent

ui/views/widget/widget_unittest.cc

Index: ui/views/widget/widget_unittest.cc
diff --git a/ui/views/widget/widget_unittest.cc b/ui/views/widget/widget_unittest.cc
index 0880a2b25ce4ff3ef531e1520dcd7f118e77b356..451752ed5077af06334d6ba42ae56f80099859fd 100644
--- a/ui/views/widget/widget_unittest.cc
+++ b/ui/views/widget/widget_unittest.cc
@@ -3862,8 +3862,20 @@ class CompositingWidgetTest : public views::test::WidgetTest {
            widget_type == Widget::InitParams::TYPE_CONTROL))
         continue;
 
+#if defined(OS_MACOSX)
+      // Mac always always has a compositing window manager, but doesn't have
+      // transparent titlebars which is what ShouldWindowContentsBeTransparent()
+      // is currently used for. Asking for transparency should get it. Note that
+      // TestViewsDelegate::use_transparent_windows_ determines the result of
+      // INFER_OPACITY: assume it is false.
+      bool should_be_transparent =
+          opacity == Widget::InitParams::TRANSLUCENT_WINDOW;
+#else
+      bool should_be_transparent = widget.ShouldWindowContentsBeTransparent();
+#endif
+
       EXPECT_EQ(IsNativeWindowTransparent(widget.GetNativeWindow()),
-                widget.ShouldWindowContentsBeTransparent());
+                should_be_transparent);

[sadrul, 2016/12/07 18:03:35]

Should NativeWidgetMac::ShouldWindowContentsBeTransparent() be updated instead?
Although from the comment above, it sounds like not?

[tapted, 2016/12/07 23:35:52]

Yeah I thought about it, but I think
NativeWidgetMac::ShouldWindowContentsBeTransparent() is returning the right
thing for now. There's a bit more in the CL description about how tab_strip.cc
uses it.

Looking a the code in TabStrip::GetInactiveAlpha(), I think if I end up needing
to change NativeWidgetMac::ShouldWindowContentsBeTransparent(), the right fix
would be to rename the function to something like `GetNativeWindowFrameAlpha()`,
and return a float (or [0,255], or an SkAlpha). (currently
TabStrip::GetInactiveAlpha() relies on #if defined(USE_ASH))

 
       // When using the Mandoline UI Service, the translucency does not rely on
       // the widget type.
@@ -3899,15 +3911,7 @@ TEST_F(CompositingWidgetTest, Transparency_DesktopWidgetOpaque) {
   CheckAllWidgetsForOpacity(Widget::InitParams::OPAQUE_WINDOW);
 }
 
-// Failing on Mac. http://cbrug.com/623421
-#if defined(OS_MACOSX)
-#define MAYBE_Transparency_DesktopWidgetTranslucent \
-    DISABLED_Transparency_DesktopWidgetTranslucent
-#else
-#define MAYBE_Transparency_DesktopWidgetTranslucent \
-    Transparency_DesktopWidgetTranslucent
-#endif
-TEST_F(CompositingWidgetTest, MAYBE_Transparency_DesktopWidgetTranslucent) {
+TEST_F(CompositingWidgetTest, Transparency_DesktopWidgetTranslucent) {
   CheckAllWidgetsForOpacity(Widget::InitParams::TRANSLUCENT_WINDOW);
 }