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

Side by Side Diff: content/renderer/accessibility/renderer_accessibility.cc

Issue 1457683009: Complete live region support in ChromeVox Next. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 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
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/renderer/accessibility/renderer_accessibility.h" 5 #include "content/renderer/accessibility/renderer_accessibility.h"
6 6
7 #include <queue> 7 #include <queue>
8 8
9 #include "base/bind.h" 9 #include "base/bind.h"
10 #include "base/location.h" 10 #include "base/location.h"
(...skipping 259 matching lines...) Expand 10 before | Expand all | Expand 10 after
270 serializer_.SerializeChanges(obj, &event_msg.update); 270 serializer_.SerializeChanges(obj, &event_msg.update);
271 event_msgs.push_back(event_msg); 271 event_msgs.push_back(event_msg);
272 272
273 // For each node in the update, set the location in our map from 273 // For each node in the update, set the location in our map from
274 // ids to locations. 274 // ids to locations.
275 for (size_t i = 0; i < event_msg.update.nodes.size(); ++i) { 275 for (size_t i = 0; i < event_msg.update.nodes.size(); ++i) {
276 locations_[event_msg.update.nodes[i].id] = 276 locations_[event_msg.update.nodes[i].id] =
277 event_msg.update.nodes[i].location; 277 event_msg.update.nodes[i].location;
278 } 278 }
279 279
280 DVLOG(0) << "Accessibility event: " << ui::ToString(event.event_type) 280 LOG(ERROR) << "Accessibility event: " << ui::ToString(event.event_type)
Peter Lundblad 2015/11/20 13:42:59 We all do the same... :-)
dmazzoni 2015/11/23 20:16:50 Yeah, this should probably be VLOG(1) or something
281 << " on node id " << event_msg.id 281 << " on node id " << event_msg.id
282 << "\n" << event_msg.update.ToString(); 282 << "\n" << event_msg.update.ToString();
283 } 283 }
284 284
285 Send(new AccessibilityHostMsg_Events(routing_id(), event_msgs, reset_token_)); 285 Send(new AccessibilityHostMsg_Events(routing_id(), event_msgs, reset_token_));
286 reset_token_ = 0; 286 reset_token_ = 0;
287 287
288 if (had_layout_complete_messages) 288 if (had_layout_complete_messages)
289 SendLocationChanges(); 289 SendLocationChanges();
290 } 290 }
291 291
292 void RendererAccessibility::SendLocationChanges() { 292 void RendererAccessibility::SendLocationChanges() {
(...skipping 267 matching lines...) Expand 10 before | Expand all | Expand 10 after
560 #ifndef NDEBUG 560 #ifndef NDEBUG
561 LOG(WARNING) << "ShowContextMenu on invalid object id " << acc_obj_id; 561 LOG(WARNING) << "ShowContextMenu on invalid object id " << acc_obj_id;
562 #endif 562 #endif
563 return; 563 return;
564 } 564 }
565 565
566 obj.showContextMenu(); 566 obj.showContextMenu();
567 } 567 }
568 568
569 } // namespace content 569 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698