Chromium Code Reviews| 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() |