Clean-up: Retires obsolete methods in views::Label and views::TextButtonBase.

ui/views/controls/label.cc

Index: ui/views/controls/label.cc
diff --git a/ui/views/controls/label.cc b/ui/views/controls/label.cc
index dcd33f6b8e69da9b332ef2acdb60e3bc453d4e2d..0a7362d41d6c15a9d016915967e3a25ef495971c 100644
--- a/ui/views/controls/label.cc
+++ b/ui/views/controls/label.cc
@@ -29,11 +29,6 @@ namespace {
 
 const int kCachedSizeLimit = 10;
 
-gfx::FontList GetDefaultFontList() {
-  return ui::ResourceBundle::GetSharedInstance().GetFontList(
-      ui::ResourceBundle::BaseFont);
-}
-
 }  // namespace
 
 namespace views {
@@ -43,21 +38,17 @@ const char Label::kViewClassName[] = "Label";
 const int Label::kFocusBorderPadding = 1;
 
 Label::Label() {
-  Init(base::string16(), GetDefaultFontList());
+  Init(base::string16(), gfx::FontList());

[sadrul, 2014/01/06 18:41:04]

Is this gfx::FontList() equivalent to the default font-list from the
ResourceBundle?

[Yuki, 2014/01/07 02:28:36]

Yes, it's now equivalent.

[sadrul, 2014/01/07 02:35:32]

Considering crbug.com/330975, if we get the FontList from the ResourceBundle,
then the FontList here will inherit the cached FontList attributes (e.g.
height), but if we create a new instance of FontList (as you do with this
patch), we lose that, and it would have to recompute that information. Is that
correct? Wouldn't that cause more perf issues?

[Yuki, 2014/01/07 02:41:32]

That's a good point.  Let me address the issue first.

[Yuki, 2014/01/07 13:00:53]

Just FYI, I've sent out a CL: http://crrev.com/124693003

 }
 
 Label::Label(const base::string16& text) {
-  Init(text, GetDefaultFontList());
+  Init(text, gfx::FontList());
 }
 
 Label::Label(const base::string16& text, const gfx::FontList& font_list) {
   Init(text, font_list);
 }
 
-Label::Label(const base::string16& text, const gfx::Font& font) {
-  Init(text, gfx::FontList(font));
-}
-
 Label::~Label() {
 }
 
@@ -68,14 +59,6 @@ void Label::SetFontList(const gfx::FontList& font_list) {
   SchedulePaint();
 }
 
-const gfx::Font& Label::font() const {
-  return font_list_.GetPrimaryFont();
-}
-
-void Label::SetFont(const gfx::Font& font) {
-  SetFontList(gfx::FontList(font));
-}
-
 void Label::SetText(const base::string16& text) {
   if (text == text_)
     return;