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

Side by Side Diff: chrome/browser/ui/browser_command_controller.h

Issue 2689383002: Commands should have consistent behaviors across different modes
Patch Set: 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 unified diff | Download patch
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef CHROME_BROWSER_UI_BROWSER_COMMAND_CONTROLLER_H_ 5 #ifndef CHROME_BROWSER_UI_BROWSER_COMMAND_CONTROLLER_H_
6 #define CHROME_BROWSER_UI_BROWSER_COMMAND_CONTROLLER_H_ 6 #define CHROME_BROWSER_UI_BROWSER_COMMAND_CONTROLLER_H_
7 7
8 #include <vector> 8 #include <vector>
9 9
10 #include "base/macros.h" 10 #include "base/macros.h"
(...skipping 52 matching lines...) Expand 10 before | Expand all | Expand 10 after
63 void PrintingStateChanged(); 63 void PrintingStateChanged();
64 void LoadingStateChanged(bool is_loading, bool force); 64 void LoadingStateChanged(bool is_loading, bool force);
65 void ExtensionStateChanged(); 65 void ExtensionStateChanged();
66 66
67 // Shared state updating: these functions are static and public to share with 67 // Shared state updating: these functions are static and public to share with
68 // outside code. 68 // outside code.
69 69
70 // Updates the open-file state. 70 // Updates the open-file state.
71 static void UpdateOpenFileState(CommandUpdater* command_updater); 71 static void UpdateOpenFileState(CommandUpdater* command_updater);
72 72
73 // Update commands whose state depends on incognito mode availability and that 73 // Update commands whose state depends on incognito mode availability and that
msw 2017/02/14 22:34:50 nit: update comment
Hzj_jie 2017/02/15 02:00:49 Done.
74 // only depend on the profile. 74 // only depend on the profile.
75 static void UpdateSharedCommandsForIncognitoAvailability( 75 static void UpdateSharedCommandsForIncognitoAvailability(
76 CommandUpdater* command_updater, 76 CommandUpdater* command_updater,
77 Profile* profile); 77 Profile* profile,
78 bool is_fullscreen);
78 79
79 private: 80 private:
80 class InterstitialObserver; 81 class InterstitialObserver;
81 82
82 // Overridden from CommandUpdaterDelegate: 83 // Overridden from CommandUpdaterDelegate:
83 void ExecuteCommandWithDisposition(int id, WindowOpenDisposition disposition) 84 void ExecuteCommandWithDisposition(int id, WindowOpenDisposition disposition)
84 override; 85 override;
85 86
86 // Overridden from TabStripModelObserver: 87 // Overridden from TabStripModelObserver:
87 void TabInsertedAt(TabStripModel* tab_strip_model, 88 void TabInsertedAt(TabStripModel* tab_strip_model,
(...skipping 104 matching lines...) Expand 10 before | Expand all | Expand 10 after
192 PrefChangeRegistrar profile_pref_registrar_; 193 PrefChangeRegistrar profile_pref_registrar_;
193 PrefChangeRegistrar local_pref_registrar_; 194 PrefChangeRegistrar local_pref_registrar_;
194 BooleanPrefMember pref_signin_allowed_; 195 BooleanPrefMember pref_signin_allowed_;
195 196
196 DISALLOW_COPY_AND_ASSIGN(BrowserCommandController); 197 DISALLOW_COPY_AND_ASSIGN(BrowserCommandController);
197 }; 198 };
198 199
199 } // namespace chrome 200 } // namespace chrome
200 201
201 #endif // CHROME_BROWSER_UI_BROWSER_COMMAND_CONTROLLER_H_ 202 #endif // CHROME_BROWSER_UI_BROWSER_COMMAND_CONTROLLER_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698