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/checkbox.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/checkbox.cc')
| -rw-r--r-- | views/controls/button/checkbox.cc | 14 |
1 files changed, 4 insertions, 10 deletions
diff --git a/views/controls/button/checkbox.cc b/views/controls/button/checkbox.cc index 404a0ea..bae0fcc 100644 --- a/views/controls/button/checkbox.cc +++ b/views/controls/button/checkbox.cc @@ -167,18 +167,12 @@ void Checkbox::WillLoseFocus() { label_->set_paint_as_focused(false); } -bool Checkbox::GetAccessibleRole(AccessibilityTypes::Role* role) { - DCHECK(role); - - *role = AccessibilityTypes::ROLE_CHECKBUTTON; - return true; +AccessibilityTypes::Role Checkbox::GetAccessibleRole() { + return AccessibilityTypes::ROLE_CHECKBUTTON; } -bool Checkbox::GetAccessibleState(AccessibilityTypes::State* state) { - DCHECK(state); - - *state = checked() ? AccessibilityTypes::STATE_CHECKED : 0; - return true; +AccessibilityTypes::State Checkbox::GetAccessibleState() { + return checked() ? AccessibilityTypes::STATE_CHECKED : 0; } std::string Checkbox::GetClassName() const { |