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

Unified Diff: ios/chrome/browser/web/navigation_egtest.mm

Issue 2642193012: Make EarlGrey matchers compliant with Chromium style. (Closed)
Patch Set: build.gn and rebase 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « ios/chrome/browser/web/js_print_egtest.mm ('k') | ios/chrome/browser/web/progress_indicator_egtest.mm » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ios/chrome/browser/web/navigation_egtest.mm
diff --git a/ios/chrome/browser/web/navigation_egtest.mm b/ios/chrome/browser/web/navigation_egtest.mm
index 739f3f93a82abb9ed0a9ceb96329b997a5858214..290ef24a5e6703ccfe1cee9b4500218ee46e5e11 100644
--- a/ios/chrome/browser/web/navigation_egtest.mm
+++ b/ios/chrome/browser/web/navigation_egtest.mm
@@ -16,10 +16,10 @@
#include "ios/web/public/test/response_providers/data_response_provider.h"
#include "ui/base/l10n/l10n_util.h"
-using chrome_test_util::backButton;
-using chrome_test_util::forwardButton;
+using chrome_test_util::BackButton;
+using chrome_test_util::ForwardButton;
using chrome_test_util::TapWebViewElementWithId;
-using chrome_test_util::webViewContainingText;
+using chrome_test_util::WebViewContainingText;
namespace {
@@ -106,13 +106,13 @@ void SetupBackAndForwardResponseProvider() {
// Matcher for the error page.
// TODO(crbug.com/638674): Evaluate if this can move to shared code. See
// ios/chrome/browser/ui/error_page_egtest.mm.
-id<GREYMatcher> errorPage() {
+id<GREYMatcher> ErrorPage() {
NSString* const kDNSError =
l10n_util::GetNSString(IDS_ERRORPAGES_HEADING_NOT_AVAILABLE);
NSString* const kInternetDisconnectedError =
l10n_util::GetNSString(IDS_ERRORPAGES_HEADING_INTERNET_DISCONNECTED);
- return grey_anyOf(chrome_test_util::staticHtmlViewContainingText(kDNSError),
- chrome_test_util::staticHtmlViewContainingText(
+ return grey_anyOf(chrome_test_util::StaticHtmlViewContainingText(kDNSError),
+ chrome_test_util::StaticHtmlViewContainingText(
kInternetDisconnectedError),
nil);
}
@@ -219,7 +219,7 @@ class RedirectResponseProvider : public web::DataResponseProvider {
const GURL windowHistoryURL =
web::test::HttpServer::MakeUrl(kWindowHistoryGoTestURL);
[ChromeEarlGrey loadURL:windowHistoryURL];
- [[EarlGrey selectElementWithMatcher:webViewContainingText(kOnLoadText)]
+ [[EarlGrey selectElementWithMatcher:WebViewContainingText(kOnLoadText)]
assertWithMatcher:grey_notNil()];
// Tap on the window.history.go() button. This will clear |kOnLoadText|, so
@@ -228,7 +228,7 @@ class RedirectResponseProvider : public web::DataResponseProvider {
[ChromeEarlGrey tapWebViewElementWithID:kGoNoParameterID];
// Verify that the onload text is reset.
- [[EarlGrey selectElementWithMatcher:webViewContainingText(kOnLoadText)]
+ [[EarlGrey selectElementWithMatcher:WebViewContainingText(kOnLoadText)]
assertWithMatcher:grey_notNil()];
}
@@ -240,7 +240,7 @@ class RedirectResponseProvider : public web::DataResponseProvider {
const GURL windowHistoryURL =
web::test::HttpServer::MakeUrl(kWindowHistoryGoTestURL);
[ChromeEarlGrey loadURL:windowHistoryURL];
- [[EarlGrey selectElementWithMatcher:webViewContainingText(kOnLoadText)]
+ [[EarlGrey selectElementWithMatcher:WebViewContainingText(kOnLoadText)]
assertWithMatcher:grey_notNil()];
// Tap on the window.history.go() button. This will clear |kOnLoadText|, so
@@ -249,7 +249,7 @@ class RedirectResponseProvider : public web::DataResponseProvider {
[ChromeEarlGrey tapWebViewElementWithID:kGoZeroID];
// Verify that the onload text is reset.
- [[EarlGrey selectElementWithMatcher:webViewContainingText(kOnLoadText)]
+ [[EarlGrey selectElementWithMatcher:WebViewContainingText(kOnLoadText)]
assertWithMatcher:grey_notNil()];
}
@@ -262,21 +262,21 @@ class RedirectResponseProvider : public web::DataResponseProvider {
const GURL windowHistoryURL =
web::test::HttpServer::MakeUrl(kWindowHistoryGoTestURL);
[ChromeEarlGrey loadURL:windowHistoryURL];
- [[EarlGrey selectElementWithMatcher:webViewContainingText(kOnLoadText)]
+ [[EarlGrey selectElementWithMatcher:WebViewContainingText(kOnLoadText)]
assertWithMatcher:grey_notNil()];
// Tap on the window.history.go(2) button. This will clear all div text, so
// the subsequent check for |kNoOpText| will only pass if no navigations have
// occurred.
[ChromeEarlGrey tapWebViewElementWithID:kGoTwoID];
- [[EarlGrey selectElementWithMatcher:webViewContainingText(kNoOpText)]
+ [[EarlGrey selectElementWithMatcher:WebViewContainingText(kNoOpText)]
assertWithMatcher:grey_notNil()];
// Tap on the window.history.go(-2) button. This will clear all div text, so
// the subsequent check for |kNoOpText| will only pass if no navigations have
// occurred.
[ChromeEarlGrey tapWebViewElementWithID:kGoBackTwoID];
- [[EarlGrey selectElementWithMatcher:webViewContainingText(kNoOpText)]
+ [[EarlGrey selectElementWithMatcher:WebViewContainingText(kNoOpText)]
assertWithMatcher:grey_notNil()];
}
@@ -302,22 +302,22 @@ class RedirectResponseProvider : public web::DataResponseProvider {
[ChromeEarlGrey loadURL:secondURL];
[ChromeEarlGrey loadURL:thirdURL];
[ChromeEarlGrey loadURL:fourthURL];
- [[EarlGrey selectElementWithMatcher:webViewContainingText("page4")]
+ [[EarlGrey selectElementWithMatcher:WebViewContainingText("page4")]
assertWithMatcher:grey_notNil()];
// Tap button to go back 3 pages.
TapWebViewElementWithId("goBack");
- [[EarlGrey selectElementWithMatcher:webViewContainingText("page1")]
+ [[EarlGrey selectElementWithMatcher:WebViewContainingText("page1")]
assertWithMatcher:grey_notNil()];
- [[EarlGrey selectElementWithMatcher:chrome_test_util::omniboxText(
+ [[EarlGrey selectElementWithMatcher:chrome_test_util::OmniboxText(
firstURL.GetContent())]
assertWithMatcher:grey_notNil()];
// Tap button to go forward 2 pages.
TapWebViewElementWithId("goForward");
- [[EarlGrey selectElementWithMatcher:webViewContainingText("page3")]
+ [[EarlGrey selectElementWithMatcher:WebViewContainingText("page3")]
assertWithMatcher:grey_notNil()];
- [[EarlGrey selectElementWithMatcher:chrome_test_util::omniboxText(
+ [[EarlGrey selectElementWithMatcher:chrome_test_util::OmniboxText(
thirdURL.GetContent())]
assertWithMatcher:grey_notNil()];
}
@@ -331,21 +331,21 @@ class RedirectResponseProvider : public web::DataResponseProvider {
const GURL windowHistoryURL =
web::test::HttpServer::MakeUrl(kWindowHistoryGoTestURL);
[ChromeEarlGrey loadURL:windowHistoryURL];
- [[EarlGrey selectElementWithMatcher:webViewContainingText(kOnLoadText)]
+ [[EarlGrey selectElementWithMatcher:WebViewContainingText(kOnLoadText)]
assertWithMatcher:grey_notNil()];
const GURL sampleURL = web::test::HttpServer::MakeUrl(kSampleFileBasedURL);
[ChromeEarlGrey loadURL:sampleURL];
[ChromeEarlGrey loadURL:windowHistoryURL];
- [[EarlGrey selectElementWithMatcher:webViewContainingText(kOnLoadText)]
+ [[EarlGrey selectElementWithMatcher:WebViewContainingText(kOnLoadText)]
assertWithMatcher:grey_notNil()];
// Tap the window.history.go(-2) button. This will clear the current page's
// |kOnLoadText|, so the subsequent check will only pass if another load
// occurs.
[ChromeEarlGrey tapWebViewElementWithID:kGoBackTwoID];
- [[EarlGrey selectElementWithMatcher:webViewContainingText(kOnLoadText)]
+ [[EarlGrey selectElementWithMatcher:WebViewContainingText(kOnLoadText)]
assertWithMatcher:grey_notNil()];
}
@@ -365,14 +365,14 @@ class RedirectResponseProvider : public web::DataResponseProvider {
// Tap the back button in the HTML and verify the first URL is loaded.
TapWebViewElementWithId(kBackHTMLButtonLabel);
- [[EarlGrey selectElementWithMatcher:chrome_test_util::omniboxText(
+ [[EarlGrey selectElementWithMatcher:chrome_test_util::OmniboxText(
firstURL.GetContent())]
assertWithMatcher:grey_notNil()];
// Tap the forward button in the toolbar and verify the second URL is loaded.
- [[EarlGrey selectElementWithMatcher:forwardButton()]
+ [[EarlGrey selectElementWithMatcher:ForwardButton()]
performAction:grey_tap()];
- [[EarlGrey selectElementWithMatcher:chrome_test_util::omniboxText(
+ [[EarlGrey selectElementWithMatcher:chrome_test_util::OmniboxText(
secondURL.GetContent())]
assertWithMatcher:grey_notNil()];
}
@@ -391,19 +391,19 @@ class RedirectResponseProvider : public web::DataResponseProvider {
// Tap the back button in the toolbar and verify the page with forward button
// is loaded.
- [[EarlGrey selectElementWithMatcher:backButton()] performAction:grey_tap()];
+ [[EarlGrey selectElementWithMatcher:BackButton()] performAction:grey_tap()];
[[EarlGrey
- selectElementWithMatcher:webViewContainingText(kForwardHTMLSentinel)]
+ selectElementWithMatcher:WebViewContainingText(kForwardHTMLSentinel)]
assertWithMatcher:grey_notNil()];
- [[EarlGrey selectElementWithMatcher:chrome_test_util::omniboxText(
+ [[EarlGrey selectElementWithMatcher:chrome_test_util::OmniboxText(
firstURL.GetContent())]
assertWithMatcher:grey_notNil()];
// Tap the forward button in the HTML and verify the second URL is loaded.
TapWebViewElementWithId(kForwardHTMLButtonLabel);
- [[EarlGrey selectElementWithMatcher:webViewContainingText(kTestPageSentinel)]
+ [[EarlGrey selectElementWithMatcher:WebViewContainingText(kTestPageSentinel)]
assertWithMatcher:grey_notNil()];
- [[EarlGrey selectElementWithMatcher:chrome_test_util::omniboxText(
+ [[EarlGrey selectElementWithMatcher:chrome_test_util::OmniboxText(
secondURL.GetContent())]
assertWithMatcher:grey_notNil()];
@@ -413,13 +413,13 @@ class RedirectResponseProvider : public web::DataResponseProvider {
if (UIApplication.sharedApplication.keyWindow.traitCollection
.horizontalSizeClass == UIUserInterfaceSizeClassCompact) {
// In horizontally compact environments, the forward button is not visible.
- [[EarlGrey selectElementWithMatcher:forwardButton()]
+ [[EarlGrey selectElementWithMatcher:ForwardButton()]
assertWithMatcher:grey_nil()];
} else {
// In horizontally regular environments, the forward button is visible and
// disabled.
id<GREYMatcher> disabledForwardButton = grey_allOf(
- forwardButton(),
+ ForwardButton(),
grey_accessibilityTrait(UIAccessibilityTraitNotEnabled), nil);
[[EarlGrey selectElementWithMatcher:disabledForwardButton]
assertWithMatcher:grey_notNil()];
@@ -438,19 +438,19 @@ class RedirectResponseProvider : public web::DataResponseProvider {
// page not available error.
const GURL badURL("http://www.badurljkljkljklfloofy.com");
[ChromeEarlGrey loadURL:badURL];
- [[EarlGrey selectElementWithMatcher:errorPage()]
+ [[EarlGrey selectElementWithMatcher:ErrorPage()]
assertWithMatcher:grey_notNil()];
// Go back to page 1 by clicking back button.
- [[EarlGrey selectElementWithMatcher:backButton()] performAction:grey_tap()];
- [[EarlGrey selectElementWithMatcher:chrome_test_util::omniboxText(
+ [[EarlGrey selectElementWithMatcher:BackButton()] performAction:grey_tap()];
+ [[EarlGrey selectElementWithMatcher:chrome_test_util::OmniboxText(
forwardURL.GetContent())]
assertWithMatcher:grey_notNil()];
// Go forward to page 2 by calling window.history.forward() and assert that
// the error page is shown.
TapWebViewElementWithId(kForwardHTMLButtonLabel);
- [[EarlGrey selectElementWithMatcher:errorPage()]
+ [[EarlGrey selectElementWithMatcher:ErrorPage()]
assertWithMatcher:grey_notNil()];
}
@@ -474,32 +474,32 @@ class RedirectResponseProvider : public web::DataResponseProvider {
// Navigate back to original URL. This should fire a hashchange event.
std::string backHashChangeContent = "backHashChange";
[self addHashChangeListenerWithContent:backHashChangeContent];
- [[EarlGrey selectElementWithMatcher:backButton()] performAction:grey_tap()];
- [[EarlGrey selectElementWithMatcher:chrome_test_util::omniboxText(
+ [[EarlGrey selectElementWithMatcher:BackButton()] performAction:grey_tap()];
+ [[EarlGrey selectElementWithMatcher:chrome_test_util::OmniboxText(
page1URL.GetContent())]
assertWithMatcher:grey_notNil()];
[[EarlGrey
- selectElementWithMatcher:webViewContainingText(backHashChangeContent)]
+ selectElementWithMatcher:WebViewContainingText(backHashChangeContent)]
assertWithMatcher:grey_notNil()];
// Navigate forward to the new URL. This should fire a hashchange event.
std::string forwardHashChangeContent = "forwardHashChange";
[self addHashChangeListenerWithContent:forwardHashChangeContent];
- [[EarlGrey selectElementWithMatcher:forwardButton()]
+ [[EarlGrey selectElementWithMatcher:ForwardButton()]
performAction:grey_tap()];
[[EarlGrey
- selectElementWithMatcher:chrome_test_util::omniboxText(
+ selectElementWithMatcher:chrome_test_util::OmniboxText(
hashChangedWithHistoryURL.GetContent())]
assertWithMatcher:grey_notNil()];
[[EarlGrey
- selectElementWithMatcher:webViewContainingText(forwardHashChangeContent)]
+ selectElementWithMatcher:WebViewContainingText(forwardHashChangeContent)]
assertWithMatcher:grey_notNil()];
// Load a hash URL directly. This shouldn't fire a hashchange event.
std::string hashChangeContent = "FAIL_loadUrlHashChange";
[self addHashChangeListenerWithContent:hashChangeContent];
[ChromeEarlGrey loadURL:hashChangedWithHistoryURL];
- [[EarlGrey selectElementWithMatcher:webViewContainingText(hashChangeContent)]
+ [[EarlGrey selectElementWithMatcher:WebViewContainingText(hashChangeContent)]
assertWithMatcher:grey_nil()];
}
@@ -519,22 +519,22 @@ class RedirectResponseProvider : public web::DataResponseProvider {
// Tap link to replace the location value.
TapWebViewElementWithId(kHashChangeWithoutHistoryLabel);
[[EarlGrey
- selectElementWithMatcher:chrome_test_util::omniboxText(
+ selectElementWithMatcher:chrome_test_util::OmniboxText(
hashChangedWithoutHistoryURL.GetContent())]
assertWithMatcher:grey_notNil()];
// Tap link to update the location.hash with a new value.
TapWebViewElementWithId(kHashChangeWithHistoryLabel);
[[EarlGrey
- selectElementWithMatcher:chrome_test_util::omniboxText(
+ selectElementWithMatcher:chrome_test_util::OmniboxText(
hashChangedWithHistoryURL.GetContent())]
assertWithMatcher:grey_notNil()];
// Navigate back and verify that the URL that replaced window.location
// has been reached.
- [[EarlGrey selectElementWithMatcher:backButton()] performAction:grey_tap()];
+ [[EarlGrey selectElementWithMatcher:BackButton()] performAction:grey_tap()];
[[EarlGrey
- selectElementWithMatcher:chrome_test_util::omniboxText(
+ selectElementWithMatcher:chrome_test_util::OmniboxText(
hashChangedWithoutHistoryURL.GetContent())]
assertWithMatcher:grey_notNil()];
}
@@ -553,7 +553,7 @@ class RedirectResponseProvider : public web::DataResponseProvider {
// Tap link to update location.hash with a new value.
TapWebViewElementWithId(kHashChangeWithHistoryLabel);
[[EarlGrey
- selectElementWithMatcher:chrome_test_util::omniboxText(
+ selectElementWithMatcher:chrome_test_util::OmniboxText(
hashChangedWithHistoryURL.GetContent())]
assertWithMatcher:grey_notNil()];
@@ -561,16 +561,16 @@ class RedirectResponseProvider : public web::DataResponseProvider {
TapWebViewElementWithId(kHashChangeWithHistoryLabel);
// Tap back once to return to original URL.
- [[EarlGrey selectElementWithMatcher:backButton()] performAction:grey_tap()];
- [[EarlGrey selectElementWithMatcher:chrome_test_util::omniboxText(
+ [[EarlGrey selectElementWithMatcher:BackButton()] performAction:grey_tap()];
+ [[EarlGrey selectElementWithMatcher:chrome_test_util::OmniboxText(
page1URL.GetContent())]
assertWithMatcher:grey_notNil()];
// Navigate forward and verify the URL.
- [[EarlGrey selectElementWithMatcher:forwardButton()]
+ [[EarlGrey selectElementWithMatcher:ForwardButton()]
performAction:grey_tap()];
[[EarlGrey
- selectElementWithMatcher:chrome_test_util::omniboxText(
+ selectElementWithMatcher:chrome_test_util::OmniboxText(
hashChangedWithHistoryURL.GetContent())]
assertWithMatcher:grey_notNil()];
}
@@ -596,20 +596,20 @@ class RedirectResponseProvider : public web::DataResponseProvider {
web::test::SetUpSimpleHttpServer(responses);
[ChromeEarlGrey loadURL:initialURL];
[ChromeEarlGrey loadURL:originURL];
- [[EarlGrey selectElementWithMatcher:chrome_test_util::omniboxText(
+ [[EarlGrey selectElementWithMatcher:chrome_test_util::OmniboxText(
destinationURL.GetContent())]
assertWithMatcher:grey_notNil()];
// Navigating back takes the user to the new tab page.
- [[EarlGrey selectElementWithMatcher:backButton()] performAction:grey_tap()];
- [[EarlGrey selectElementWithMatcher:chrome_test_util::omniboxText(
+ [[EarlGrey selectElementWithMatcher:BackButton()] performAction:grey_tap()];
+ [[EarlGrey selectElementWithMatcher:chrome_test_util::OmniboxText(
initialURL.GetContent())]
assertWithMatcher:grey_notNil()];
// Navigating forward take the user to destination page.
- [[EarlGrey selectElementWithMatcher:forwardButton()]
+ [[EarlGrey selectElementWithMatcher:ForwardButton()]
performAction:grey_tap()];
- [[EarlGrey selectElementWithMatcher:chrome_test_util::omniboxText(
+ [[EarlGrey selectElementWithMatcher:chrome_test_util::OmniboxText(
destinationURL.GetContent())]
assertWithMatcher:grey_notNil()];
}
@@ -665,29 +665,29 @@ class RedirectResponseProvider : public web::DataResponseProvider {
// to the proper destination.
[ChromeEarlGrey loadURL:indexURL];
TapWebViewElementWithId(redirectLabel);
- [[EarlGrey selectElementWithMatcher:chrome_test_util::omniboxText(
+ [[EarlGrey selectElementWithMatcher:chrome_test_util::OmniboxText(
destinationURL.GetContent())]
assertWithMatcher:grey_notNil()];
// Navigate to a new URL, navigate back and assert that the resulting page is
// the proper destination.
[ChromeEarlGrey loadURL:lastURL];
- [[EarlGrey selectElementWithMatcher:backButton()] performAction:grey_tap()];
- [[EarlGrey selectElementWithMatcher:chrome_test_util::omniboxText(
+ [[EarlGrey selectElementWithMatcher:BackButton()] performAction:grey_tap()];
+ [[EarlGrey selectElementWithMatcher:chrome_test_util::OmniboxText(
destinationURL.GetContent())]
assertWithMatcher:grey_notNil()];
// Navigate back and assert that the resulting page is the initial index.
- [[EarlGrey selectElementWithMatcher:backButton()] performAction:grey_tap()];
- [[EarlGrey selectElementWithMatcher:chrome_test_util::omniboxText(
+ [[EarlGrey selectElementWithMatcher:BackButton()] performAction:grey_tap()];
+ [[EarlGrey selectElementWithMatcher:chrome_test_util::OmniboxText(
indexURL.GetContent())]
assertWithMatcher:grey_notNil()];
// Navigate forward and assert the the resulting page is the proper
// destination.
- [[EarlGrey selectElementWithMatcher:forwardButton()]
+ [[EarlGrey selectElementWithMatcher:ForwardButton()]
performAction:grey_tap()];
- [[EarlGrey selectElementWithMatcher:chrome_test_util::omniboxText(
+ [[EarlGrey selectElementWithMatcher:chrome_test_util::OmniboxText(
destinationURL.GetContent())]
assertWithMatcher:grey_notNil()];
}
« no previous file with comments | « ios/chrome/browser/web/js_print_egtest.mm ('k') | ios/chrome/browser/web/progress_indicator_egtest.mm » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698