Add mac UI for password generation

chrome/browser/ui/cocoa/browser_window_cocoa.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.
 
 #include "chrome/browser/ui/cocoa/browser_window_cocoa.h"
 
 #include "base/bind.h"
 #include "base/command_line.h"
 #include "base/logging.h"
 #include "base/mac/mac_util.h"
 #include "base/message_loop.h"
 #include "base/sys_string_conversions.h"
 #include "chrome/app/chrome_command_ids.h"
 #include "chrome/browser/bookmarks/bookmark_utils.h"
 #include "chrome/browser/download/download_shelf.h"
 #include "chrome/browser/extensions/tab_helper.h"
+#include "chrome/browser/password_manager/password_manager.h"
 #include "chrome/browser/prefs/pref_service.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/ui/browser.h"
 #include "chrome/browser/ui/browser_command_controller.h"
 #include "chrome/browser/ui/browser_commands.h"
 #include "chrome/browser/ui/browser_list.h"
 #include "chrome/browser/ui/browser_tabstrip.h"
 #include "chrome/browser/ui/browser_window_state.h"
 #import "chrome/browser/ui/cocoa/browser/avatar_button_controller.h"
 #import "chrome/browser/ui/cocoa/browser/avatar_menu_bubble_controller.h"
 #import "chrome/browser/ui/cocoa/browser/edit_search_engine_cocoa_controller.h"
+#import "chrome/browser/ui/cocoa/browser/password_generation_bubble_controller.h"
 #import "chrome/browser/ui/cocoa/browser_window_controller.h"
 #import "chrome/browser/ui/cocoa/browser_window_utils.h"
 #import "chrome/browser/ui/cocoa/chrome_event_processing_window.h"
 #import "chrome/browser/ui/cocoa/download/download_shelf_controller.h"
 #include "chrome/browser/ui/cocoa/find_bar/find_bar_bridge.h"
 #import "chrome/browser/ui/cocoa/info_bubble_view.h"
 #import "chrome/browser/ui/cocoa/location_bar/location_bar_view_mac.h"
 #import "chrome/browser/ui/cocoa/nsmenuitem_additions.h"
 #include "chrome/browser/ui/cocoa/restart_browser.h"
 #include "chrome/browser/ui/cocoa/status_bubble_mac.h"
 #include "chrome/browser/ui/cocoa/task_manager_mac.h"
 #import "chrome/browser/ui/cocoa/toolbar/toolbar_controller.h"
 #import "chrome/browser/ui/cocoa/web_dialog_window_controller.h"
 #import "chrome/browser/ui/cocoa/website_settings_bubble_controller.h"
 #include "chrome/browser/ui/page_info_bubble.h"
 #include "chrome/browser/ui/tab_contents/tab_contents.h"
 #include "chrome/common/chrome_notification_types.h"
 #include "chrome/common/pref_names.h"
 #include "content/public/browser/native_web_keyboard_event.h"
 #include "content/public/browser/notification_details.h"
 #include "content/public/browser/notification_source.h"
 #include "content/public/browser/web_contents.h"
+#include "content/public/common/password_form.h"
 #include "grit/chromium_strings.h"
 #include "grit/generated_resources.h"
 #include "ui/base/l10n/l10n_util_mac.h"
 #include "ui/gfx/rect.h"
 
 #if defined(ENABLE_ONE_CLICK_SIGNIN)
 #import "chrome/browser/ui/cocoa/one_click_signin_bubble_controller.h"
 #endif
 
 using content::NativeWebKeyboardEvent;
@@ skipping 575 matching lines @@
   [controller_ destroyBrowser];
 
   // at this point the controller is dead (autoreleased), so
   // make sure we don't try to reference it any more.
 }
 
 NSWindow* BrowserWindowCocoa::window() const {
   return [controller_ window];
 }
 
-void BrowserWindowCocoa::ShowAvatarBubble(WebContents* web_contents,
-                                          const gfx::Rect& rect) {
+NSPoint BrowserWindowCocoa::GetPointForBubble(
+    content::WebContents* web_contents,
+    const gfx::Rect& rect) const {

[Scott Hess - ex-Googler, 2012/11/17 01:22:41]

This seems to only need the passed parameters - suggest moving it to the
anonymous namespace at the top fo the file, rather than exposing another helper
in the interface.

[Garrett Casto, 2012/11/19 22:17:42]

Done.

   NSView* view = web_contents->GetNativeView();
   NSRect bounds = [view bounds];
   NSPoint point;
   point.x = NSMinX(bounds) + rect.right();
   // The view's origin is at the bottom but |rect|'s origin is at the top.
   point.y = NSMaxY(bounds) - rect.bottom();
   point = [view convertPoint:point toView:nil];
   point = [[view window] convertBaseToScreen:point];
+  return point;
+}
+
+void BrowserWindowCocoa::ShowAvatarBubble(WebContents* web_contents,
+                                          const gfx::Rect& rect) {
+  NSPoint point = GetPointForBubble(web_contents, rect);
 
   // |menu| will automatically release itself on close.
   AvatarMenuBubbleController* menu =
       [[AvatarMenuBubbleController alloc] initWithBrowser:browser_
                                                anchoredAt:point];
   [[menu bubble] setAlignment:info_bubble::kAlignEdgeToAnchorEdge];
   [menu showWindow:nil];
 }
 
 void BrowserWindowCocoa::ShowAvatarBubbleFromAvatarButton() {
   [[controller_ avatarButtonController] showAvatarBubble];
 }
+
+void BrowserWindowCocoa::ShowPasswordGenerationBubble(
+    const gfx::Rect& rect,
+    const content::PasswordForm& form,
+    autofill::PasswordGenerator* password_generator) {
+  TabContents* tab_contents = chrome::GetActiveTabContents(browser_);
+  WebContents* web_contents = tab_contents->web_contents();
+  NSPoint point = GetPointForBubble(web_contents, rect);
+  // We want to point to the middle of the rect instead of the right side.
+  point.x -= rect.width()/2;

[Scott Hess - ex-Googler, 2012/11/17 01:22:41]

This is expressing a bit more knowledge of the internals of GetPointForBubble()
than I'm comfortable with.  I would prefer adjusting the input rect before
passing it rather than adjusting the output value based on an input value. 
Perhaps inset the rect by width()/2 before calling GetPointForBubble().  Or
maybe just pass in the x and y points relative to rect.

I mean like this for avatar:
  NSPoint point = GetPointForBubble(web_contents, rect.right, rect.bottom);
and like this for here:
  NSPoint point = GetPointForBubble(web_contents, rect.x() + rect.width() / 2,
rect.bottom);

[Garrett Casto, 2012/11/19 22:17:42]

Done.

[Garrett Casto, 2012/11/19 22:17:42]

Done.

+
+  PasswordGenerationBubbleController* controller =
+      [[PasswordGenerationBubbleController alloc]
+        initWithBrowser:browser_
+             anchoredAt:point
+         renderViewHost:web_contents->GetRenderViewHost()
+        passwordManager:PasswordManager::FromWebContents(web_contents)
+         usingGenerator:password_generator
+                forForm:form];
+  [controller showWindow:nil];
+}