Mac: Give bubble windows an initial position before showing them.

chrome/browser/ui/cocoa/base_bubble_controller_unittest.mm

 // 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.
 
 #import "chrome/browser/ui/cocoa/base_bubble_controller.h"
 
 #include "base/mac/mac_util.h"
 #import "base/mac/scoped_nsobject.h"
 #import "chrome/browser/ui/cocoa/cocoa_test_helper.h"
 #import "chrome/browser/ui/cocoa/info_bubble_view.h"
@@ skipping 214 matching lines @@
 
   NSRect frame = [[controller_ window] frame];
   // Make sure the bubble size hasn't changed.
   EXPECT_EQ(frame.size.width, kBubbleWindowWidth);
   EXPECT_EQ(frame.size.height, kBubbleWindowHeight);
   // Make sure the bubble arrow points to the anchor.
   EXPECT_EQ(NSMidX(frame), kAnchorPointX);
   EXPECT_GE(NSMaxY(frame), kAnchorPointY);
 }
 
+// Test that the window is given an initial position before being shown. This
+// ensures offscreen initialization is done using correct screen metrics.
+TEST_F(BaseBubbleControllerTest, PositionedBeforeShow) {
+  // Verify default alignment settings, used when initialized in SetUp().
+  EXPECT_EQ(info_bubble::kTopRight, [[controller_ bubble] arrowLocation]);
+  EXPECT_EQ(info_bubble::kAlignArrowToAnchor, [[controller_ bubble] alignment]);
+
+  // Verify the default frame (positioned relative to the test_window() origin).
+  NSRect frame = [[controller_ window] frame];
+  EXPECT_EQ(NSMaxX(frame) - info_bubble::kBubbleArrowXOffset -
+      floorf(info_bubble::kBubbleArrowWidth / 2.0), kAnchorPointX);
+  EXPECT_EQ(NSMaxY(frame), kAnchorPointY);
+}
+
 // Tests that when a new window gets key state (and the bubble resigns) that
 // the key window changes.
 TEST_F(BaseBubbleControllerTest, ResignKeyCloses) {
   base::scoped_nsobject<NSWindow> other_window(
       [[NSWindow alloc] initWithContentRect:NSMakeRect(500, 500, 500, 500)
                                   styleMask:NSTitledWindowMask
                                     backing:NSBackingStoreBuffered
                                       defer:YES]);
 
   base::scoped_nsobject<BaseBubbleController> keep_alive = ShowBubble();
@@ skipping 129 matching lines @@
   [panel close];
 
   EXPECT_FALSE([bubble_window_ attachedSheet]);
   EXPECT_TRUE([bubble_window_ isVisible]);
   EXPECT_FALSE([panel isVisible]);
 
   // Now that the sheet is gone, a key status change should close the bubble.
   SimulateKeyStatusChange();
   EXPECT_FALSE([bubble_window_ isVisible]);
 }