diff options
author | xji@chromium.org <xji@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-05-06 15:25:15 +0000 |
---|---|---|
committer | xji@chromium.org <xji@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-05-06 15:25:15 +0000 |
commit | 114ed79bd30bcab7916d68ddcf3e585a3810d3ed (patch) | |
tree | 987a3cc3e8d3c569374c13e4e5145d975ac401e6 /chrome | |
parent | 902cdf777ff20af9fc5ecc2877781dcb9571df69 (diff) | |
download | chromium_src-114ed79bd30bcab7916d68ddcf3e585a3810d3ed.zip chromium_src-114ed79bd30bcab7916d68ddcf3e585a3810d3ed.tar.gz chromium_src-114ed79bd30bcab7916d68ddcf3e585a3810d3ed.tar.bz2 |
This CL fixes issue 11479 - RTL Regression: Ellipsis showing at the right side of the button on "Google Chrome Options" dialog.
(http://crbug.com/11479)
This is another regression caused by code refactoring.
Instead of assign data member "label_" directly as the input parameter in constructor, calling SetLabel() which takes care of label layout in RTL locale.
BUG=http://crbug.com/11479)
Review URL: http://codereview.chromium.org/113010
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@15410 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome')
-rw-r--r-- | chrome/views/controls/button/native_button.cc | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/chrome/views/controls/button/native_button.cc b/chrome/views/controls/button/native_button.cc index 511f45b..0e6646f 100644 --- a/chrome/views/controls/button/native_button.cc +++ b/chrome/views/controls/button/native_button.cc @@ -32,10 +32,10 @@ NativeButton::NativeButton(ButtonListener* listener) NativeButton::NativeButton(ButtonListener* listener, const std::wstring& label) : Button(listener), native_wrapper_(NULL), - label_(label), is_default_(false), ignore_minimum_size_(false), minimum_size_(50, 14) { + SetLabel(label); // SetLabel takes care of label layout in RTL UI. // The min size in DLUs comes from // http://msdn.microsoft.com/library/default.asp?url=/library/en-us/dnwue/html/ch14e.asp InitBorder(); |