Commit b0ad439a authored by noms@chromium.org's avatar noms@chromium.org

Profile chooser bubble overlaps avatar button and shouldn't.

There was a change in BubbleBorder (https://codereview.chromium.org/53703002) that removed the 10px insets on the border, so we no longer need to compensate for them.

BUG=NONE
TEST=With --new-profile-management enabled, click on the avatar button.
The bubble should show up just below it.

Review URL: https://codereview.chromium.org/57513004

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@232943 0039d316-1c4b-4281-b951-d872f2087c98
parent 1c175a5c
...@@ -51,7 +51,6 @@ const int kLargeImageSide = 64; ...@@ -51,7 +51,6 @@ const int kLargeImageSide = 64;
const int kSmallImageSide = 32; const int kSmallImageSide = 32;
const int kMinMenuWidth = 250; const int kMinMenuWidth = 250;
const int kButtonHeight = 29; const int kButtonHeight = 29;
const int kArrowHeight = 10;
// Current profile avatar image. // Current profile avatar image.
views::View* CreateProfileImageView(const gfx::Image& icon) { views::View* CreateProfileImageView(const gfx::Image& icon) {
...@@ -225,8 +224,6 @@ ProfileChooserView::ProfileChooserView(views::View* anchor_view, ...@@ -225,8 +224,6 @@ ProfileChooserView::ProfileChooserView(views::View* anchor_view,
browser_(browser) { browser_(browser) {
// Reset the default margins inherited from the BubbleDelegateView. // Reset the default margins inherited from the BubbleDelegateView.
set_margins(gfx::Insets()); set_margins(gfx::Insets());
// Compensate for built-in vertical padding in the anchor view's image.
set_anchor_view_insets(gfx::Insets(kArrowHeight, 0, kArrowHeight, 0));
ResetLinksAndButtons(); ResetLinksAndButtons();
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment