Make views::Label and views::Link auto-color themselves to be readable over their background colo...

chrome/browser/chromeos/login/new_user_view.cc

 // Copyright (c) 2011 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 "chrome/browser/chromeos/login/new_user_view.h"
 
 #include <signal.h>
 #include <sys/types.h>
 
 #include <algorithm>
@@ skipping 13 matching lines @@
 #include "chrome/browser/chromeos/user_cros_settings_provider.h"
 #include "chrome/browser/chromeos/views/copy_background.h"
 #include "chrome/browser/prefs/pref_service.h"
 #include "chrome/common/pref_names.h"
 #include "grit/chromium_strings.h"
 #include "grit/generated_resources.h"
 #include "grit/ui_resources.h"
 #include "ui/base/keycodes/keyboard_codes.h"
 #include "ui/base/l10n/l10n_util.h"
 #include "ui/base/resource/resource_bundle.h"
+#include "ui/gfx/color_utils.h"
 #include "ui/gfx/font.h"
 #include "views/controls/button/menu_button.h"
 #include "views/controls/button/text_button.h"
 #include "views/controls/label.h"
 #include "views/controls/link.h"
 #include "views/controls/textfield/textfield.h"
 #include "views/controls/throbber.h"
 
 using views::View;
 
@@ skipping 90 matching lines @@
     need_create_account_ = true;
     if (need_guest_link)
       need_guest_link_ = true;
   }
 }
 
 NewUserView::~NewUserView() {
 }
 
 void NewUserView::Init() {
-  if (need_border_) {
-    // Use rounded rect background.
-    set_border(CreateWizardBorder(&BorderDefinition::kUserBorder));
-    views::Painter* painter = CreateWizardPainter(
-        &BorderDefinition::kUserBorder);
-    set_background(views::Background::CreateBackgroundPainter(true, painter));
-  }
-
   title_label_ = new views::Label();
   title_label_->SetHorizontalAlignment(views::Label::ALIGN_LEFT);
   title_label_->SetMultiLine(true);
   AddChildView(title_label_);
 
   title_hint_label_ = new views::Label();
   title_hint_label_->SetHorizontalAlignment(views::Label::ALIGN_LEFT);
-  title_hint_label_->SetColor(SK_ColorGRAY);
+  title_hint_label_->SetEnabledColor(SK_ColorGRAY);
   title_hint_label_->SetMultiLine(true);
   AddChildView(title_hint_label_);
 
   splitter_up1_ = CreateSplitter(kSplitterUp1Color);
   splitter_up2_ = CreateSplitter(kSplitterUp2Color);
   splitter_down1_ = CreateSplitter(kSplitterDown1Color);
   splitter_down2_ = CreateSplitter(kSplitterDown2Color);
 
   username_field_ = new UsernameField(this);
   username_field_->set_background(new CopyBackground(this));
@@ skipping 11 matching lines @@
 
   AddChildView(sign_in_button_);
   if (need_guest_link_) {
     InitLink(&guest_link_);
   }
   if (need_create_account_) {
     InitLink(&create_account_link_);
   }
   AddChildView(languages_menubutton_);
 
+  if (need_border_) {
+    // Use rounded rect background.
+    set_border(CreateWizardBorder(&BorderDefinition::kUserBorder));
+    views::Painter* painter = CreateWizardPainter(
+        &BorderDefinition::kUserBorder);
+    set_background(views::Background::CreateBackgroundPainter(true, painter));
+    SkColor background_color = color_utils::AlphaBlend(
+        BorderDefinition::kUserBorder.top_color,
+        BorderDefinition::kUserBorder.bottom_color, 128);
+    title_label_->SetBackgroundColor(background_color);
+    title_hint_label_->SetBackgroundColor(background_color);
+    if (guest_link_)
+      guest_link_->SetBackgroundColor(background_color);
+    if (create_account_link_)
+      guest_link_->SetBackgroundColor(background_color);
+  } else {
+    // In theory, we should actually set the real background color for all the
+    // labels and links whether or not |need_border_| is set, but since without
+    // it the background color is non-obvious, just disable auto color
+    // readability.
+    title_label_->SetAutoColorReadabilityEnabled(false);
+    title_hint_label_->SetAutoColorReadabilityEnabled(false);
+    if (guest_link_)
+      guest_link_->SetAutoColorReadabilityEnabled(false);
+    if (create_account_link_)
+      create_account_link_->SetAutoColorReadabilityEnabled(false);
+  }
+
   // Set up accelerators.
   AddAccelerator(accel_focus_user_);
   AddAccelerator(accel_focus_pass_);
   AddAccelerator(accel_enterprise_enrollment_);
   AddAccelerator(accel_login_off_the_record_);
   AddAccelerator(accel_toggle_accessibility_);
 
   OnLocaleChanged();
 
   // Controller to handle events from textfields
@@ skipping 352 matching lines @@
   if (need_guest_link_) {
     guest_link_->SetEnabled(enabled);
   }
   if (need_create_account_) {
     create_account_link_->SetEnabled(enabled);
   }
   UpdateSignInButtonState();
 }
 
 bool NewUserView::NavigateAway() {
-  if (username_field_->text().empty() &&
-      password_field_->text().empty()) {
-    delegate_->NavigateAway();
-    return true;
-  } else {
+  if (!username_field_->text().empty() ||
+      !password_field_->text().empty())
     return false;
-  }
+  delegate_->NavigateAway();
+  return true;
 }
 
 void NewUserView::InitLink(views::Link** link) {
   *link = new views::Link(string16());
   (*link)->set_listener(this);
-  (*link)->SetNormalColor(login::kLinkColor);
-  (*link)->SetHighlightedColor(login::kLinkColor);
+  (*link)->SetEnabledColor(login::kLinkColor);
+  (*link)->SetPressedColor(login::kLinkColor);
   AddChildView(*link);
 }
 
 }  // namespace chromeos