Commit 780610d2 authored by Guillaume Jenkins's avatar Guillaume Jenkins Committed by Commit Bot

Revert "Re-enable the RightClickMenuShowsCopyUrl test on mac."

This reverts commit 21497283.

Reason for revert: It seems the timeouts are still happening, see the builder page here: https://ci.chromium.org/p/chromium/builders/ci/Mac10.11%20Tests

Original change's description:
> Re-enable the RightClickMenuShowsCopyUrl test on mac.
> 
> Test timeout on CQ seems to have mysteriously fixed itself.
> 
> Bug:988323
> Fixed:988323
> 
> Change-Id: Ia96d8a687bc161cf74f67fbf819ffaf7bb896755
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2325330
> Auto-Submit: Glen Robertson <glenrob@chromium.org>
> Commit-Queue: Evan Stade <estade@chromium.org>
> Reviewed-by: Evan Stade <estade@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#792705}

TBR=estade@chromium.org,glenrob@chromium.org

Change-Id: I5de2c6b986703f81b43977072951f1923ce3bb8f
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 988323
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2326733Reviewed-by: default avatarGuillaume Jenkins <gujen@google.com>
Commit-Queue: Guillaume Jenkins <gujen@google.com>
Cr-Commit-Position: refs/heads/master@{#792762}
parent 50292560
...@@ -422,8 +422,14 @@ IN_PROC_BROWSER_TEST_P(CustomTabBarViewBrowserTest, ...@@ -422,8 +422,14 @@ IN_PROC_BROWSER_TEST_P(CustomTabBarViewBrowserTest,
} }
// Right-click menu on CustomTabBar should have Copy URL option. // Right-click menu on CustomTabBar should have Copy URL option.
// TODO(crbug.com/988323): Times out on Mac.
#if defined(OS_MACOSX)
#define MAYBE_RightClickMenuShowsCopyUrl DISABLED_RightClickMenuShowsCopyUrl
#else
#define MAYBE_RightClickMenuShowsCopyUrl RightClickMenuShowsCopyUrl
#endif
IN_PROC_BROWSER_TEST_P(CustomTabBarViewBrowserTest, IN_PROC_BROWSER_TEST_P(CustomTabBarViewBrowserTest,
RightClickMenuShowsCopyUrl) { MAYBE_RightClickMenuShowsCopyUrl) {
const GURL& app_url = https_server()->GetURL("app.com", "/ssl/google.html"); const GURL& app_url = https_server()->GetURL("app.com", "/ssl/google.html");
InstallPWA(app_url); InstallPWA(app_url);
EXPECT_TRUE(app_browser_->is_type_app()); EXPECT_TRUE(app_browser_->is_type_app());
......
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