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

Side by Side Diff: content/browser/accessibility/browser_accessibility_cocoa.mm

Issue 1488893002: Stops using SYSTEM_STATE_INDETERMINATE as it makes radio buttons appear to have a mixed state. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fixed a few test expectations. Created 5 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include <execinfo.h> 5 #include <execinfo.h>
6 6
7 #import "content/browser/accessibility/browser_accessibility_cocoa.h" 7 #import "content/browser/accessibility/browser_accessibility_cocoa.h"
8 8
9 #include <map> 9 #include <map>
10 10
(...skipping 1315 matching lines...) Expand 10 before | Expand all | Expand 10 after
1326 } else if ([role isEqualToString:NSAccessibilityCheckBoxRole] || 1326 } else if ([role isEqualToString:NSAccessibilityCheckBoxRole] ||
1327 [role isEqualToString:NSAccessibilityRadioButtonRole]) { 1327 [role isEqualToString:NSAccessibilityRadioButtonRole]) {
1328 int value = 0; 1328 int value = 0;
1329 value = GetState( 1329 value = GetState(
1330 browserAccessibility_, ui::AX_STATE_CHECKED) ? 1 : 0; 1330 browserAccessibility_, ui::AX_STATE_CHECKED) ? 1 : 0;
1331 value = GetState( 1331 value = GetState(
1332 browserAccessibility_, ui::AX_STATE_SELECTED) ? 1332 browserAccessibility_, ui::AX_STATE_SELECTED) ?
1333 1 : 1333 1 :
1334 value; 1334 value;
1335 1335
1336 if (browserAccessibility_->GetBoolAttribute( 1336 if (browserAccessibility_->GetBoolAttribute(ui::AX_ATTR_STATE_MIXED)) {
1337 ui::AX_ATTR_BUTTON_MIXED)) {
1338 value = 2; 1337 value = 2;
1339 } 1338 }
1340 return [NSNumber numberWithInt:value]; 1339 return [NSNumber numberWithInt:value];
1341 } else if ([role isEqualToString:NSAccessibilityProgressIndicatorRole] || 1340 } else if ([role isEqualToString:NSAccessibilityProgressIndicatorRole] ||
1342 [role isEqualToString:NSAccessibilitySliderRole] || 1341 [role isEqualToString:NSAccessibilitySliderRole] ||
1343 [role isEqualToString:NSAccessibilityIncrementorRole] || 1342 [role isEqualToString:NSAccessibilityIncrementorRole] ||
1344 [role isEqualToString:NSAccessibilityScrollBarRole]) { 1343 [role isEqualToString:NSAccessibilityScrollBarRole]) {
1345 float floatValue; 1344 float floatValue;
1346 if (browserAccessibility_->GetFloatAttribute( 1345 if (browserAccessibility_->GetFloatAttribute(
1347 ui::AX_ATTR_VALUE_FOR_RANGE, &floatValue)) { 1346 ui::AX_ATTR_VALUE_FOR_RANGE, &floatValue)) {
(...skipping 757 matching lines...) Expand 10 before | Expand all | Expand 10 after
2105 if (!browserAccessibility_) 2104 if (!browserAccessibility_)
2106 return [super hash]; 2105 return [super hash];
2107 return browserAccessibility_->GetId(); 2106 return browserAccessibility_->GetId();
2108 } 2107 }
2109 2108
2110 - (BOOL)accessibilityShouldUseUniqueId { 2109 - (BOOL)accessibilityShouldUseUniqueId {
2111 return YES; 2110 return YES;
2112 } 2111 }
2113 2112
2114 @end 2113 @end
OLDNEW
« no previous file with comments | « chrome/common/extensions/api/automation.idl ('k') | content/browser/accessibility/browser_accessibility_win.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698