Renames gfx::FontList::Derive* family.

ash/sticky_keys/sticky_keys_overlay.cc

Index: ash/sticky_keys/sticky_keys_overlay.cc
diff --git a/ash/sticky_keys/sticky_keys_overlay.cc b/ash/sticky_keys/sticky_keys_overlay.cc
index 19e5acbcb517793488e9f287c269ba70bb68c524..cc2462fdbfe8519e52e8512568365e5b84b6f23a 100644
--- a/ash/sticky_keys/sticky_keys_overlay.cc
+++ b/ash/sticky_keys/sticky_keys_overlay.cc
@@ -11,6 +11,7 @@
 #include "base/strings/utf_string_conversions.h"
 #include "grit/ash_strings.h"
 #include "ui/base/l10n/l10n_util.h"
+#include "ui/base/resource/resource_bundle.h"
 #include "ui/compositor/scoped_layer_animation_settings.h"
 #include "ui/gfx/canvas.h"
 #include "ui/gfx/font_list.h"
@@ -71,8 +72,7 @@ StickyKeyOverlayLabel::StickyKeyOverlayLabel(const std::string& key_name)
     : state_(STICKY_KEY_STATE_DISABLED) {
   SetText(base::UTF8ToUTF16(key_name));
   SetHorizontalAlignment(gfx::ALIGN_LEFT);
-  SetFontList(
-      font_list().DeriveFontListWithSize(18));
+  SetFontList(font_list().DeriveWithSizeDelta(4));

[Alexei Svitkine (slow), 2014/01/23 18:47:54]

Can you comment where the 4 comes from? Are you sure this doesn't result in a
change of behaviour? (i.e. was font list size always 14 on all platforms
before)?

[Yuki, 2014/01/24 10:23:20]

The default font size depends on platforms and user's settings.  But it's about
14px in most cases, I think.  I wasn't sure if we should use MediumFont (+3px)
or not.  I'm now inclined to MediumFont.

For the reason why +4px, I cannot tell you the reason because I don't know why
it was 18px.  Probably there's no special reason.

If you're okay, can I specify MediumFont here?

My point is that we shouldn't specify the absolute font size as much as
possible.
For this case, the original author surely meant this label to be larger than the
default text, but it's possible that this label could be smaller than the
default text if the default font is enough large.

[Alexei Svitkine (slow), 2014/01/24 15:53:41]

I agree with the motivation and also agree that if we could use MediumFont, it
would be better. However, I'm worried that making these changes blindly will
regress some UI. So I'd rather you double check this (i.e. post screenshots of
before and after for this view) to make sure it still looks ok after the change.

[Yuki, 2014/01/27 07:03:49]

Here are screenshots (on Pixel).  I took screenshots of the full screen so you
can get a feeling compared to other text.  You can see the sticky keys'
indicator at top-left corner.

Original 18px font:
https://docs.google.com/a/chromium.org/file/d/0B3Ozb9XcozcMekt2djE0bnlnMkU/view

MediumFont:
https://docs.google.com/a/chromium.org/file/d/0B3Ozb9XcozcMRXcxZ2RhSlIwOUE/view

LargeFont:
https://docs.google.com/a/chromium.org/file/d/0B3Ozb9XcozcMcDUtYlFSVHZPUXM/view

After taking screenshots, I think the LargeFont is best.  Let me use LargeFont
here, if you guys are okay.

[Alexei Svitkine (slow), 2014/01/27 16:00:38]

Thanks for checking!

Switching to LargeFont indeed seems fine to me here. (Though to be certain, you
might want to double check with the feature author.)

   SetAutoColorReadabilityEnabled(false);
   SetFocusable(false);
   SetEnabledColor(SkColorSetARGB(0x80, 0xFF, 0xFF, 0xFF));
@@ -102,7 +102,7 @@ void StickyKeyOverlayLabel::SetKeyState(StickyKeyState state) {
 
   SetEnabledColor(label_color);
   SetDisabledColor(label_color);
-  SetFontList(font_list().DeriveFontListWithSizeDeltaAndStyle(0, style));
+  SetFontList(font_list().Derive(0, style));
 }
 
 void StickyKeyOverlayLabel::PaintText(gfx::Canvas* canvas,