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

Unified Diff: ios/clean/chrome/browser/ui/actions/settings_actions.h

Issue 2810603002: [ios clean] Adds ToolsMenu to TabGrid (Closed)
Patch Set: Rebase Created 3 years, 8 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/clean/chrome/browser/ui/actions/settings_actions.h
diff --git a/ios/clean/chrome/browser/ui/actions/settings_actions.h b/ios/clean/chrome/browser/ui/actions/settings_actions.h
index 0a914f78cba9dd12ef36408124292dedf0dc0bb9..d7e664af80e3c9c379cfd4efdf4a6960292cf776 100644
--- a/ios/clean/chrome/browser/ui/actions/settings_actions.h
+++ b/ios/clean/chrome/browser/ui/actions/settings_actions.h
@@ -6,14 +6,17 @@
#define IOS_CLEAN_CHROME_BROWSER_UI_ACTIONS_SETTINGS_ACTIONS_H_
#import <Foundation/Foundation.h>
-
-// Target/Action methods relating to the Settings UI.
-// (Actions should only be used to communicate into or between the View
-// Controller layer).
+// HACK: This whole file and the two remaining action calls need to
+// dissapear. We just need to decide how VC's will deal with tap/actions that
+// were set by the mediator or some other file. Target/Action methods relating
+// to the Settings UI. (Actions should only be used to communicate into or
+// between the View Controller layer).
@protocol SettingsActions
@optional
// Show the Settings UI over whatever UI is currently active.
- (void)showSettings:(id)sender;
+// Shows the ToolsMenu from the TabGrid button. This way we can access settings.
+- (void)showToolsMenu:(id)sender;
@end
#endif // IOS_CLEAN_CHROME_BROWSER_UI_ACTIONS_SETTINGS_ACTIONS_H_

Powered by Google App Engine
This is Rietveld 408576698