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

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

Issue 2210763002: MacViews a11y: Sync VoiceOver cursor with keyboard focus. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 4 months 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 "content/browser/accessibility/browser_accessibility_manager_mac.h" 5 #include "content/browser/accessibility/browser_accessibility_manager_mac.h"
6 6
7 #include "base/logging.h"
7 #import "base/mac/mac_util.h" 8 #import "base/mac/mac_util.h"
8 #import "base/mac/sdk_forward_declarations.h" 9 #import "base/mac/sdk_forward_declarations.h"
9 #include "base/logging.h"
10 #include "base/strings/sys_string_conversions.h" 10 #include "base/strings/sys_string_conversions.h"
11 #include "base/strings/utf_string_conversions.h" 11 #include "base/strings/utf_string_conversions.h"
12 #import "content/browser/accessibility/browser_accessibility_cocoa.h" 12 #import "content/browser/accessibility/browser_accessibility_cocoa.h"
13 #import "content/browser/accessibility/browser_accessibility_mac.h" 13 #import "content/browser/accessibility/browser_accessibility_mac.h"
14 #include "content/common/accessibility_messages.h" 14 #include "content/common/accessibility_messages.h"
15 15
16 #import "tmp-traceclass.h"
17
16 namespace { 18 namespace {
17 19
18 // Declare undocumented accessibility constants and enums only present in 20 // Declare undocumented accessibility constants and enums only present in
19 // WebKit. 21 // WebKit.
20 22
21 enum AXTextStateChangeType { 23 enum AXTextStateChangeType {
22 AXTextStateChangeTypeUnknown, 24 AXTextStateChangeTypeUnknown,
23 AXTextStateChangeTypeEdit, 25 AXTextStateChangeTypeEdit,
24 AXTextStateChangeTypeSelectionMove, 26 AXTextStateChangeTypeSelectionMove,
25 AXTextStateChangeTypeSelectionExtend 27 AXTextStateChangeTypeSelectionExtend
(...skipping 153 matching lines...) Expand 10 before | Expand all | Expand 10 after
179 // |NSAccessibilityFocusedUIElementChangedNotification|, but this is 181 // |NSAccessibilityFocusedUIElementChangedNotification|, but this is
180 // handled elsewhere. 182 // handled elsewhere.
181 return; 183 return;
182 } 184 }
183 break; 185 break;
184 case ui::AX_EVENT_AUTOCORRECTION_OCCURED: 186 case ui::AX_EVENT_AUTOCORRECTION_OCCURED:
185 mac_notification = NSAccessibilityAutocorrectionOccurredNotification; 187 mac_notification = NSAccessibilityAutocorrectionOccurredNotification;
186 break; 188 break;
187 case ui::AX_EVENT_FOCUS: 189 case ui::AX_EVENT_FOCUS:
188 mac_notification = NSAccessibilityFocusedUIElementChangedNotification; 190 mac_notification = NSAccessibilityFocusedUIElementChangedNotification;
191 [Tracer startTraceBrowserAccessibility];
192 NSAccessibilityPostNotification(native_node, mac_notification);
193 [Tracer browserAccessibilityEndTrace];
194 return;
Patti Lor 2016/08/04 02:06:26 With tracing removed, this hunk is deleted.
189 break; 195 break;
190 case ui::AX_EVENT_LAYOUT_COMPLETE: 196 case ui::AX_EVENT_LAYOUT_COMPLETE:
191 mac_notification = NSAccessibilityLayoutCompleteNotification; 197 mac_notification = NSAccessibilityLayoutCompleteNotification;
192 break; 198 break;
193 case ui::AX_EVENT_LOAD_COMPLETE: 199 case ui::AX_EVENT_LOAD_COMPLETE:
194 mac_notification = NSAccessibilityLoadCompleteNotification; 200 mac_notification = NSAccessibilityLoadCompleteNotification;
195 break; 201 break;
196 case ui::AX_EVENT_INVALID_STATUS_CHANGED: 202 case ui::AX_EVENT_INVALID_STATUS_CHANGED:
197 mac_notification = NSAccessibilityInvalidStatusChangedNotification; 203 mac_notification = NSAccessibilityInvalidStatusChangedNotification;
198 break; 204 break;
(...skipping 266 matching lines...) Expand 10 before | Expand all | Expand 10 after
465 } 471 }
466 472
467 return @{ 473 return @{
468 NSAccessibilityTextStateChangeTypeKey : @(AXTextStateChangeTypeEdit), 474 NSAccessibilityTextStateChangeTypeKey : @(AXTextStateChangeTypeEdit),
469 NSAccessibilityTextChangeValues : changes, 475 NSAccessibilityTextChangeValues : changes,
470 NSAccessibilityTextChangeElement : native_node 476 NSAccessibilityTextChangeElement : native_node
471 }; 477 };
472 } 478 }
473 479
474 } // namespace content 480 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698