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

Unified Diff: content/browser/accessibility/accessibility_win_browsertest.cc

Issue 12378095: Disable some accessibility tests on Win64 build (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 7 years, 10 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 | « no previous file | content/browser/accessibility/cross_platform_accessibility_browsertest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/browser/accessibility/accessibility_win_browsertest.cc
===================================================================
--- content/browser/accessibility/accessibility_win_browsertest.cc (revision 185913)
+++ content/browser/accessibility/accessibility_win_browsertest.cc (working copy)
@@ -29,6 +29,13 @@
using std::vector;
using std::wstring;
+// TODO(dmazzoni): Disabled accessibility tests on Win64. crbug.com/179717
+#if defined(ARCH_CPU_X86_64)
+#define MAYBE(x) DISABLED_##x
+#else
+#define MAYBE(x) x
+#endif
+
namespace content {
namespace {
@@ -412,7 +419,7 @@
}
IN_PROC_BROWSER_TEST_F(AccessibilityWinBrowserTest,
- TestBusyAccessibilityTree) {
+ MAYBE(TestBusyAccessibilityTree)) {
NavigateToURL(shell(), GURL(chrome::kAboutBlankURL));
// The initial accessible returned should have state STATE_SYSTEM_BUSY while
@@ -517,7 +524,7 @@
}
IN_PROC_BROWSER_TEST_F(AccessibilityWinBrowserTest,
- TestNotificationCheckedStateChanged) {
+ MAYBE(TestNotificationCheckedStateChanged)) {
LoadInitialAccessibilityTreeFromHtml(
"<body><input type='checkbox' /></body>");
@@ -547,7 +554,7 @@
}
IN_PROC_BROWSER_TEST_F(AccessibilityWinBrowserTest,
- TestNotificationChildrenChanged) {
+ MAYBE(TestNotificationChildrenChanged)) {
// The role attribute causes the node to be in the accessibility tree.
LoadInitialAccessibilityTreeFromHtml("<body role=group></body>");
@@ -574,7 +581,7 @@
}
IN_PROC_BROWSER_TEST_F(AccessibilityWinBrowserTest,
- TestNotificationChildrenChanged2) {
+ MAYBE(TestNotificationChildrenChanged2)) {
// The role attribute causes the node to be in the accessibility tree.
LoadInitialAccessibilityTreeFromHtml(
"<div role=group style='visibility: hidden'>text</div>");
@@ -602,7 +609,7 @@
}
IN_PROC_BROWSER_TEST_F(AccessibilityWinBrowserTest,
- TestNotificationFocusChanged) {
+ MAYBE(TestNotificationFocusChanged)) {
// The role attribute causes the node to be in the accessibility tree.
LoadInitialAccessibilityTreeFromHtml("<div role=group tabindex='-1'></div>");
@@ -650,7 +657,7 @@
}
IN_PROC_BROWSER_TEST_F(AccessibilityWinBrowserTest,
- TestNotificationValueChanged) {
+ MAYBE(TestNotificationValueChanged)) {
LoadInitialAccessibilityTreeFromHtml(
"<body><input type='text' value='old value'/></body>");
@@ -687,7 +694,7 @@
// that wraps the tab contents returns the IAccessible implementation
// provided by RenderWidgetHostViewWin in GetNativeViewAccessible().
IN_PROC_BROWSER_TEST_F(AccessibilityWinBrowserTest,
- ContainsRendererAccessibilityTree) {
+ MAYBE(ContainsRendererAccessibilityTree)) {
LoadInitialAccessibilityTreeFromHtml(
"<html><head><title>MyDocument</title></head>"
"<body>Content</body></html>");
@@ -782,7 +789,7 @@
}
IN_PROC_BROWSER_TEST_F(AccessibilityWinBrowserTest,
- SupportsISimpleDOM) {
+ MAYBE(SupportsISimpleDOM)) {
LoadInitialAccessibilityTreeFromHtml(
"<body><input type='checkbox' /></body>");
@@ -842,7 +849,7 @@
EXPECT_EQ(0, num_children);
}
-IN_PROC_BROWSER_TEST_F(AccessibilityWinBrowserTest, TestRoleGroup) {
+IN_PROC_BROWSER_TEST_F(AccessibilityWinBrowserTest, MAYBE(TestRoleGroup)) {
LoadInitialAccessibilityTreeFromHtml(
"<fieldset></fieldset><div role=group></div>");
« no previous file with comments | « no previous file | content/browser/accessibility/cross_platform_accessibility_browsertest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698