diff options
| author | rsesek@chromium.org <rsesek@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-10-06 22:43:37 +0000 |
|---|---|---|
| committer | rsesek@chromium.org <rsesek@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-10-06 22:43:37 +0000 |
| commit | ef402c3ba1147c39ab8b1f803b7bf8ebf1bbdefe (patch) | |
| tree | 9865fa8dda11b79eca262abf0f57de23362c0091 | |
| parent | a598ba74fcf8f98efcd09111071b6c394603c311 (diff) | |
| download | chromium_src-ef402c3ba1147c39ab8b1f803b7bf8ebf1bbdefe.zip chromium_src-ef402c3ba1147c39ab8b1f803b7bf8ebf1bbdefe.tar.gz chromium_src-ef402c3ba1147c39ab8b1f803b7bf8ebf1bbdefe.tar.bz2 | |
[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
| -rw-r--r-- | chrome/browser/ui/cocoa/browser/avatar_menu_bubble_controller.mm | 5 |
1 files changed, 4 insertions, 1 deletions
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: |