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

Unified Diff: chrome/browser/site_per_process_interactive_browsertest.cc

Issue 2844553002: Revert of OOPIF: Instrument TabAndMouseNavigation for WIN Flakiness. (Closed)
Patch Set: Created 3 years, 8 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/site_per_process_interactive_browsertest.cc
diff --git a/chrome/browser/site_per_process_interactive_browsertest.cc b/chrome/browser/site_per_process_interactive_browsertest.cc
index 15a795317966c8f1a7587da029d5665b03031ff8..d3598fa45c32bc7bcf68432d9ce7ce32b3914e83 100644
--- a/chrome/browser/site_per_process_interactive_browsertest.cc
+++ b/chrome/browser/site_per_process_interactive_browsertest.cc
@@ -327,7 +327,7 @@
EXPECT_EQ(main_frame, web_contents->GetFocusedFrame());
}
-#if (defined(OS_LINUX) && !defined(USE_OZONE)) || defined(OS_WIN)
+#if (defined(OS_LINUX) && !defined(USE_OZONE))
// Ensures that renderers know to advance focus to sibling frames and parent
// frames in the presence of mouse click initiated focus changes.
// Verifies against regression of https://crbug.com/702330
@@ -367,7 +367,6 @@
std::string script =
"function onFocus(e) {"
" domAutomationController.setAutomationId(0);"
- " console.log(window.name + '-focused-' + e.target.id);"
" domAutomationController.send(window.name + '-focused-' + e.target.id);"
"}"
""
@@ -381,13 +380,7 @@
" return Math.floor(rect.left) +','+"
" Math.floor(rect.top);"
"}"
- "function onClick(e) {"
- " console.log('Click event ' + window.name + ' at: ' + e.x + ', ' + e.y "
- " + ' screen: ' + e.screenX + ', ' + e.screenY);"
- "}"
""
- "window.addEventListener('click', onClick);"
- "console.log(document.location.origin);"
"document.styleSheets[0].insertRule('input {width:100%;margin:0;}', 1);"
"document.styleSheets[0].insertRule('h2 {margin:0;}', 1);"
"var input1 = document.createElement('input');"
@@ -447,7 +440,6 @@
std::string reply;
SimulateKeyPress(web_contents, ui::DomKey::TAB, ui::DomCode::TAB,
ui::VKEY_TAB, false, reverse /* shift */, false, false);
- LOG(INFO) << "Press tab";
EXPECT_TRUE(msg_queue.WaitForMessage(&reply));
return reply;
};
@@ -462,7 +454,6 @@
ui_controls::SendMouseClick(ui_controls::LEFT);
std::string reply;
- LOG(INFO) << "Click element";
EXPECT_TRUE(msg_queue.WaitForMessage(&reply));
return reply;
};
« 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