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

Unified Diff: ios/chrome/browser/ui/tools_menu/tools_menu_view_controller.mm

Issue 2643753002: Add the suggestions UI in chrome (Closed)
Patch Set: Convert to ARC 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
Index: ios/chrome/browser/ui/tools_menu/tools_menu_view_controller.mm
diff --git a/ios/chrome/browser/ui/tools_menu/tools_menu_view_controller.mm b/ios/chrome/browser/ui/tools_menu/tools_menu_view_controller.mm
index 84adea1713b824939d1baac28267bf0f4fbb3371..3cfd1f6d9755d6f669bd70d7728142dcfe1a987b 100644
--- a/ios/chrome/browser/ui/tools_menu/tools_menu_view_controller.mm
+++ b/ios/chrome/browser/ui/tools_menu/tools_menu_view_controller.mm
@@ -53,6 +53,7 @@ NSString* const kToolsMenuReaderMode = @"kToolsMenuReaderMode";
NSString* const kToolsMenuRequestDesktopId = @"kToolsMenuRequestDesktopId";
NSString* const kToolsMenuSettingsId = @"kToolsMenuSettingsId";
NSString* const kToolsMenuHelpId = @"kToolsMenuHelpId";
+NSString* const kToolsMenuSuggestionsUIId = @"kToolsMenuSuggestionsUIId";
namespace {
@@ -120,6 +121,9 @@ static MenuItemInfo itemInfoList[] = {
{ IDS_IOS_TOOLS_MENU_READING_LIST, kToolsMenuReadingListId,
IDC_SHOW_READING_LIST, kToolbarTypeWebAll,
0, [ReadingListMenuViewItem class] },
+ { IDS_IOS_TOOLS_MENU_SUGGESTIONS_UI, kToolsMenuSuggestionsUIId,
+ IDC_SHOW_SUGGESTIONS_UI, kToolbarTypeWebAll,
+ 0, nil },
{ IDS_IOS_TOOLS_MENU_RECENT_TABS, kToolsMenuOtherDevicesId,
IDC_SHOW_OTHER_DEVICES, kToolbarTypeWebAll,
kVisibleNotIncognitoOnly, nil },
@@ -177,6 +181,12 @@ NS_INLINE BOOL ItemShouldBeVisible(const MenuItemInfo& item,
}
}
+ if (item.title_id == IDS_IOS_TOOLS_MENU_SUGGESTIONS_UI) {
+ if (!experimental_flags::IsSuggestionsUIEnabled()) {
marq (ping after 24h) 2017/01/19 16:32:32 Do you need experimental flag checks both here and
gambard 2017/01/19 17:50:12 I used the same pattern as reading list
+ return NO;
+ }
+ }
+
if (item.title_id == IDS_IOS_OPTIONS_REPORT_AN_ISSUE) {
if (!ios::GetChromeBrowserProvider()
->GetUserFeedbackProvider()

Powered by Google App Engine
This is Rietveld 408576698