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

Unified Diff: ui/views/controls/textfield/textfield.h

Issue 2313123002: Harmony - set correct textfield placeholder text color (Closed)
Patch Set: Created 4 years, 3 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
« no previous file with comments | « no previous file | ui/views/controls/textfield/textfield.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ui/views/controls/textfield/textfield.h
diff --git a/ui/views/controls/textfield/textfield.h b/ui/views/controls/textfield/textfield.h
index 35f50991a76a6b67b43129b8258bad472a155b94..22ff37b869acab431c7710407f0566326106d93e 100644
--- a/ui/views/controls/textfield/textfield.h
+++ b/ui/views/controls/textfield/textfield.h
@@ -152,7 +152,6 @@ class VIEWS_EXPORT Textfield : public View,
}
base::string16 GetPlaceholderText() const;
- SkColor placeholder_text_color() const { return placeholder_text_color_; }
void set_placeholder_text_color(SkColor color) {
placeholder_text_color_ = color;
}
@@ -428,6 +427,7 @@ class VIEWS_EXPORT Textfield : public View,
base::string16 placeholder_text_;
// Placeholder text color.
+ // TODO(estade): remove this when Harmony/MD is default.
SkColor placeholder_text_color_;
// The accessible name of the text field.
« no previous file with comments | « no previous file | ui/views/controls/textfield/textfield.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698