Fix first run ui hanging. The threads need to get started earlier....

Fix first run ui hanging.  The threads need to get started earlier.

BUG=26614
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=31046

[kuchhal, 2009-11-05 00:10:06]

lgtm.

On Wed, Nov 4, 2009 at 4:03 PM, <jam@chromium.org> wrote:

> Reviewers: kuchhal,
>
> Description:
> Fix first run ui hanging.  The threads need to get started earlier.
>
> BUG=26614
>
> Please review this at http://codereview.chromium.org/360035
>
> SVN Base: svn://chrome-svn/chrome/trunk/src/
>
> Affected files:
>  M     chrome/browser/browser_main.cc
>
>
> Index: chrome/browser/browser_main.cc
> ===================================================================
> --- chrome/browser/browser_main.cc      (revision 30973)
> +++ chrome/browser/browser_main.cc      (working copy)
> @@ -620,6 +620,13 @@
>     }
>   }
>
> +  // Create the child threads.  We need to do this since
> ChromeThread::PostTask
> +  // silently deletes a posted task if the target message loop isn't
> created.
> +  // Note: must be done before FirstRun code is run.
> +  browser_process->db_thread();
> +  browser_process->file_thread();
> +  browser_process->io_thread();
> +
>   // Importing other browser settings is done in a browser-like process
>   // that exits when this task has finished.
>  #if defined(OS_WIN)
> @@ -701,12 +708,6 @@
>   net::EnsureWinsockInit();
>  #endif  // defined(OS_WIN)
>
> -  // Create the child threads.  We need to do this since
> ChromeThread::PostTask
> -  // silently deletes a posted task if the target message loop isn't
> created.
> -  browser_process->db_thread();
> -  browser_process->file_thread();
> -  browser_process->io_thread();
> -
>   // Initialize and maintain DNS prefetcher module.
>   chrome_browser_net::DnsPrefetcherInit dns_prefetch(user_prefs,
> local_state);
>
>
>
>