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

Unified Diff: chrome/browser/views/browser_keyboard_accessibility_test_win.cc

Issue 3071011: GTTF: Add more logging to BrowserKeyboardAccessibility test... (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 10 years, 5 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 | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/views/browser_keyboard_accessibility_test_win.cc
===================================================================
--- chrome/browser/views/browser_keyboard_accessibility_test_win.cc (revision 54372)
+++ chrome/browser/views/browser_keyboard_accessibility_test_win.cc (working copy)
@@ -35,6 +35,9 @@
current_event_type_ = event_type;
// Are we within a message loop waiting for a particular event?
+ // TODO(phajdan.jr): remove logging after fixing http://crbug.com/50663.
+ LOG(ERROR) << "Got notification; is_waiting_ is "
+ << (is_waiting_ ? "true" : "false");
if (is_waiting_) {
is_waiting_ = false;
MessageLoop::current()->Quit();
@@ -98,7 +101,14 @@
// Tab through each of the three tabs.
for (int i = 0; i < 3; ++i) {
+ // TODO(phajdan.jr): remove logging after fixing http://crbug.com/50663.
+ LOG(ERROR) << "Iteration no. " << i;
+
TabCyclerForwardAndBack(current_view_native_window());
+
+
+ // TODO(phajdan.jr): remove logging after fixing http://crbug.com/50663.
+ LOG(ERROR) << "Sending TAB key event...";
ui_controls::SendKeyPressNotifyWhenDone(current_view_native_window(),
base::VKEY_TAB,
true, false, false, false,
@@ -152,9 +162,13 @@
// Wait for a focus event on the provided native window.
while (current_event() != AccessibilityTypes::EVENT_FOCUS ||
current_view_native_window() != hwnd) {
+ // TODO(phajdan.jr): remove logging after fixing http://crbug.com/50663.
+ LOG(ERROR) << "Runnig message loop...";
set_waiting(true);
ui_test_utils::RunMessageLoop();
}
+ // TODO(phajdan.jr): remove logging after fixing http://crbug.com/50663.
+ LOG(ERROR) << "After the loop.";
views::View* first_focused_item = current_view();
@@ -164,12 +178,16 @@
// Keep tabbing until we reach the originally focused view.
do {
+ // TODO(phajdan.jr): remove logging after fixing http://crbug.com/50663.
+ LOG(ERROR) << "Sending TAB key event.";
ui_controls::SendKeyPressNotifyWhenDone(hwnd, base::VKEY_TAB,
false, !forward_tab, false, false, new MessageLoop::QuitTask());
set_waiting(true);
ui_test_utils::RunMessageLoop();
next_focused_item = current_view();
} while (first_focused_item != next_focused_item);
+ // TODO(phajdan.jr): remove logging after fixing http://crbug.com/50663.
+ LOG(ERROR) << "After second loop.";
}
} // namespace
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698