Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(419)

Issue 1102006: Add Accessibility name for native buttons. (Closed)

Created:
10 years, 9 months ago by Mohamed Mansour
Modified:
9 years, 6 months ago
Reviewers:
Chris Guillory
CC:
chromium-reviews, Jonas Klink (Google)
Visibility:
Public.

Description

Add Accessibility name for native buttons. Remove the accessible name from Checkbox since it inherits from NativeButton. Dialogs with autogenerated buttons now have MSAA name attached to it, as well as all the checkboxes. BUG=38987 TEST=Ran it in AccExplorer Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=42401

Patch Set 1 #

Patch Set 2 : Add Accessibility for Native Buttons #

Unified diffs Side-by-side diffs Delta from patch set Stats (+3 lines, -7 lines) Patch
M views/controls/button/checkbox.h View 1 chunk +0 lines, -1 line 0 comments Download
M views/controls/button/checkbox.cc View 1 chunk +0 lines, -6 lines 0 comments Download
M views/controls/button/native_button.cc View 1 chunk +3 lines, -0 lines 0 comments Download

Messages

Total messages: 8 (0 generated)
Mohamed Mansour
One line review, thanks Chris!
10 years, 9 months ago (2010-03-23 00:21:51 UTC) #1
Chris Guillory
Why not put this in the NativeButton constructor?
10 years, 9 months ago (2010-03-23 01:42:26 UTC) #2
Mohamed Mansour
On 2010/03/23 01:42:26, Chris Guillory wrote: > Why not put this in the NativeButton constructor? ...
10 years, 9 months ago (2010-03-23 01:47:55 UTC) #3
Mohamed Mansour
On 2010/03/23 01:47:55, Mohamed Mansour wrote: > On 2010/03/23 01:42:26, Chris Guillory wrote: > > ...
10 years, 9 months ago (2010-03-23 01:48:37 UTC) #4
Mohamed Mansour
On 2010/03/23 01:48:37, Mohamed Mansour wrote: > Can we assume that every buttons's accessible name ...
10 years, 9 months ago (2010-03-23 01:52:51 UTC) #5
Chris Guillory
> Now if we use NativeButton, that means we can remove the overridden > accessibility ...
10 years, 9 months ago (2010-03-23 02:08:40 UTC) #6
Mohamed Mansour
Please review. I removed the accessible name from Checkbox since it inherits everything from NativeButton. ...
10 years, 9 months ago (2010-03-23 03:41:04 UTC) #7
Chris Guillory
10 years, 9 months ago (2010-03-23 17:08:27 UTC) #8
LGTM

Powered by Google App Engine
This is Rietveld 408576698