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

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

Issue 2561963002: base: Remove the string logging from CHECK(). (Closed)
Patch Set: checkstring: rebase Created 4 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 "content/browser/accessibility/browser_accessibility_manager.h" 5 #include "content/browser/accessibility/browser_accessibility_manager.h"
6 6
7 #include <stddef.h> 7 #include <stddef.h>
8 8
9 #include "base/logging.h" 9 #include "base/logging.h"
10 #include "build/build_config.h" 10 #include "build/build_config.h"
(...skipping 324 matching lines...) Expand 10 before | Expand all | Expand 10 after
335 void BrowserAccessibilityManager::OnAccessibilityEvents( 335 void BrowserAccessibilityManager::OnAccessibilityEvents(
336 const std::vector<AXEventNotificationDetails>& details) { 336 const std::vector<AXEventNotificationDetails>& details) {
337 // Process all changes to the accessibility tree first. 337 // Process all changes to the accessibility tree first.
338 for (uint32_t index = 0; index < details.size(); ++index) { 338 for (uint32_t index = 0; index < details.size(); ++index) {
339 const AXEventNotificationDetails& detail = details[index]; 339 const AXEventNotificationDetails& detail = details[index];
340 if (!tree_->Unserialize(detail.update)) { 340 if (!tree_->Unserialize(detail.update)) {
341 if (delegate_) { 341 if (delegate_) {
342 LOG(ERROR) << tree_->error(); 342 LOG(ERROR) << tree_->error();
343 delegate_->AccessibilityFatalError(); 343 delegate_->AccessibilityFatalError();
344 } else { 344 } else {
345 CHECK(false) << tree_->error(); 345 CHECK(false);
346 } 346 }
347 return; 347 return;
348 } 348 }
349 } 349 }
350 350
351 // If the root's parent is in another accessibility tree but it wasn't 351 // If the root's parent is in another accessibility tree but it wasn't
352 // previously connected, post the proper notifications on the parent. 352 // previously connected, post the proper notifications on the parent.
353 BrowserAccessibility* parent = GetParentNodeFromParentTree(); 353 BrowserAccessibility* parent = GetParentNodeFromParentTree();
354 if (parent) { 354 if (parent) {
355 if (!connected_to_parent_tree_node_) { 355 if (!connected_to_parent_tree_node_) {
(...skipping 889 matching lines...) Expand 10 before | Expand all | Expand 10 after
1245 hit_test_result = parent; 1245 hit_test_result = parent;
1246 parent = parent->GetParent(); 1246 parent = parent->GetParent();
1247 } 1247 }
1248 1248
1249 last_hover_ax_tree_id_ = hit_test_result->manager()->ax_tree_id(); 1249 last_hover_ax_tree_id_ = hit_test_result->manager()->ax_tree_id();
1250 last_hover_node_id_ = hit_test_result->GetId(); 1250 last_hover_node_id_ = hit_test_result->GetId();
1251 last_hover_bounds_ = hit_test_result->GetScreenBoundsRect(); 1251 last_hover_bounds_ = hit_test_result->GetScreenBoundsRect();
1252 } 1252 }
1253 1253
1254 } // namespace content 1254 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698