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

Side by Side Diff: ios/web/shell/test/page_state_egtest.mm

Issue 2623333004: Update EarlGrey bug references for device tests. (Closed)
Patch Set: Created 3 years, 11 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 unified diff | Download patch
« no previous file with comments | « ios/web/shell/test/context_menu_egtest.mm ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #import <CoreGraphics/CoreGraphics.h> 5 #import <CoreGraphics/CoreGraphics.h>
6 #import <EarlGrey/EarlGrey.h> 6 #import <EarlGrey/EarlGrey.h>
7 #import <Foundation/Foundation.h> 7 #import <Foundation/Foundation.h>
8 #import <XCTest/XCTest.h> 8 #import <XCTest/XCTest.h>
9 9
10 #include "base/ios/ios_util.h" 10 #include "base/ios/ios_util.h"
(...skipping 39 matching lines...) Expand 10 before | Expand all | Expand 10 after
50 } // namespace 50 } // namespace
51 51
52 using web::test::HttpServer; 52 using web::test::HttpServer;
53 53
54 // Page state test cases for the web shell. 54 // Page state test cases for the web shell.
55 @interface PageStateTestCase : ShellBaseTestCase 55 @interface PageStateTestCase : ShellBaseTestCase
56 @end 56 @end
57 57
58 @implementation PageStateTestCase 58 @implementation PageStateTestCase
59 59
60 // TODO(crbug.com/675015): Re-enable this test on device. 60 // TODO(crbug.com/675247): Re-enable this test on device.
Eugene But (OOO till 7-30) 2017/01/13 00:01:14 I've removed this already: https://codereview.chro
baxley 2017/01/13 15:08:58 Done.
61 #if TARGET_IPHONE_SIMULATOR 61 #if TARGET_IPHONE_SIMULATOR
62 #define MAYBE_testScrollPositionRestoring testScrollPositionRestoring 62 #define MAYBE_testScrollPositionRestoring testScrollPositionRestoring
63 #else 63 #else
64 #define MAYBE_testScrollPositionRestoring FLAKY_testScrollPositionRestoring 64 #define MAYBE_testScrollPositionRestoring FLAKY_testScrollPositionRestoring
65 #endif 65 #endif
66 // Tests that page scroll position of a page is restored upon returning to the 66 // Tests that page scroll position of a page is restored upon returning to the
67 // page via the back/forward buttons. 67 // page via the back/forward buttons.
68 - (void)MAYBE_testScrollPositionRestoring { 68 - (void)MAYBE_testScrollPositionRestoring {
69 // TODO(crbug.com/670700): Re-enable this test. 69 // TODO(crbug.com/670700): Re-enable this test.
70 if (!base::ios::IsRunningOnIOS10OrLater()) { 70 if (!base::ios::IsRunningOnIOS10OrLater()) {
(...skipping 26 matching lines...) Expand all
97 assertWithMatcher:contentOffset(CGPointMake(0, kScrollOffset1))]; 97 assertWithMatcher:contentOffset(CGPointMake(0, kScrollOffset1))];
98 98
99 // Go forward and verify that the second page offset has been restored. 99 // Go forward and verify that the second page offset has been restored.
100 [[EarlGrey selectElementWithMatcher:web::forwardButton()] 100 [[EarlGrey selectElementWithMatcher:web::forwardButton()]
101 performAction:grey_tap()]; 101 performAction:grey_tap()];
102 [[EarlGrey selectElementWithMatcher:web::webViewScrollView()] 102 [[EarlGrey selectElementWithMatcher:web::webViewScrollView()]
103 assertWithMatcher:contentOffset(CGPointMake(0, kScrollOffset2))]; 103 assertWithMatcher:contentOffset(CGPointMake(0, kScrollOffset2))];
104 } 104 }
105 105
106 @end 106 @end
OLDNEW
« no previous file with comments | « ios/web/shell/test/context_menu_egtest.mm ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698