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

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

Issue 2714813002: [iOS] Add Request Mobile Site cell to tools menu (Closed)
Patch Set: Implement "Request Mobile Site" UI Created 3 years, 10 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.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 d1f4d507089665f24bb2db7b1559e2f53f6b8a6f..246cfd76aaba55785f9acf75f9061ec3f521eccc 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
@@ -66,12 +66,21 @@ extern NSString* const kToolsMenuSuggestionsId;
// can tell that the Tools menu is scrollable.
- (CGFloat)optimalHeight:(CGFloat)suggestedHeight;
+// Hide a menu item by IDC value.
+- (void)hideItemWithTag:(NSInteger)tag;
+
// Enable or disable menu item by IDC value.
- (void)setItemEnabled:(BOOL)enabled withTag:(NSInteger)tag;
// Called when the current tab loading state changes.
- (void)setIsTabLoading:(BOOL)isTabLoading;
+// Informs tools popup menu to hide "Request Desktop Site" or
+// "Request Mobile Site", note that only one of them is expected to be called
+// at any time.
+- (void)hideRequestDesktopSite;
+- (void)hideRequestMobileSite;
+
// TODO(stuartmorgan): Should the set of options that are passed in to the
// constructor just have the ability to specify whether commands should be
// enabled or disabled rather than having these individual setters? crbug/228506
@@ -88,6 +97,9 @@ extern NSString* const kToolsMenuSuggestionsId;
// Informs tools popup menu whether "Request Desktop Site" can be enabled.
- (void)setCanUseDesktopUserAgent:(BOOL)value;
+// Informs tools popup menu whether "Request Mobile Site" can be enabled.
+- (void)setCanUseMobileUserAgent:(BOOL)value;
+
- (void)animateContentIn;
- (void)hideContent;

Powered by Google App Engine
This is Rietveld 408576698