diff options
| author | jam@chromium.org <jam@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-10-28 20:31:58 +0000 |
|---|---|---|
| committer | jam@chromium.org <jam@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-10-28 20:31:58 +0000 |
| commit | fae20799882ef797ae9cb5a169c3a60680fa66c2 (patch) | |
| tree | aa54776c6e6b1b95fb1ad7737440c82c022eff01 /chrome/browser/dom_ui/new_tab_ui.cc | |
| parent | 419834aa659b728039d8dd398f7156b823f572d8 (diff) | |
| download | chromium_src-fae20799882ef797ae9cb5a169c3a60680fa66c2.zip chromium_src-fae20799882ef797ae9cb5a169c3a60680fa66c2.tar.gz chromium_src-fae20799882ef797ae9cb5a169c3a60680fa66c2.tar.bz2 | |
First of several patches to get rid of MessageLoop caching now that we have ChromeThread::PostTask.
BUG=25354
Review URL: http://codereview.chromium.org/342020
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@30383 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/dom_ui/new_tab_ui.cc')
| -rw-r--r-- | chrome/browser/dom_ui/new_tab_ui.cc | 39 |
1 files changed, 19 insertions, 20 deletions
diff --git a/chrome/browser/dom_ui/new_tab_ui.cc b/chrome/browser/dom_ui/new_tab_ui.cc index 76c44ae..6d34031 100644 --- a/chrome/browser/dom_ui/new_tab_ui.cc +++ b/chrome/browser/dom_ui/new_tab_ui.cc @@ -18,7 +18,7 @@ #include "base/string_piece.h" #include "base/thread.h" #include "chrome/browser/browser.h" -#include "chrome/browser/browser_process.h" +#include "chrome/browser/chrome_thread.h" #include "chrome/browser/dom_ui/dom_ui_theme_source.h" #include "chrome/browser/dom_ui/most_visited_handler.h" #include "chrome/browser/dom_ui/new_tab_page_sync_handler.h" @@ -548,9 +548,10 @@ NewTabUI::NewTabUI(TabContents* contents) IncognitoTabHTMLSource* html_source = new IncognitoTabHTMLSource( GetProfile()->GetPrefs()->GetBoolean(prefs::kShowBookmarkBar)); - - g_browser_process->io_thread()->message_loop()->PostTask(FROM_HERE, - NewRunnableMethod(&chrome_url_data_manager, + ChromeThread::PostTask( + ChromeThread::IO, FROM_HERE, + NewRunnableMethod( + &chrome_url_data_manager, &ChromeURLDataManager::AddDataSource, html_source)); } else { @@ -569,15 +570,14 @@ NewTabUI::NewTabUI(TabContents* contents) AddMessageHandler((new NewTabPageSetHomepageHandler())->Attach(this)); - // In testing mode there may not be an I/O thread. - if (g_browser_process->io_thread()) { - InitializeCSSCaches(); - NewTabHTMLSource* html_source = new NewTabHTMLSource(GetProfile()); - g_browser_process->io_thread()->message_loop()->PostTask(FROM_HERE, - NewRunnableMethod(&chrome_url_data_manager, - &ChromeURLDataManager::AddDataSource, - html_source)); - } + InitializeCSSCaches(); + NewTabHTMLSource* html_source = new NewTabHTMLSource(GetProfile()); + ChromeThread::PostTask( + ChromeThread::IO, FROM_HERE, + NewRunnableMethod( + &chrome_url_data_manager, + &ChromeURLDataManager::AddDataSource, + html_source)); } // Listen for theme installation. @@ -614,13 +614,12 @@ void NewTabUI::Observe(NotificationType type, } void NewTabUI::InitializeCSSCaches() { - // In testing mode there may not be an I/O thread. - if (g_browser_process->io_thread()) { - g_browser_process->io_thread()->message_loop()->PostTask(FROM_HERE, - NewRunnableMethod(&chrome_url_data_manager, - &ChromeURLDataManager::AddDataSource, - new DOMUIThemeSource(GetProfile()))); - } + ChromeThread::PostTask( + ChromeThread::IO, FROM_HERE, + NewRunnableMethod( + &chrome_url_data_manager, + &ChromeURLDataManager::AddDataSource, + new DOMUIThemeSource(GetProfile()))); } // static |