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

Side by Side Diff: chrome/browser/accessibility/browser_accessibility_manager.cc

Issue 4337001: Fix 2 accessibility events that were not being sent with the appro... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: Created 10 years, 1 month 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 | Annotate | Revision Log
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2010 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2010 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 "chrome/browser/accessibility/browser_accessibility_manager.h" 5 #include "chrome/browser/accessibility/browser_accessibility_manager.h"
6 6
7 #include "chrome/browser/accessibility/browser_accessibility.h" 7 #include "chrome/browser/accessibility/browser_accessibility.h"
8 8
9 using webkit_glue::WebAccessibility; 9 using webkit_glue::WebAccessibility;
10 10
(...skipping 158 matching lines...) Expand 10 before | Expand all | Expand 10 after
169 169
170 void BrowserAccessibilityManager::OnAccessibilityObjectValueChange( 170 void BrowserAccessibilityManager::OnAccessibilityObjectValueChange(
171 const WebAccessibility& acc_obj) { 171 const WebAccessibility& acc_obj) {
172 BrowserAccessibility* new_browser_acc = UpdateTree(acc_obj); 172 BrowserAccessibility* new_browser_acc = UpdateTree(acc_obj);
173 if (!new_browser_acc) 173 if (!new_browser_acc)
174 return; 174 return;
175 175
176 NotifyAccessibilityEvent( 176 NotifyAccessibilityEvent(
177 ViewHostMsg_AccessibilityNotification_Params:: 177 ViewHostMsg_AccessibilityNotification_Params::
178 NOTIFICATION_TYPE_VALUE_CHANGED, 178 NOTIFICATION_TYPE_VALUE_CHANGED,
179 root_); 179 new_browser_acc);
180 } 180 }
181 181
182 void BrowserAccessibilityManager::OnAccessibilityObjectTextChange( 182 void BrowserAccessibilityManager::OnAccessibilityObjectTextChange(
183 const WebAccessibility& acc_obj) { 183 const WebAccessibility& acc_obj) {
184 BrowserAccessibility* new_browser_acc = UpdateTree(acc_obj); 184 BrowserAccessibility* new_browser_acc = UpdateTree(acc_obj);
185 if (!new_browser_acc) 185 if (!new_browser_acc)
186 return; 186 return;
187 187
188 NotifyAccessibilityEvent( 188 NotifyAccessibilityEvent(
189 ViewHostMsg_AccessibilityNotification_Params:: 189 ViewHostMsg_AccessibilityNotification_Params::
190 NOTIFICATION_TYPE_SELECTED_TEXT_CHANGED, 190 NOTIFICATION_TYPE_SELECTED_TEXT_CHANGED,
191 root_); 191 new_browser_acc);
192 } 192 }
193 193
194 void BrowserAccessibilityManager::GotFocus() { 194 void BrowserAccessibilityManager::GotFocus() {
195 // TODO(ctguil): Remove when tree update logic handles focus changes. 195 // TODO(ctguil): Remove when tree update logic handles focus changes.
196 if (!focus_) 196 if (!focus_)
197 return; 197 return;
198 198
199 NotifyAccessibilityEvent( 199 NotifyAccessibilityEvent(
200 ViewHostMsg_AccessibilityNotification_Params:: 200 ViewHostMsg_AccessibilityNotification_Params::
201 NOTIFICATION_TYPE_FOCUS_CHANGED, 201 NOTIFICATION_TYPE_FOCUS_CHANGED,
(...skipping 107 matching lines...) Expand 10 before | Expand all | Expand 10 after
309 if ((src.state >> WebAccessibility::STATE_FOCUSED) & 1) 309 if ((src.state >> WebAccessibility::STATE_FOCUSED) & 1)
310 focus_ = instance; 310 focus_ = instance;
311 for (int i = 0; i < static_cast<int>(src.children.size()); ++i) { 311 for (int i = 0; i < static_cast<int>(src.children.size()); ++i) {
312 BrowserAccessibility* child = CreateAccessibilityTree( 312 BrowserAccessibility* child = CreateAccessibilityTree(
313 instance, GetNextChildID(), src.children[i], i); 313 instance, GetNextChildID(), src.children[i], i);
314 instance->AddChild(child); 314 instance->AddChild(child);
315 } 315 }
316 316
317 return instance; 317 return instance;
318 } 318 }
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698