Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1496)

Unified Diff: chrome/browser/ui/views/autofill/autofill_dialog_views.cc

Issue 19531006: requestAutocomplete: fix some text colors (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: +PreferredSizeChanged override Created 7 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/views/autofill/autofill_dialog_views.cc
diff --git a/chrome/browser/ui/views/autofill/autofill_dialog_views.cc b/chrome/browser/ui/views/autofill/autofill_dialog_views.cc
index a3523a64b6d3cce3809e2a2de617c2a9a847a29a..f253c5f0472ef86e178928cd03f787a9709d4246 100644
--- a/chrome/browser/ui/views/autofill/autofill_dialog_views.cc
+++ b/chrome/browser/ui/views/autofill/autofill_dialog_views.cc
@@ -109,6 +109,11 @@ const int kOverlayTextPadding = 20;
// Spacing between lines of text in the overlay view.
const int kOverlayTextInterlineSpacing = 10;
+// A dimmer text color used in various parts of the dialog. TODO(estade): should
+// this be part of NativeTheme? Currently the value is duplicated in several
+// places.
+const SkColor kGreyTextColor = SkColorSetRGB(102, 102, 102);
+
const char kDecoratedTextfieldClassName[] = "autofill/DecoratedTextfield";
const char kNotificationAreaClassName[] = "autofill/NotificationArea";
const char kOverlayViewClassName[] = "autofill/OverlayView";
@@ -1490,7 +1495,9 @@ views::View* AutofillDialogViews::CreateFootnoteView() {
views::Background::CreateSolidBackground(kShadingColor));
legal_document_view_ = new views::StyledLabel(string16(), this);
- legal_document_view_->SetDisplayedOnBackgroundColor(kShadingColor);
+ views::StyledLabel::RangeStyleInfo default_style;
+ default_style.color = kGreyTextColor;
+ legal_document_view_->SetDefaultStyle(default_style);
footnote_view_->AddChildView(legal_document_view_);
footnote_view_->SetVisible(false);

Powered by Google App Engine
This is Rietveld 408576698