diff options
| author | niranjan@google.com <niranjan@google.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2008-11-21 08:15:38 +0000 |
|---|---|---|
| committer | niranjan@google.com <niranjan@google.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2008-11-21 08:15:38 +0000 |
| commit | acfab0510bc977a795578a560e16aecb2cb2e552 (patch) | |
| tree | 098a79e00b4c6e1383f97803ce77ab8cea024646 /chrome/test/accessibility/constants.h | |
| parent | 11e020fcd65c415dc46d10ae9dbe6ba63600ee4c (diff) | |
| download | chromium_src-acfab0510bc977a795578a560e16aecb2cb2e552.zip chromium_src-acfab0510bc977a795578a560e16aecb2cb2e552.tar.gz chromium_src-acfab0510bc977a795578a560e16aecb2cb2e552.tar.bz2 | |
Fixed all the remaining hardcoded strings in AccessibilityTests
Changed the GetRole() and GetState() methods to return DWORD constants instead of strings. We didn't really need the string values of Roles and States and this led us to compare strings in the test cases when we should have been just comparing constants. This change fixes all that.
All the accessibility tests should now run on locales other than en-us and as a nice side effect, the tests should run very very marginally faster.
TEST=ui_tests.exe --gtest_filter=AccessibilityTest*
Review URL: http://codereview.chromium.org/11558
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@5821 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/test/accessibility/constants.h')
| -rw-r--r-- | chrome/test/accessibility/constants.h | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/chrome/test/accessibility/constants.h b/chrome/test/accessibility/constants.h index 967de0c..a3d6060 100644 --- a/chrome/test/accessibility/constants.h +++ b/chrome/test/accessibility/constants.h @@ -36,6 +36,9 @@ #define CHROME_APP_ACC_INDEX (3) #define CHROME_CLIENT_ACC_INDEX (0) +#define ROLE_ERROR (-1) +#define STATE_ERROR (-1) + // Chrome Client chidren. #ifdef NEW_FRAMES #define BROWSER_VIEW_ACC_INDEX (4) |