Fix missing shadows for tooltip and menu

ui/base/x/x11_util_unittest.cc

+// Copyright 2016 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 "base/environment.h"
+#include "testing/gtest/include/gtest/gtest.h"
+#include "ui/base/x/x11_util_internal.h"
+#include "ui/gfx/x/x11_types.h"
+
+namespace ui {
+
+class X11UtilTest : public testing::Test {
+ public:
+  X11UtilTest() {}
+  ~X11UtilTest() override {}
+};
+
+TEST_F(X11UtilTest, ChooseVisualForWindow) {
+  XDisplay* display = gfx::GetXDisplay();
+  XWindowAttributes windowAttribs;

[sadrul, 2016/09/02 19:53:24]

window_attribs, or just attribs

[Julien Isorce Samsung, 2016/09/02 20:59:08]

Done.

+  Window root = XDefaultRootWindow(display);
+  Status status = XGetWindowAttributes(display, root, &windowAttribs);
+  DCHECK(status != 0);

[sadrul, 2016/09/02 19:53:24]

DCHECK_NE

[Julien Isorce Samsung, 2016/09/02 20:59:08]

Done.

+
+  int depth = 0;
+  bool has_compositing_manager = false;
+  ui::ChooseVisualForWindow(has_compositing_manager, NULL, &depth);

[sadrul, 2016/09/02 19:53:24]

nullptr instead of NULL

[Julien Isorce Samsung, 2016/09/02 20:59:08]

Done.

+  EXPECT_EQ(windowAttribs.depth, depth);
+
+  // Setting to true has no effect because it has been called with false before.
+  has_compositing_manager = true;
+  ui::ChooseVisualForWindow(has_compositing_manager, NULL, &depth);
+  EXPECT_EQ(windowAttribs.depth, depth);
+}
+
+}  // namespace ui