From ef402c3ba1147c39ab8b1f803b7bf8ebf1bbdefe Mon Sep 17 00:00:00 2001 From: "rsesek@chromium.org" Date: Thu, 6 Oct 2011 22:43:37 +0000 Subject: [Mac] Do not cause a layout when the AvatarMenuModel changes. BUG=99314 TEST=From the avatar bubble menu, click "New user". The bubble doesn't jump up. Review URL: http://codereview.chromium.org/8173013 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@104381 0039d316-1c4b-4281-b951-d872f2087c98 --- chrome/browser/ui/cocoa/browser/avatar_menu_bubble_controller.mm | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/chrome/browser/ui/cocoa/browser/avatar_menu_bubble_controller.mm b/chrome/browser/ui/cocoa/browser/avatar_menu_bubble_controller.mm index 939bebb..e1efc80 100644 --- a/chrome/browser/ui/cocoa/browser/avatar_menu_bubble_controller.mm +++ b/chrome/browser/ui/cocoa/browser/avatar_menu_bubble_controller.mm @@ -37,7 +37,10 @@ class Bridge : public AvatarMenuModelObserver { // AvatarMenuModelObserver: void OnAvatarMenuModelChanged(AvatarMenuModel* model) { - [controller_ performLayout]; + // Do nothing. Rebuilding while the bubble is open will cause it to be + // positioned incorrectly. Since the bubble will be dismissed on losing key + // status, it's impossible for the user to edit the information in a + // meaningful way such that it would need to be redrawn. } private: -- cgit v1.1