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/scrollbar | |
| 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/scrollbar')
| -rw-r--r-- | views/controls/scrollbar/scroll_bar.cc | 7 | ||||
| -rw-r--r-- | views/controls/scrollbar/scroll_bar.h | 2 |
2 files changed, 3 insertions, 6 deletions
diff --git a/views/controls/scrollbar/scroll_bar.cc b/views/controls/scrollbar/scroll_bar.cc index 9a7a9e3..94360bb 100644 --- a/views/controls/scrollbar/scroll_bar.cc +++ b/views/controls/scrollbar/scroll_bar.cc @@ -22,11 +22,8 @@ ScrollBar::ScrollBar(bool is_horiz) : is_horiz_(is_horiz), ScrollBar::~ScrollBar() { } -bool ScrollBar::GetAccessibleRole(AccessibilityTypes::Role* role) { - DCHECK(role); - - *role = AccessibilityTypes::ROLE_SCROLLBAR; - return true; +AccessibilityTypes::Role ScrollBar::GetAccessibleRole() { + return AccessibilityTypes::ROLE_SCROLLBAR; } bool ScrollBar::IsHorizontal() const { diff --git a/views/controls/scrollbar/scroll_bar.h b/views/controls/scrollbar/scroll_bar.h index cd23a0f..3a26b04 100644 --- a/views/controls/scrollbar/scroll_bar.h +++ b/views/controls/scrollbar/scroll_bar.h @@ -60,7 +60,7 @@ class ScrollBar : public View { virtual ~ScrollBar(); // Overridden from View: - virtual bool GetAccessibleRole(AccessibilityTypes::Role* role); + virtual AccessibilityTypes::Role GetAccessibleRole(); // Return whether this scrollbar is horizontal bool IsHorizontal() const; |