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

Unified Diff: chrome/browser/browser_accessibility_unittest.cc

Issue 2121004: Windows accessibility improvements: 1. All WebKit roles are now passed to the... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 10 years, 7 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/browser/browser_accessibility_manager.cc ('k') | chrome/browser/renderer_host/render_view_host.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/browser_accessibility_unittest.cc
===================================================================
--- chrome/browser/browser_accessibility_unittest.cc (revision 47913)
+++ chrome/browser/browser_accessibility_unittest.cc (working copy)
@@ -47,13 +47,13 @@
WebAccessibility button;
button.id = 2;
button.name = L"Button";
- button.role = WebAccessibility::ROLE_PUSHBUTTON;
+ button.role = WebAccessibility::ROLE_BUTTON;
button.state = 0;
WebAccessibility checkbox;
checkbox.id = 3;
checkbox.name = L"Checkbox";
- checkbox.role = WebAccessibility::ROLE_CHECKBUTTON;
+ checkbox.role = WebAccessibility::ROLE_CHECKBOX;
checkbox.state = 0;
WebAccessibility root;
@@ -71,7 +71,8 @@
CountedBrowserAccessibility::global_obj_count_ = 0;
BrowserAccessibilityManager* manager =
new BrowserAccessibilityManager(
- GetDesktopWindow(), root, new CountedBrowserAccessibilityFactory());
+ GetDesktopWindow(), root, NULL,
+ new CountedBrowserAccessibilityFactory());
ASSERT_EQ(3, CountedBrowserAccessibility::global_obj_count_);
// Delete the manager and test that all 3 instances are deleted.
@@ -81,7 +82,8 @@
// Construct a manager again, and this time use the IAccessible interface
// to get new references to two of the three nodes in the tree.
manager = new BrowserAccessibilityManager(
- GetDesktopWindow(), root, new CountedBrowserAccessibilityFactory());
+ GetDesktopWindow(), root, NULL,
+ new CountedBrowserAccessibilityFactory());
ASSERT_EQ(3, CountedBrowserAccessibility::global_obj_count_);
BrowserAccessibility* root_accessible = manager->GetRoot();
IDispatch* root_iaccessible = NULL;
« no previous file with comments | « chrome/browser/browser_accessibility_manager.cc ('k') | chrome/browser/renderer_host/render_view_host.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698