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

Unified Diff: chrome/browser/ui/browser_init.h

Issue 8093016: Show sync promo at startup (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: fix build error Created 9 years, 2 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
« no previous file with comments | « chrome/browser/ui/browser_browsertest.cc ('k') | chrome/browser/ui/browser_init.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/ui/browser_init.h
diff --git a/chrome/browser/ui/browser_init.h b/chrome/browser/ui/browser_init.h
index 24466ea0e2d13c12f5f31765d42c55d5669de65e..46ae72faafdf9d2c64983fd38ce5b10908daa59f 100644
--- a/chrome/browser/ui/browser_init.h
+++ b/chrome/browser/ui/browser_init.h
@@ -24,6 +24,15 @@ class TabContentsWrapper;
// initialize the profile.
class BrowserInit {
public:
+ enum IsProcessStartup {
+ IS_NOT_PROCESS_STARTUP,
+ IS_PROCESS_STARTUP
+ };
+ enum IsFirstRun {
+ IS_NOT_FIRST_RUN,
+ IS_FIRST_RUN
+ };
+
BrowserInit();
~BrowserInit();
@@ -64,8 +73,12 @@ class BrowserInit {
// be the command line passed to this process. |cur_dir| can be empty, which
// implies that the directory of the executable should be used.
// |process_startup| indicates whether this is the first browser.
- bool LaunchBrowser(const CommandLine& command_line, Profile* profile,
- const FilePath& cur_dir, bool process_startup,
+ // |is_first_run| indicates that this is a new profile.
+ bool LaunchBrowser(const CommandLine& command_line,
+ Profile* profile,
+ const FilePath& cur_dir,
+ IsProcessStartup is_process_startup,
+ IsFirstRun is_first_run,
int* return_code);
// LaunchWithProfile ---------------------------------------------------------
@@ -97,10 +110,14 @@ class BrowserInit {
// There are two ctors. The first one implies a NULL browser_init object
// and thus no access to distribution-specific first-run behaviors. The
// second one is always called when the browser starts even if it is not
- // the first run.
- LaunchWithProfile(const FilePath& cur_dir, const CommandLine& command_line);
- LaunchWithProfile(const FilePath& cur_dir, const CommandLine& command_line,
- BrowserInit* browser_init);
+ // the first run. |is_first_run| indicates that this is a new profile.
+ LaunchWithProfile(const FilePath& cur_dir,
+ const CommandLine& command_line,
+ IsFirstRun is_first_run);
+ LaunchWithProfile(const FilePath& cur_dir,
+ const CommandLine& command_line,
+ BrowserInit* browser_init,
+ IsFirstRun is_first_run);
~LaunchWithProfile();
// Creates the necessary windows for startup. Returns true on success,
@@ -213,6 +230,7 @@ class BrowserInit {
const CommandLine& command_line_;
Profile* profile_;
BrowserInit* browser_init_;
+ bool is_first_run_;
DISALLOW_COPY_AND_ASSIGN(LaunchWithProfile);
};
« no previous file with comments | « chrome/browser/ui/browser_browsertest.cc ('k') | chrome/browser/ui/browser_init.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698