Let chrome_prefs handle Preferences initialization from master_preferences.

chrome/browser/first_run/first_run.h

Index: chrome/browser/first_run/first_run.h
diff --git a/chrome/browser/first_run/first_run.h b/chrome/browser/first_run/first_run.h
index 58911f3062a8d4a4a1282b9ee076ddd1b5fb619d..412fef610aad084dc6fd5d3de77b65fe772c98d2 100644
--- a/chrome/browser/first_run/first_run.h
+++ b/chrome/browser/first_run/first_run.h
@@ -160,16 +160,16 @@ uint16 auto_import_state();
 // Set a master preferences file path that overrides platform defaults.
 void SetMasterPrefsPathForTesting(const base::FilePath& master_prefs);
 
-// The master preferences is a JSON file with the same entries as the
+// The master_preferences is a JSON file with the same entries as the
 // 'Default\Preferences' file. This function locates this file from a standard
-// location and processes it so it becomes the default preferences in the
-// profile pointed to by |user_data_dir|. After processing the file, the
-// function returns a value from the ProcessMasterPreferencesResult enum,
-// indicating whether the first run flow should be shown, skipped, or whether
-// the browser should exit.
+// location, processes it, and hands it to ProfileImpl so it becomes the default

[erikwright (departed), 2014/02/04 20:55:28]

I find "hands it to ProfileImpl" to be an implementation detail.

[gab, 2014/02/04 22:32:11]

Done.

+// preferences in the profile pointed to by |user_data_dir|. After processing
+// the file, this function returns a value from the
+// ProcessMasterPreferencesResult enum, indicating whether the first run flow
+// should be shown, skipped, or whether the browser should exit.
 //
-// This function destroys any existing prefs file and it is meant to be
-// invoked only on first run.
+// This function overwrites any existing Preferences file and is only meant to
+// be invoked on first run.
 //
 // See chrome/installer/util/master_preferences.h for a description of
 // 'master_preferences' file.