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

Unified Diff: content/browser/browser_main_loop.h

Issue 9150016: Move creation and ownership of ResourceDispatcherHost and PluginService to content. This gives a ... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: fix chromeos ui_tests Created 8 years, 11 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: content/browser/browser_main_loop.h
===================================================================
--- content/browser/browser_main_loop.h (revision 117096)
+++ content/browser/browser_main_loop.h (working copy)
@@ -13,6 +13,7 @@
class CommandLine;
class HighResolutionTimerManager;
class MessageLoop;
+class ResourceDispatcherHost;
class SystemMessageWindowWin;
namespace base {
@@ -58,6 +59,9 @@
void InitializeMainThread();
+ // Called right after the browser threads have been started.
+ void BrowserThreadsStarted();
+
// Members initialized on construction ---------------------------------------
const content::MainFunctionParams& parameters_;
const CommandLine& parsed_command_line_;
@@ -80,6 +84,9 @@
// This must get destroyed before other threads that are created in parts_.
scoped_ptr<BrowserThreadImpl> main_thread_;
+ // Members initialized in |BrowserThreadsStarted()| --------------------------
+ scoped_ptr<ResourceDispatcherHost> resource_dispatcher_host_;
+
// Members initialized in |RunMainMessageLoopParts()| ------------------------
scoped_ptr<BrowserProcessSubThread> db_thread_;
scoped_ptr<WebKitThread> webkit_thread_;

Powered by Google App Engine
This is Rietveld 408576698