Chromium Code Reviews| Index: content/browser/accessibility/browser_accessibility_cocoa.mm |
| diff --git a/content/browser/accessibility/browser_accessibility_cocoa.mm b/content/browser/accessibility/browser_accessibility_cocoa.mm |
| index 2ed21f48b0eee2977a82df1c0077dad7e8c60b6b..a4ec8733b768040951c5f1abecdf84d9acb37483 100644 |
| --- a/content/browser/accessibility/browser_accessibility_cocoa.mm |
| +++ b/content/browser/accessibility/browser_accessibility_cocoa.mm |
| @@ -1550,7 +1550,13 @@ NSDictionary* attributeToMethodNameMap = nil; |
| [self delegate]->AccessibilityDoDefaultAction( |
| browserAccessibility_->GetId()); |
| } else if ([action isEqualToString:NSAccessibilityShowMenuAction]) { |
| - [self delegate]->AccessibilityShowMenu(browserAccessibility_->GetId()); |
| + NSPoint objOrigin = [self origin]; |
| + NSSize size = [[self size] sizeValue]; |
| + gfx::Point origin = [self delegate]->AccessibilityOriginInScreen( |
| + gfx::Rect(objOrigin.x, objOrigin.y, size.width, size.height)); |
| + origin.set_x(origin.x() + size.width / 2); |
| + origin.set_y(origin.y() + size.width / 2); |
|
ncarter (slow)
2014/07/15 01:23:28
Should this be size.height instead of .width?
You
dmazzoni
2014/07/15 07:55:03
Good catch, thanks!
|
| + [self delegate]->AccessibilityShowMenu(origin); |
| } |
| } |
| @@ -1575,10 +1581,11 @@ NSDictionary* attributeToMethodNameMap = nil; |
| return; |
| if ([attribute isEqualToString:NSAccessibilityFocusedAttribute]) { |
| + BrowserAccessibilityManager* manager = browserAccessibility_->manager(); |
| NSNumber* focusedNumber = value; |
| BOOL focused = [focusedNumber intValue]; |
| if (focused) |
| - [self delegate]->AccessibilitySetFocus(browserAccessibility_->GetId()); |
| + manager->SetFocus(browserAccessibility_, true); |
| } |
| if ([attribute isEqualToString:NSAccessibilitySelectedTextRangeAttribute]) { |
| NSRange range = [(NSValue*)value rangeValue]; |