Remove views::Label::SetDisabledColor(). Replace with typography colors.

chrome/browser/ui/views/profiles/profile_chooser_view.cc

Index: chrome/browser/ui/views/profiles/profile_chooser_view.cc
diff --git a/chrome/browser/ui/views/profiles/profile_chooser_view.cc b/chrome/browser/ui/views/profiles/profile_chooser_view.cc
index e7535c427a7e64acecb5a002f9a28dca7c355654..1d9da7c649b1cc971040f63adb7b328fac2f4f16 100644
--- a/chrome/browser/ui/views/profiles/profile_chooser_view.cc
+++ b/chrome/browser/ui/views/profiles/profile_chooser_view.cc
@@ -261,8 +261,7 @@ class BackgroundColorHoverButton : public views::LabelButton {
       title_->SetEnabledColor(text_color);
 
     if (subtitle_) {
-      DCHECK(!subtitle_->enabled());
-      subtitle_->SetDisabledColor(GetNativeTheme()->GetSystemColor(
+      subtitle_->SetEnabledColor(GetNativeTheme()->GetSystemColor(
           is_selected
               ? ui::NativeTheme::kColorId_DisabledMenuItemForegroundColor
               : ui::NativeTheme::kColorId_LabelDisabledColor));

[Peter Kasting, 2017/06/01 04:55:58]

Is there a TODO here to use the typography provider to set the colors?  Seems
like this isn't really "disabled" any more than the other cases you're fixing.

[tapted, 2017/06/01 11:22:18]

Heh, yeah I'd considered this. But if the samples on the typography spec at
https://bugs.chromium.org/p/chromium/issues/attachment?aid=286827&inline=1 can
be trusted then it's likely that the email should be primary, and the text above
it should be the "hint". (In which case the TODO might belong closer to
`email_label`). So it may be more involved than just switching these to use
GetFont, but we won't know until this UI has a mock.

Left a note.

@@ -1171,7 +1170,6 @@ views::View* ProfileChooserView::CreateCurrentProfileView(
       current_profile_card->set_subtitle(email_label);
       email_label->SetAutoColorReadabilityEnabled(false);
       email_label->SetElideBehavior(gfx::ELIDE_EMAIL);
-      email_label->SetEnabled(false);
       email_label->SetHorizontalAlignment(gfx::ALIGN_LEFT);
       grid_layout->StartRow(1, 0);
       // Skip first column for the profile icon.