MacViews: Fix positioning of top-level Widgets with parents

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 987 matching lines @@
 // An extension to WidgetBoundsChanged to ensure notifications are forwarded
 // by the NativeWidget implementation.
 TEST_F(WidgetObserverTest, WidgetBoundsChangedNative) {
   // Don't use NewWidget(), so that the Init() flow can be observed to ensure
   // consistency across platforms.
   Widget* widget = new Widget();  // Note: owned by NativeWidget.
   widget->AddObserver(this);
 
   EXPECT_FALSE(widget_bounds_changed());
 
-  // Init causes a bounds change, even while not showing.
-  widget->Init(CreateParams(Widget::InitParams::TYPE_WINDOW));
+  // Init causes a bounds change, even while not showing. Note some platforms
+  // cause a bounds change even when the bounds are empty. Mac does not.
+  Widget::InitParams params = CreateParams(Widget::InitParams::TYPE_WINDOW);
+  params.bounds = gfx::Rect(0, 0, 100, 100);
+  widget->Init(params);
   EXPECT_TRUE(widget_bounds_changed());
   reset();
 
   // Resizing while hidden, triggers a change.
   widget->SetSize(gfx::Size(160, 100));
   EXPECT_FALSE(widget->IsVisible());
   EXPECT_TRUE(widget_bounds_changed());
   reset();
 
   // Setting the same size does nothing.
@@ skipping 2389 matching lines @@
   bool handled = false;
   target->HandleKeyboardMessage(WM_CHAR, 0, 0, &handled);
   target->HandleKeyboardMessage(WM_SYSCHAR, 0, 0, &handled);
   target->HandleKeyboardMessage(WM_SYSDEADCHAR, 0, 0, &handled);
   widget.CloseNow();
 }
 #endif
 
 }  // namespace test
 }  // namespace views