Chromium Code Reviews| Index: ios/chrome/browser/ui/tools_menu/tools_menu_view_controller.h |
| diff --git a/ios/chrome/browser/ui/tools_menu/tools_menu_view_controller.h b/ios/chrome/browser/ui/tools_menu/tools_menu_view_controller.h |
| index 1f7958319fd19e23ec2330f66a33ceb6fc098884..5bdd7e77bc0613f677cacbe847e385437c3444fd 100644 |
| --- a/ios/chrome/browser/ui/tools_menu/tools_menu_view_controller.h |
| +++ b/ios/chrome/browser/ui/tools_menu/tools_menu_view_controller.h |
| @@ -7,7 +7,7 @@ |
| #import <UIKit/UIKit.h> |
| -@class ToolsMenuContext; |
| +@class ToolsMenuConfiguration; |
| // TODO(crbug.com/228521): Remove this once the new command/metric handling is |
| // implemented. This is a temporary workaround to allow metrics recording to |
| @@ -54,10 +54,13 @@ extern NSString* const kToolsMenuReadingListId; |
| // The tool button to be shown hovering above the popup. |
| @property(nonatomic, readonly) UIButton* toolsButton; |
| +// Keeps track of the items in tools menu. |
| +@property(nonatomic, retain) NSMutableArray* menuItems; |
|
Eugene But (OOO till 7-30)
2017/03/07 02:26:00
This should not be mutable and it should be a copy
liaoyuke
2017/03/07 06:24:29
Done.
|
| + |
| @property(nonatomic, assign) id<ToolsPopupTableDelegate> delegate; |
| // Initializes the Tools popup menu. |
| -- (void)initializeMenu:(ToolsMenuContext*)context; |
| +- (void)initializeMenuWithConfiguration:(ToolsMenuConfiguration*)configuration; |
| // Returns the optimal height needed to display the menu items. |
| // The height returned is usually less than the |suggestedHeight| unless |
| @@ -86,9 +89,6 @@ extern NSString* const kToolsMenuReadingListId; |
| // Informs tools popup menu whether the switch to reader mode is possible. |
| - (void)setCanUseReaderMode:(BOOL)enabled; |
| -// Informs tools popup menu whether "Request Desktop Site" can be enabled. |
| -- (void)setCanUseDesktopUserAgent:(BOOL)value; |
| - |
| - (void)animateContentIn; |
| - (void)hideContent; |