Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(5095)

Unified Diff: chrome/browser/chrome_browser_main.cc

Issue 2630443003: Add thread checks to NaClBrowser, and make it leaky (Closed)
Patch Set: Update NaClGdbDebugStubTest Created 3 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/chrome_browser_main.cc
diff --git a/chrome/browser/chrome_browser_main.cc b/chrome/browser/chrome_browser_main.cc
index 828945097ff5f5465cc636cfdeb2d39190b83a41..7e171bf377922483638ce67f43ae3ab54abbb679 100644
--- a/chrome/browser/chrome_browser_main.cc
+++ b/chrome/browser/chrome_browser_main.cc
@@ -1662,12 +1662,11 @@ int ChromeBrowserMainParts::PreMainMessageLoopRunImpl() {
content::WebUIControllerFactory::RegisterFactory(
ChromeWebUIControllerFactory::GetInstance());
+#if !defined(DISABLE_NACL)
// NaClBrowserDelegateImpl is accessed inside PostProfileInit().
// So make sure to create it before that.
-#if !defined(DISABLE_NACL)
- NaClBrowserDelegateImpl* delegate =
- new NaClBrowserDelegateImpl(browser_process_->profile_manager());
- nacl::NaClBrowser::SetDelegate(delegate);
+ nacl::NaClBrowser::SetDelegate(base::MakeUnique<NaClBrowserDelegateImpl>(
+ browser_process_->profile_manager()));
#endif // !defined(DISABLE_NACL)
// TODO(stevenjb): Move WIN and MACOSX specific code to appropriate Parts.
« no previous file with comments | « no previous file | chrome/browser/nacl_host/test/gdb_debug_stub_browsertest.cc » ('j') | components/nacl/browser/nacl_browser.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698