Commit f0aff178 authored by Yi Gu's avatar Yi Gu Committed by Commit Bot

[VizHitTesting] Enable SitePerProcessHitTestBrowserTest.NestedPopupMenuTest

The reasons of disabling the test are no longer valid.

Bug: 599730
Change-Id: I13d74de9b332ee9b3f22bc0d9f68600772b2c642
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1754784Reviewed-by: default avatarKen Buchanan <kenrb@chromium.org>
Commit-Queue: Yi Gu <yigu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#687319}
parent 7cada88c
...@@ -5492,12 +5492,11 @@ IN_PROC_BROWSER_TEST_P(SitePerProcessHitTestBrowserTest, PopupMenuTest) { ...@@ -5492,12 +5492,11 @@ IN_PROC_BROWSER_TEST_P(SitePerProcessHitTestBrowserTest, PopupMenuTest) {
// its out-of-process iframe. This verifies that screen positioning information // its out-of-process iframe. This verifies that screen positioning information
// is propagating down the frame tree correctly. // is propagating down the frame tree correctly.
#if defined(OS_ANDROID) #if defined(OS_ANDROID)
// Surface-based hit testing and coordinate translation is not yet avaiable on // On Android the reported menu coordinates are relative to the OOPIF, and its
// Android. // screen position is computed later, so this test isn't relevant.
#define MAYBE_NestedPopupMenuTest DISABLED_NestedPopupMenuTest #define MAYBE_NestedPopupMenuTest DISABLED_NestedPopupMenuTest
#else #else
// Times out frequently. https://crbug.com/599730. #define MAYBE_NestedPopupMenuTest NestedPopupMenuTest
#define MAYBE_NestedPopupMenuTest DISABLED_NestedPopupMenuTest
#endif #endif
IN_PROC_BROWSER_TEST_P(SitePerProcessHitTestBrowserTest, IN_PROC_BROWSER_TEST_P(SitePerProcessHitTestBrowserTest,
MAYBE_NestedPopupMenuTest) { MAYBE_NestedPopupMenuTest) {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment