From 6bc200d5ff9f3f466144d55504fa8d94755ee8ef Mon Sep 17 00:00:00 2001 From: "msw@chromium.org" Date: Fri, 28 Sep 2012 21:01:51 +0000 Subject: Support obscured RenderTextWin passwords. Use asterisks instead of actual text when obscured on Win. Use GetLayoutText() and index conversion in RenderTextWin. Add UpdateObscuredText(), called on Set[Text|Obscured](). Shortcut word breaking when obscured like RenderTextLinux. Expand upon tests and enable on Windows; refactoring; etc. Fix and simplify TextfieldViewsModelTest.Clipboard. ( It incorrectly expected obscured text to support word-break cursor movement ) ( It was disabled on Linux Aura for long-fixed http://crbug.com/97845 ) BUG=97845,138222 TEST=Views examples and ftp auth --enable-views-textfield password textfields work, etc. R=xji@chromium.org,asvitkine@chromium.org TBR=sky@chromium.org Review URL: https://chromiumcodereview.appspot.com/10821079 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@159335 0039d316-1c4b-4281-b951-d872f2087c98 --- ui/views/controls/textfield/textfield.h | 3 --- 1 file changed, 3 deletions(-) (limited to 'ui/views/controls/textfield/textfield.h') diff --git a/ui/views/controls/textfield/textfield.h b/ui/views/controls/textfield/textfield.h index 94edd1b..ac273fe 100644 --- a/ui/views/controls/textfield/textfield.h +++ b/ui/views/controls/textfield/textfield.h @@ -62,9 +62,6 @@ class VIEWS_EXPORT Textfield : public View { // Gets/sets the STYLE_OBSCURED bit, controlling whether characters in this // Textfield are displayed as asterisks/bullets. - // TODO(bryeung): Currently SetObscured is only used in - // chrome/browser/chromeos/options/wifi_config_view.cc, which is being - // converted to WebUI. Please remove it when that happens. bool IsObscured() const; void SetObscured(bool obscured); -- cgit v1.1