diff options
| author | ctguil@chromium.org <ctguil@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-09-28 19:30:23 +0000 |
|---|---|---|
| committer | ctguil@chromium.org <ctguil@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-09-28 19:30:23 +0000 |
| commit | 4c671e16908c0f60f38aa26051b89d399b5eb0f0 (patch) | |
| tree | 36801dcd79034457409f700a0bf42d0f1304c8e1 /views/controls/button/button.cc | |
| parent | 485149b279565e436b1c2a6157086b81b4e9ede7 (diff) | |
| download | chromium_src-4c671e16908c0f60f38aa26051b89d399b5eb0f0.zip chromium_src-4c671e16908c0f60f38aa26051b89d399b5eb0f0.tar.gz chromium_src-4c671e16908c0f60f38aa26051b89d399b5eb0f0.tar.bz2 | |
Make accessible getters return values instead of accepting an output parameter. GetAccessibleName still needs to be converted.
BUG=none
TEST=trybots
Review URL: http://codereview.chromium.org/3416033
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@60828 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'views/controls/button/button.cc')
| -rw-r--r-- | views/controls/button/button.cc | 13 |
1 files changed, 4 insertions, 9 deletions
diff --git a/views/controls/button/button.cc b/views/controls/button/button.cc index 52d5747..f35b178 100644 --- a/views/controls/button/button.cc +++ b/views/controls/button/button.cc @@ -32,17 +32,12 @@ bool Button::GetTooltipText(const gfx::Point& p, std::wstring* tooltip) { return true; } -bool Button::GetAccessibleKeyboardShortcut(std::wstring* shortcut) { - if (!accessible_shortcut_.empty()) { - *shortcut = accessible_shortcut_; - return true; - } - return false; +std::wstring Button::GetAccessibleKeyboardShortcut() { + return accessible_shortcut_; } -bool Button::GetAccessibleRole(AccessibilityTypes::Role* role) { - *role = AccessibilityTypes::ROLE_PUSHBUTTON; - return true; +AccessibilityTypes::Role Button::GetAccessibleRole() { + return AccessibilityTypes::ROLE_PUSHBUTTON; } //////////////////////////////////////////////////////////////////////////////// |