[Mac] The GAIA profile photo should be resized in the Users controller menu

chrome/browser/profiles/profile_info_util.h

Index: chrome/browser/profiles/profile_info_util.h
diff --git a/chrome/browser/profiles/profile_info_util.h b/chrome/browser/profiles/profile_info_util.h
index ce1b78e4eed1d5709fdfbc5ba6cb0a9cc940a7df..21d4b485c9e71ae5ad29e21d77a7c87e334a4057 100644
--- a/chrome/browser/profiles/profile_info_util.h
+++ b/chrome/browser/profiles/profile_info_util.h
@@ -13,6 +13,10 @@ extern const int kAvatarIconWidth;
 extern const int kAvatarIconHeight;
 extern const int kAvatarIconPadding;
 
+// Return whether or not |image| needs to be resized to a suitable size
+// as given by kAvatarIconWidth and kAvatarIconHeight.
+bool AvatarIconNeedsResizing(const gfx::Image& image);

[Alexei Svitkine (slow), 2014/01/29 20:23:56]

I am guessing this is here because you're going to re-use this logic from other
UI code?

If so, why not make the shared function actually return a correctly sized image?
(i.e. either the image or one that's correctly sized).

[noms (inactive), 2014/01/29 20:31:29]

I don't really know where I would reuse it. There's not a lot of consistency in
how the images are resized (with or without a border), so I couldn't make a
decision as to which would be needed in the "general" case. I'm going to punt
this api enthusiasm and just do the resize-needed check in the controller. :)

On 2014/01/29 20:23:56, Alexei Svitkine wrote:

+
 // Returns a version of |image| of a specific size and with a grey border.
 // Note that no checks are done on the width/height so make sure they're
 // reasonable values; in the range of 16-256 is probably best.