[Win, Linux] Always display the avatar button text at the same size.

ui/gfx/font_list_unittest.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "ui/gfx/font_list.h"
 
 #include <algorithm>
 #include <string>
 #include <vector>
 
@@ skipping 268 matching lines @@
   FontList font_list_mix(fonts);
   // ascent of FontList == max(ascent of Fonts)
   EXPECT_EQ(std::max(font1.GetHeight() - font1.GetBaseline(),
                      font2.GetHeight() - font2.GetBaseline()),
             font_list_mix.GetHeight() - font_list_mix.GetBaseline());
   // descent of FontList == max(descent of Fonts)
   EXPECT_EQ(std::max(font1.GetBaseline(), font2.GetBaseline()),
             font_list_mix.GetBaseline());
 }
 
+TEST(FontListTest, Fonts_DeriveWithHeightUpperBound) {
+  std::vector<Font> fonts;
+
+  fonts.push_back(gfx::Font("Arial", 18).Derive(0, Font::NORMAL));

[msw, 2014/08/28 21:43:51]

Why do you need to derive the normal style for these?

[noms (inactive), 2014/08/29 17:10:29]

Done.

+  fonts.push_back(gfx::Font("Sans serif", 18).Derive(0, Font::NORMAL));
+  fonts.push_back(gfx::Font("Symbol", 18).Derive(0, Font::NORMAL));
+  FontList font_list = FontList(fonts);
+
+  FontList derived = font_list.DeriveWithHeightUpperBound(

[msw, 2014/08/28 21:43:51]

Maybe just do something like:

const int height = 14;
FontList derived = font_list.DeriveWithHeightUpperBound(height);
EXPECT_LE(height, derived.GetHeight());
EXPECT_LT(font_list.GetHeight(), derived.GetHeight());

Then do the same deriving another list with height of 22.

[noms (inactive), 2014/08/29 17:10:29]

I've done something along these lines, but without explicitly  hard-coding the
heights (turns out 22 wasn't large enough).

On 2014/08/28 21:43:51, msw wrote:

+      font_list.GetHeight() - 6);
+  const std::vector<Font>& derived_fonts = derived.GetFonts();
+
+  EXPECT_EQ(3U, derived_fonts.size());
+  EXPECT_EQ("Arial|13", FontToString(derived_fonts[0]));
+  EXPECT_EQ("Sans serif|13", FontToString(derived_fonts[1]));
+  EXPECT_EQ("Symbol|13", FontToString(derived_fonts[2]));
+}
+
 }  // namespace gfx