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

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

Issue 2558933002: Add more fine-grained accessibility modes. (Closed)
Patch Set: Reformat enums as uppercase 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 side-by-side diff with in-line comments
Download patch
Index: content/browser/accessibility/accessibility_ipc_error_browsertest.cc
diff --git a/content/browser/accessibility/accessibility_ipc_error_browsertest.cc b/content/browser/accessibility/accessibility_ipc_error_browsertest.cc
index 7d9f0042da928cbb06257b235f727303a19fb9fa..cb4d55bff638528a2ffc41a691b792e53a6b4115 100644
--- a/content/browser/accessibility/accessibility_ipc_error_browsertest.cc
+++ b/content/browser/accessibility/accessibility_ipc_error_browsertest.cc
@@ -60,12 +60,12 @@ IN_PROC_BROWSER_TEST_F(AccessibilityIpcErrorBrowserTest,
ASSERT_EQ(nullptr, frame->GetOrCreateBrowserAccessibilityManager());
{
- // Enable accessibility (passing AccessibilityModeComplete to
+ // Enable accessibility (passing ACCESSIBILITY_MODE_COMPLETE to
// AccessibilityNotificationWaiter does this automatically) and wait for
// the first event.
AccessibilityNotificationWaiter waiter(
shell()->web_contents(),
- AccessibilityModeComplete,
+ ACCESSIBILITY_MODE_COMPLETE,
ui::AX_EVENT_LAYOUT_COMPLETE);
waiter.WaitForNotification();
}
@@ -87,7 +87,7 @@ IN_PROC_BROWSER_TEST_F(AccessibilityIpcErrorBrowserTest,
// notification triggered by the hide.
AccessibilityNotificationWaiter waiter(
shell()->web_contents(),
- AccessibilityModeComplete,
+ ACCESSIBILITY_MODE_COMPLETE,
ui::AX_EVENT_LIVE_REGION_CHANGED);
ASSERT_TRUE(ExecuteScript(
shell(), "document.getElementById('p1').style.display = 'none';"));
@@ -104,7 +104,8 @@ IN_PROC_BROWSER_TEST_F(AccessibilityIpcErrorBrowserTest,
const ui::AXTree* tree = nullptr;
{
AccessibilityNotificationWaiter waiter(
- shell()->web_contents(), AccessibilityModeComplete, ui::AX_EVENT_FOCUS);
+ shell()->web_contents(), ACCESSIBILITY_MODE_COMPLETE,
+ ui::AX_EVENT_FOCUS);
ASSERT_TRUE(
ExecuteScript(shell(), "document.getElementById('button').focus();"));
waiter.WaitForNotification();
@@ -150,12 +151,12 @@ IN_PROC_BROWSER_TEST_F(AccessibilityIpcErrorBrowserTest,
shell()->web_contents()->GetMainFrame());
{
- // Enable accessibility (passing AccessibilityModeComplete to
+ // Enable accessibility (passing ACCESSIBILITY_MODE_COMPLETE to
// AccessibilityNotificationWaiter does this automatically) and wait for
// the first event.
AccessibilityNotificationWaiter waiter(
shell()->web_contents(),
- AccessibilityModeComplete,
+ ACCESSIBILITY_MODE_COMPLETE,
ui::AX_EVENT_LAYOUT_COMPLETE);
waiter.WaitForNotification();
}
@@ -185,7 +186,7 @@ IN_PROC_BROWSER_TEST_F(AccessibilityIpcErrorBrowserTest,
AccessibilityNotificationWaiter waiter(
shell()->web_contents(),
- AccessibilityModeComplete,
+ ACCESSIBILITY_MODE_COMPLETE,
ui::AX_EVENT_LOAD_COMPLETE);
waiter.WaitForNotification();
}

Powered by Google App Engine
This is Rietveld 408576698