Fix the first_run regression issue related to skip_first_run_ui setting processing.

chrome/browser/first_run/first_run_internal.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CHROME_BROWSER_FIRST_RUN_FIRST_RUN_INTERNAL_H_
 #define CHROME_BROWSER_FIRST_RUN_FIRST_RUN_INTERNAL_H_
 #pragma once
 
 #include <vector>
 
@@ skipping 45 matching lines @@
 // Sets up master preferences by preferences passed by installer.
 void SetupMasterPrefsFromInstallPrefs(
     MasterPrefs* out_prefs,
     installer::MasterPreferences* install_prefs);
 
 void SetShowWelcomePagePrefIfNeeded(
     installer::MasterPreferences* install_prefs);
 
 void SetDefaultBrowser(installer::MasterPreferences* install_prefs);
 
-// Returns true if the first-run-ui process should be skipped.
+// Returns true if first run ui should be skipped, which is the case that
+// skip_first_run_ui setting is set to true. In the case the setting is
+// not found or specified, it returns false by default.
 bool SkipFirstRunUI(installer::MasterPreferences* install_prefs);
 
 // Sets ping_delay.
 void SetRLZPref(first_run::MasterPrefs* out_prefs,
                 installer::MasterPreferences* install_prefs);
 
 // -- Platform-specific functions --
 
 // Gives the full path to the sentinel file. The file might not exist.
 // This function has a common implementation on OS_POSIX and a windows specific
@@ skipping 34 matching lines @@
                                    const CommandLine& cmdline);
 
 #if !defined(OS_WIN)
 bool ImportBookmarks(const FilePath& import_bookmarks_path);
 #endif
 
 }  // namespace internal
 }  // namespace first_run
 
 #endif  // CHROME_BROWSER_FIRST_RUN_FIRST_RUN_INTERNAL_H_