Commit aa206989 authored by Justin Cohen's avatar Justin Cohen Committed by Commit Bot

[ios] Disable more context menu tests on iOS13.

These tests are also failing on iPhone now.

Bug: 989550
Change-Id: If673678464e4c1158a859a1adfafa118bbfc3698
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1752080Reviewed-by: default avatarPeter Lee <pkl@chromium.org>
Reviewed-by: default avatarMike Dougherty <michaeldo@chromium.org>
Commit-Queue: Justin Cohen <justincohen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#686503}
parent 05de3663
...@@ -655,9 +655,7 @@ void CloseTabAtIndexAndSync(NSUInteger i) { ...@@ -655,9 +655,7 @@ void CloseTabAtIndexAndSync(NSUInteger i) {
- (void)testOpenLinkInNewTab { - (void)testOpenLinkInNewTab {
// TODO(crbug.com/989550) Disable broken context menu tests on Xc11b5. // TODO(crbug.com/989550) Disable broken context menu tests on Xc11b5.
if (@available(iOS 13, *)) { if (@available(iOS 13, *)) {
if ([ChromeEarlGrey isIPadIdiom]) { EARL_GREY_TEST_DISABLED(@"Test disabled on iOS13.");
EARL_GREY_TEST_DISABLED(@"Test disabled on iPad.");
}
} }
// Create map of canned responses and set up the test HTML server. // Create map of canned responses and set up the test HTML server.
std::map<GURL, std::string> responses; std::map<GURL, std::string> responses;
......
...@@ -174,9 +174,7 @@ bool WaitForDownloadButton() { ...@@ -174,9 +174,7 @@ bool WaitForDownloadButton() {
- (void)testDownloadInNewTab { - (void)testDownloadInNewTab {
// TODO(crbug.com/989550) Disable broken context menu tests on Xc11b5. // TODO(crbug.com/989550) Disable broken context menu tests on Xc11b5.
if (@available(iOS 13, *)) { if (@available(iOS 13, *)) {
if ([ChromeEarlGrey isIPadIdiom]) { EARL_GREY_TEST_DISABLED(@"Test disabled on iOS13.");
EARL_GREY_TEST_DISABLED(@"Test disabled on iPad.");
}
} }
[ChromeEarlGrey loadURL:self.testServer->GetURL("/")]; [ChromeEarlGrey loadURL:self.testServer->GetURL("/")];
......
...@@ -627,9 +627,7 @@ void FocusOmnibox() { ...@@ -627,9 +627,7 @@ void FocusOmnibox() {
- (void)testNavigationButtons { - (void)testNavigationButtons {
// TODO(crbug.com/989550) Disable broken context menu tests on Xc11b5. // TODO(crbug.com/989550) Disable broken context menu tests on Xc11b5.
if (@available(iOS 13, *)) { if (@available(iOS 13, *)) {
if ([ChromeEarlGrey isIPadIdiom]) { EARL_GREY_TEST_DISABLED(@"Test disabled on iOS13.");
EARL_GREY_TEST_DISABLED(@"Test disabled on iPad.");
}
} }
// Setup the server. // Setup the server.
self.testServer->RegisterRequestHandler( self.testServer->RegisterRequestHandler(
......
...@@ -55,9 +55,7 @@ const char kLinksTestURL2Text[] = "arrived"; ...@@ -55,9 +55,7 @@ const char kLinksTestURL2Text[] = "arrived";
- (void)testChildTabOrdering { - (void)testChildTabOrdering {
// TODO(crbug.com/989550) Disable broken context menu tests on Xc11b5. // TODO(crbug.com/989550) Disable broken context menu tests on Xc11b5.
if (@available(iOS 13, *)) { if (@available(iOS 13, *)) {
if ([ChromeEarlGrey isIPadIdiom]) { EARL_GREY_TEST_DISABLED(@"Test disabled on iOS13.");
EARL_GREY_TEST_DISABLED(@"Test disabled on iPad.");
}
} }
GREYAssertTrue(self.testServer->Start(), @"Test server failed to start."); GREYAssertTrue(self.testServer->Start(), @"Test server failed to start.");
......
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