Add in resize logic for the non-badged icons.

chrome/browser/ui/views/profiles/avatar_menu_bubble_view.cc

Index: chrome/browser/ui/views/profiles/avatar_menu_bubble_view.cc
diff --git a/chrome/browser/ui/views/profiles/avatar_menu_bubble_view.cc b/chrome/browser/ui/views/profiles/avatar_menu_bubble_view.cc
index aa3b695147aa85ca215b3e0b86f10069d11cea3d..dcf5c97cb3ba688c88be4a69296bed4fadc33e63 100644
--- a/chrome/browser/ui/views/profiles/avatar_menu_bubble_view.cc
+++ b/chrome/browser/ui/views/profiles/avatar_menu_bubble_view.cc
@@ -260,10 +260,14 @@ ProfileItemView::ProfileItemView(const AvatarMenu::Item& item,
 
   image_view_ = new ProfileImageView();
   gfx::ImageSkia profile_icon = *item_.icon.ToImageSkia();
-  if (item_.active || item_.signin_required)
+  if (item_.active || item_.signin_required) {
     image_view_->SetImage(GetBadgedIcon(profile_icon));
-  else
+  } else {
+    // The image may be too large are require resizing

[msw, 2014/04/28 17:56:52]

nit: s/are/and/; also add a period at the end.

[Mike Lerman, 2014/04/28 18:21:07]

Done.

+    image_view_->SetImageSize(

[msw, 2014/04/28 17:56:52]

Should this be set all the time, not just in this else case?

[Mike Lerman, 2014/04/28 18:21:07]

The other case, the GetBadgedIcon method, already does a scaling.

[msw, 2014/04/28 18:40:19]

Yeah, I saw that, but BadgeImageSource::ComputeSize seems to increase the
supplied sizes according to a badge overlap ratio; so their resulting composite
images could also be too large, afaict. Perhaps if the ImageView itself scales
the content, GetBadgedIcon could be simplified (but that's not strictly
necessary for this CL).

+        gfx::Size(profiles::kAvatarIconWidth, kItemHeight));
     image_view_->SetImage(profile_icon);
+  }
   AddChildView(image_view_);
 
   // Add a label to show the profile name.