MacViews: Ensure the RootView is in place before the first call to ReorderNativeViews

ui/views/widget/widget_unittest.cc

 // Copyright (c) 2012 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 <algorithm>
 #include <set>
 
 #include "base/basictypes.h"
 #include "base/bind.h"
 #include "base/memory/scoped_ptr.h"
@@ skipping 3076 matching lines @@
   // Drargging. Cover HasCapture() and NativeWidgetPrivate::IsMouseButtonDown().
   generator.DragMouseTo(gfx::Point(40, 40));
   EXPECT_EQ(3, view->GetEventCount(ui::ET_MOUSE_PRESSED));
   EXPECT_EQ(3, view->GetEventCount(ui::ET_MOUSE_RELEASED));
   EXPECT_EQ(1, view->GetEventCount(ui::ET_MOUSE_DRAGGED));
   EXPECT_EQ(ui::EF_LEFT_MOUSE_BUTTON, view->last_flags());
 
   widget->CloseNow();
 }
 
+// Tests that the root view is correctly set up for Widget types that do not
+// require a non-client view, before any other views are added to the widget.
+// That is, before Widget::ReorderNativeViews() is called which, if called with
+// a root view not set, could cause the root view to get resized to the widget.
+TEST_F(WidgetTest, NonClientWindowValidAfterInit) {
+  Widget* widget = CreateTopLevelFramelessPlatformWidget();
+  View* root_view = widget->GetRootView();
+
+  // Size the root view to exceed the widget bounds.
+  const gfx::Rect kTestRect(0, 0, 500, 500);

[sky, 2014/09/10 14:48:36]

nit: k is only for global or constants defined within a class, not functional
locals.

[tapted, 2014/09/11 00:17:16]

Done.

+  root_view->SetBoundsRect(kTestRect);
+
+  EXPECT_NE(kTestRect.size(), widget->GetWindowBoundsInScreen().size());
+
+  EXPECT_EQ(kTestRect, root_view->bounds());
+  widget->ReorderNativeViews();
+  EXPECT_EQ(kTestRect, root_view->bounds());
+
+  widget->CloseNow();
+}
+
 }  // namespace test
 }  // namespace views