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

Unified Diff: ios/chrome/browser/context_menu/context_menu_egtest.mm

Issue 2748873002: Make testContextMenuOpenInNewTabFromTallPage less flaky. (Closed)
Patch Set: fix blank lines Created 3 years, 9 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: ios/chrome/browser/context_menu/context_menu_egtest.mm
diff --git a/ios/chrome/browser/context_menu/context_menu_egtest.mm b/ios/chrome/browser/context_menu/context_menu_egtest.mm
index 923a41ed8126b9f97ce4a1d10439a39cdcad8989..a8cfb8038e0ef3baed7ee53a7a15f52678666260 100644
--- a/ios/chrome/browser/context_menu/context_menu_egtest.mm
+++ b/ios/chrome/browser/context_menu/context_menu_egtest.mm
@@ -208,6 +208,12 @@ void SelectTabAtIndexInCurrentMode(NSUInteger index) {
[ChromeEarlGrey loadURL:initialURL];
chrome_test_util::AssertMainTabCount(1U);
+ // Scroll down on the web view to make sure the toolbar disappears.
+ [[EarlGrey
+ selectElementWithMatcher:WebViewScrollView(
+ chrome_test_util::GetCurrentWebState())]
+ performAction:grey_swipeFastInDirection(kGREYDirectionUp)];
Eugene But (OOO till 7-30) 2017/03/13 22:35:01 Instead of scrolling twice should we keep scrollin
baxley 2017/03/13 22:43:41 This API does (or should) scroll to the bottom. I'
Eugene But (OOO till 7-30) 2017/03/13 22:54:28 Maybe this is because WebView's height has changed
baxley 2017/03/14 23:32:10 Yes, that's my guess. This method is used in fulls
+
// Scroll down on the web view to make the link visible.
[[EarlGrey
selectElementWithMatcher:WebViewScrollView(
« 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