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

Unified Diff: ios/chrome/browser/ui/settings/settings_egtest.mm

Issue 2894883004: Removed IsNativeAppLauncherEnabled() experiment (Closed)
Patch Set: rebase Created 3 years, 6 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
Index: ios/chrome/browser/ui/settings/settings_egtest.mm
diff --git a/ios/chrome/browser/ui/settings/settings_egtest.mm b/ios/chrome/browser/ui/settings/settings_egtest.mm
index 7ba1703d3c95c25c07f0a6591a74348eeb7f998b..bf4a1fe488be3ae68ac3816482c6eb443b5f903b 100644
--- a/ios/chrome/browser/ui/settings/settings_egtest.mm
+++ b/ios/chrome/browser/ui/settings/settings_egtest.mm
@@ -21,7 +21,6 @@
#import "ios/chrome/app/main_controller.h"
#include "ios/chrome/browser/browser_state/chrome_browser_state.h"
#include "ios/chrome/browser/content_settings/host_content_settings_map_factory.h"
-#include "ios/chrome/browser/experimental_flags.h"
#include "ios/chrome/browser/pref_names.h"
#import "ios/chrome/browser/ui/browser_view_controller.h"
#import "ios/chrome/browser/ui/settings/clear_browsing_data_collection_view_controller.h"
@@ -131,10 +130,6 @@ id<GREYMatcher> SearchEngineButton() {
id<GREYMatcher> AutofillButton() {
return ButtonWithAccessibilityLabelId(IDS_IOS_AUTOFILL);
}
-// Matcher for the Google Apps cell on the main Settings screen.
-id<GREYMatcher> GoogleAppsButton() {
- return ButtonWithAccessibilityLabelId(IDS_IOS_GOOGLE_APPS_SM_SETTINGS);
-}
// Matcher for the Google Chrome cell on the main Settings screen.
id<GREYMatcher> GoogleChromeButton() {
return ButtonWithAccessibilityLabelId(IDS_IOS_PRODUCT_NAME);
@@ -881,17 +876,6 @@ bool IsCertificateCleared() {
[self closeSubSettingsMenu];
}
-// Verifies the UI elements are accessible on the Google Apps page.
-- (void)testAccessibilityOnGoogleApps {
- // TODO(crbug/711511): Remove when Native App Launcher is full deprecated.
- if (!experimental_flags::IsNativeAppLauncherEnabled())
- return;
- [ChromeEarlGreyUI openSettingsMenu];
- [ChromeEarlGreyUI tapSettingsMenuButton:GoogleAppsButton()];
- chrome_test_util::VerifyAccessibilityForCurrentScreen();
- [self closeSubSettingsMenu];
-}
-
// Verifies the UI elements are accessible on the About Chrome page.
- (void)testAccessibilityOnGoogleChrome {
[ChromeEarlGreyUI openSettingsMenu];

Powered by Google App Engine
This is Rietveld 408576698