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 eb7b3a26c4a95be515748c9899f08e72b60173aa..a3963605a420435bb03f5c098804d10430c7849a 100644 |
--- a/chrome/browser/site_per_process_interactive_browsertest.cc |
+++ b/chrome/browser/site_per_process_interactive_browsertest.cc |
@@ -302,11 +302,11 @@ IN_PROC_BROWSER_TEST_F(SitePerProcessInteractiveBrowserTest, |
EXPECT_EQ(main_frame, web_contents->GetFocusedFrame()); |
} |
-// TODO(https://crbug.com/702330): Enable this test. |
+#if (defined(OS_LINUX) && !defined(USE_OZONE)) || defined(OS_WIN) |
alexmos
2017/03/30 17:16:11
What's wrong with the test on the excluded platfor
avallee
2017/04/02 02:27:30
For Mac, I wrote a sample that just prints mousemo
|
// Ensures that renderers know to advance focus to sibling frames and parent |
// frames in the presence of mouse click initiated focus changes. |
alexmos
2017/03/30 17:16:11
nit: I'd keep the reference to the bug here.
avallee
2017/04/02 02:27:30
Done.
|
IN_PROC_BROWSER_TEST_F(SitePerProcessInteractiveBrowserTest, |
- DISABLED_TabAndMouseFocusNavigation) { |
+ TabAndMouseFocusNavigation) { |
GURL main_url(embedded_test_server()->GetURL( |
"a.com", "/cross_site_iframe_factory.html?a(b,c)")); |
ui_test_utils::NavigateToURL(browser(), main_url); |
@@ -336,7 +336,6 @@ IN_PROC_BROWSER_TEST_F(SitePerProcessInteractiveBrowserTest, |
// iframe: 55,18;55,67 |
std::string script = |
"function onFocus(e) {" |
- " console.log(window.name+'-focused-'+ e.target.id);" |
" domAutomationController.setAutomationId(0);" |
" domAutomationController.send(window.name + '-focused-' + e.target.id);" |
"}" |
@@ -474,6 +473,7 @@ IN_PROC_BROWSER_TEST_F(SitePerProcessInteractiveBrowserTest, |
click_element_and_wait_for_message(main_frame_input_coords[0])) |
<< "Unexpected extra focus events."; |
} |
+#endif |
namespace { |