Bring back support for 38x31 avatars.

chrome/browser/profiles/profile_avatar_icon_util.h

Index: chrome/browser/profiles/profile_avatar_icon_util.h
diff --git a/chrome/browser/profiles/profile_avatar_icon_util.h b/chrome/browser/profiles/profile_avatar_icon_util.h
index ecc3642692cd6dd8563f4fedab80607d5d024526..a82d7e747b893e2e7e995c31e7ee387e173ad413 100644
--- a/chrome/browser/profiles/profile_avatar_icon_util.h
+++ b/chrome/browser/profiles/profile_avatar_icon_util.h
@@ -17,6 +17,8 @@ namespace gfx {
 class Image;
 }
 
+class SkBitmap;
+
 namespace profiles {
 
 // Avatar access.
@@ -63,6 +65,7 @@ bool IsDefaultAvatarIconIndex(size_t index);
 // Checks if the given URL points to one of the default avatar icons. If it
 // is, returns true and its index through |icon_index|. If not, returns false.
 bool IsDefaultAvatarIconUrl(const std::string& icon_url, size_t *icon_index);
+
 // Returns a version of |image| of a specific size. 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.
@@ -85,6 +88,12 @@ gfx::Image GetAvatarIconForTitleBar(const gfx::Image& image,
                                     int dst_width,
                                     int dst_height);
 
+// Returns a bitmap with a couple of columns shaved off so it is more square,
+// so that when resized to a square aspect ratio it looks pretty.
+void GetAvatarIconAsSquare(const SkBitmap& source_bitmap,
+                           int scale_factor,
+                           SkBitmap& square_bitmap);

[sky, 2014/05/09 20:52:19]

Style guide says refs passed to functions must be const. If you need non-const,
then pass a pointer. That said, why not make this return a SkBitmap? SkBitmap
has copy semantics.

[Mike Lerman, 2014/05/12 15:35:47]

I had it like this originally, but then changed it to a parameter. Back to a
returned value.

+
 }  // namespace profiles
 
 #endif  // CHROME_BROWSER_PROFILES_PROFILE_AVATAR_ICON_UTIL_H_