[iOS] Add Request Mobile Site cell to tools menu

ios/chrome/browser/ui/tools_menu/tools_menu_view_controller.h

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef IOS_CHROME_BROWSER_UI_TOOLS_MENU_TOOLS_MENU_VIEW_CONTROLLER_H_
 #define IOS_CHROME_BROWSER_UI_TOOLS_MENU_TOOLS_MENU_VIEW_CONTROLLER_H_
 
 #import <UIKit/UIKit.h>
 
 @class ToolsMenuContext;
@@ skipping 38 matching lines @@
 @interface ToolsMenuViewController : UIViewController
 // Keeps track of the state (Bookmarked or not) of the current visible page.
 // This is used to alter the state of the popup menu (i.e. Add/Edit bookmark).
 @property(nonatomic, assign) BOOL isCurrentPageBookmarked;
 @property(nonatomic, assign) BOOL isTabLoading;
 // The tool button to be shown hovering above the popup.
 @property(nonatomic, readonly) UIButton* toolsButton;
 
 @property(nonatomic, assign) id<ToolsPopupTableDelegate> delegate;
 
+// The user agent type used to load the current page, which also decides the
+// visibility and enability of "Request Desktop Site" and "Request Mobile Site"
+// cells in the tools popup menu.
+@property(nonatomic, assign) web::UserAgentType userAgentType;
+
 // Initializes the Tools popup menu.
 - (void)initializeMenu:(ToolsMenuContext*)context;
 
 // Returns the optimal height needed to display the menu items.
 // The height returned is usually less than the |suggestedHeight| unless
 // the last row of the menu puts the height just over the |suggestedHeight|.
 // If the Tools menu items is taller than the |suggestedHeight| by at least
 // one menu item, the last visible menu item will be shown partially so user
 // can tell that the Tools menu is scrollable.
 - (CGFloat)optimalHeight:(CGFloat)suggestedHeight;
 
 // 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;
 
+// Called when the current tab user agent type changes.
+- (void)setUserAgentType:(web::UserAgentType)type;

[kkhorimoto, 2017/02/27 23:44:24]

This isn't necessary since it's automatically created for your property.

[liaoyuke, 2017/02/28 02:01:00]

Done.

+
 // 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
 // Informs tools popup menu whether "Find In Page..." command should be
 // enabled.
 - (void)setCanShowFindBar:(BOOL)enabled;
 
 // Informs tools popup menu whether "Share..." command should be enabled.
 - (void)setCanShowShareMenu:(BOOL)enabled;
 
 // 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;
 
 @end
 
 #endif  // IOS_CHROME_BROWSER_UI_TOOLS_MENU_TOOLS_MENU_VIEW_CONTROLLER_H_