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

Unified Diff: chrome/browser/ui/startup/startup_browser_creator.h

Issue 10702097: Adds browser preferences to configure first-run flow on Windows 8. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Addressed grt@'s remarks. Created 8 years, 5 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: chrome/browser/ui/startup/startup_browser_creator.h
diff --git a/chrome/browser/ui/startup/startup_browser_creator.h b/chrome/browser/ui/startup/startup_browser_creator.h
index 96bdbedb700878d2c68cf4cbb0f24356907b6ac4..6a4dc55f0fa79cf3b6b789c35a8ad1b0187a10bc 100644
--- a/chrome/browser/ui/startup/startup_browser_creator.h
+++ b/chrome/browser/ui/startup/startup_browser_creator.h
@@ -83,6 +83,14 @@ class StartupBrowserCreator {
const CommandLine& command_line,
Profile* profile);
+ void set_suppressed_set_as_default_dialog(bool new_value) {
+ suppressed_set_as_default_dialog_ = new_value;
+ }
+
+ bool get_suppressed_set_as_default_dialog() const {
sky 2012/07/25 16:09:57 Inline accessors should match the name of the fiel
motek. 2012/07/26 12:11:04 Sure. The original came about to avoid set_set set
+ return suppressed_set_as_default_dialog_;
+ }
+
private:
friend class CloudPrintProxyPolicyTest;
friend class CloudPrintProxyPolicyStartupTest;
@@ -118,6 +126,11 @@ class StartupBrowserCreator {
// Additional tabs to open during first run.
std::vector<GURL> first_run_tabs_;
+ // True if the set-as-default dialog has been explicitly supressed.
+ // This information is used to allow the default browser prompt to show on
+ // first-run when the dialog has been suppressed.
+ bool suppressed_set_as_default_dialog_;
+
// True if we have already read and reset the preference kWasRestarted. (A
// member variable instead of a static variable inside WasRestarted because
// of testing.)

Powered by Google App Engine
This is Rietveld 408576698