First part of the pre-set bookmarks feature

- master prefs format (just url)
- defer actual bookmark fu, because early on the bookmarkmodel and other services are not yet running.
- cleanup master prefs test, too much code duplication
- cleanup ProcessMasterPreferences it had too many params

BUG=32728
TEST=unit test included

Review URL: http://codereview.chromium.org/660116

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@40150 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 9 insertions, 0 deletions

diff --git a/chrome/browser/browser_init.h b/chrome/browser/browser_init.h
index c427741..28653f1 100644
--- a/chrome/browser/browser_init.h
+++ b/chrome/browser/browser_init.h
@@ -31,6 +31,13 @@ class BrowserInit {
     first_run_tabs_.push_back(url);
   }
 
+  // Adds a url that will become a bookmark during first run, just as if the
+  // user had done it. A network request is generated. Icon and title will
+  // be provided by the server.
+  void AddDefaultBookmark(const GURL& url) {
+    default_bookmarks_.push_back(url);
+  }
+
   // This function is equivalent to ProcessCommandLine but should only be
   // called during actual process startup.
   bool Start(const CommandLine& cmd_line, const std::wstring& cur_dir,
@@ -150,6 +157,8 @@ class BrowserInit {
 
   // Additional tabs to open during first run.
   std::vector<GURL> first_run_tabs_;
+  // Additional bookmarks to set during first run.
+  std::vector<GURL> default_bookmarks_;
 
   DISALLOW_COPY_AND_ASSIGN(BrowserInit);
 };