Some improvement of validation message bubble UI.

chrome/browser/ui/cocoa/validation_message_bubble_cocoa.mm

 // Copyright (c) 2013 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/memory/scoped_ptr.h"
 #include "base/strings/sys_string_conversions.h"
 #import "chrome/browser/ui/cocoa/info_bubble_view.h"
 #import "chrome/browser/ui/cocoa/info_bubble_window.h"
 #import "chrome/browser/ui/cocoa/validation_message_bubble_controller.h"
 #include "chrome/browser/ui/validation_message_bubble.h"
 #include "content/public/browser/render_widget_host.h"
 #include "content/public/browser/render_widget_host_view.h"
 #include "grit/theme_resources.h"
 #import "third_party/GTM/AppKit/GTMUILocalizerAndLayoutTweaker.h"
+#import "ui/base/cocoa/base_view.h"
 #import "ui/base/cocoa/flipped_view.h"
 #include "ui/base/resource/resource_bundle.h"
 
 const CGFloat kWindowInitialWidth = 200;
 const CGFloat kWindowInitialHeight = 100;
 const CGFloat kWindowMinWidth = 64;
 const CGFloat kWindowMaxWidth = 256;
 const CGFloat kWindowPadding = 8;
 const CGFloat kIconTextMargin = 4;
 const CGFloat kTextVerticalMargin = 4;
@@ skipping 102 matching lines @@
   return contentView;
 }
 
 
 @end  // implementation ValidationMessageBubbleCocoa
 
 // ----------------------------------------------------------------
 
 namespace {
 
+// Converts |anchor_in_root_view| in rwhv coordinates to cocoa screen
+// coordinates, and returns an NSPoint at the center of the bottom side of the
+// converted rectangle.
+NSPoint GetAnchorPoint(content::RenderWidgetHost* widget_host,
+                       const gfx::Rect& anchor_in_root_view) {
+  BaseView* view =
+      static_cast<BaseView*>(widget_host->GetView()->GetNativeView());

[Nico, 2013/06/10 14:51:40]

consider base::mac::ObjCCastStrict<> instead of static_cast<>

[tkent, 2013/06/11 07:02:43]

Done.

+  NSRect cocoaRect = [view flipRectToNSRect:anchor_in_root_view];
+  NSRect windowRect = [view convertRect:cocoaRect toView:nil];
+  NSPoint point = NSMakePoint(NSMidX(windowRect), NSMinY(windowRect));
+  return [[view window] convertBaseToScreen:point];
+}
+
 class ValidationMessageBubbleCocoa : public chrome::ValidationMessageBubble {
  public:
   ValidationMessageBubbleCocoa(content::RenderWidgetHost* widget_host,
-                               const gfx::Rect& anchor_in_screen,
+                               const gfx::Rect& anchor_in_root_view,
                                const string16& main_text,
                                const string16& sub_text) {
-    NSWindow* parent_window = [widget_host->GetView()->GetNativeView() window];
-    NSPoint anchor_point = NSMakePoint(
-        anchor_in_screen.x() + anchor_in_screen.width() / 2,
-        NSHeight([[parent_window screen] frame])
-            - (anchor_in_screen.y() + anchor_in_screen.height()));
     controller_.reset([[[ValidationMessageBubbleController alloc]
-                        init:parent_window
-                  anchoredAt:anchor_point
+                        init:[widget_host->GetView()->GetNativeView() window]
+                  anchoredAt:GetAnchorPoint(widget_host, anchor_in_root_view)
                     mainText:main_text
                      subText:sub_text] retain]);
   }
 
   virtual ~ValidationMessageBubbleCocoa() {
     [controller_.get() close];
   }
 
+  virtual void SetPositionRelativeToAnchor(
+      content::RenderWidgetHost* widget_host,
+      const gfx::Rect& anchor_in_root_view) OVERRIDE {
+    [controller_.get()
+        setAnchorPoint:GetAnchorPoint(widget_host, anchor_in_root_view)];
+  }
+
  private:
   scoped_nsobject<ValidationMessageBubbleController> controller_;
 };
 
 }
 
 // ----------------------------------------------------------------
 
 namespace chrome {
 
 scoped_ptr<ValidationMessageBubble> ValidationMessageBubble::CreateAndShow(
     content::RenderWidgetHost* widget_host,
-    const gfx::Rect& anchor_in_screen,
+    const gfx::Rect& anchor_in_root_view,
     const string16& main_text,
     const string16& sub_text) {
   return scoped_ptr<ValidationMessageBubble>(new ValidationMessageBubbleCocoa(
-      widget_host, anchor_in_screen, main_text, sub_text)).Pass();
+      widget_host, anchor_in_root_view, main_text, sub_text)).Pass();
 }
 
 }