MacViews: Fix positioning of top-level Widgets with parents

ui/views/cocoa/bridged_native_widget.h

 // Copyright 2014 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.
 
 #ifndef UI_VIEWS_COCOA_BRIDGED_NATIVE_WIDGET_H_
 #define UI_VIEWS_COCOA_BRIDGED_NATIVE_WIDGET_H_
 
 #import <Cocoa/Cocoa.h>
 #include <vector>
 
@@ skipping 44 matching lines @@
   ~BridgedNativeWidget() override;
 
   // Initialize the bridge, "retains" ownership of |window|.
   void Init(base::scoped_nsobject<NSWindow> window,
             const Widget::InitParams& params);
 
   // Sets or clears the focus manager to use for tracking focused views.
   // This does NOT take ownership of |focus_manager|.
   void SetFocusManager(FocusManager* focus_manager);
 
-  // Changes the bounds of the window and the hosted layer if present.
+  // Changes the bounds of the window and the hosted layer if present. The
+  // origin is a location in screen coordinates except for "child" windows,
+  // which are positioned relative to their parent(). SetBounds() considers a
+  // "child" window to be one initialized with InitParams specifying all of:
+  // a |parent| NSWindow, the |child| attribute, and a |type| that
+  // views::GetAuraWindowTypeForWidgetType does not consider a "popup" type.
   void SetBounds(const gfx::Rect& new_bounds);
 
   // Set or clears the views::View bridged by the content view. This does NOT
   // take ownership of |view|.
   void SetRootView(views::View* view);
 
   // Sets the desired visibility of the window and updates the visibility of
   // descendant windows where necessary.
   void SetVisibilityState(WindowVisibilityState new_state);
 
@@ skipping 128 matching lines @@
       const std::vector<ui::LatencyInfo>& latency_info) override;
   void AcceleratedWidgetHitError() override;
 
   views::NativeWidgetMac* native_widget_mac_;  // Weak. Owns this.
   base::scoped_nsobject<NSWindow> window_;
   base::scoped_nsobject<ViewsNSWindowDelegate> window_delegate_;
   base::scoped_nsobject<BridgedContentView> bridged_view_;
   scoped_ptr<ui::InputMethod> input_method_;
   scoped_ptr<CocoaMouseCapture> mouse_capture_;
   FocusManager* focus_manager_;  // Weak. Owned by our Widget.
+  Widget::InitParams::Type widget_type_;
 
   BridgedNativeWidget* parent_;  // Weak. If non-null, owns this.
   std::vector<BridgedNativeWidget*> child_windows_;
 
   base::scoped_nsobject<NSView> compositor_superview_;
   scoped_ptr<ui::AcceleratedWidgetMac> compositor_widget_;
   scoped_ptr<ui::Compositor> compositor_;
 
   // Tracks the bounds when the window last started entering fullscreen. Used to
   // provide an answer for GetRestoredBounds(), but not ever sent to Cocoa (it
@@ skipping 15 matching lines @@
   // If true, the window is either visible, or wants to be visible but is
   // currently hidden due to having a hidden parent.
   bool wants_to_be_visible_;
 
   DISALLOW_COPY_AND_ASSIGN(BridgedNativeWidget);
 };
 
 }  // namespace views
 
 #endif  // UI_VIEWS_COCOA_BRIDGED_NATIVE_WIDGET_H_