diff options
| author | marshall@chromium.org <marshall@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-02-06 16:05:13 +0000 |
|---|---|---|
| committer | marshall@chromium.org <marshall@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-02-06 16:05:13 +0000 |
| commit | 6f2b39bccfbb74a7633deff531a33f03efe9accf (patch) | |
| tree | 4e27e58cc514dd3d553f790bb0ca299b33245e2c /chrome/app/chrome_main_delegate.cc | |
| parent | b3cc0c25614566f1261b9c3759d6a076e327fd6c (diff) | |
| download | chromium_src-6f2b39bccfbb74a7633deff531a33f03efe9accf.zip chromium_src-6f2b39bccfbb74a7633deff531a33f03efe9accf.tar.gz chromium_src-6f2b39bccfbb74a7633deff531a33f03efe9accf.tar.bz2 | |
Support sharing of ContentMain and BrowserMain code with embedded use cases.
For the browser use case it is convenient to have a single ContentMain entry point function that handles all initialization, run and shutdown. For embedded use cases it is often necessary to integrate with existing application message loops where initialization and shutdown must be handled separately.
To support sharing of this code the following changes were required:
1. Refactor the ContentMain function to create a ContentMainRunner class containing separate initialization, run and shutdown functions.
2. Refactor the BrowserMain function and BrowserMainLoop class to create a BrowserMainRunner class containing separate initialization, run and shutdown functions.
3. Add a new BrowserMainParts::GetMainMessageLoop method. This is necessary to support creation of a custom MessageLoop implementation while sharing BrowserMainRunner initialization and shutdown code.
BUG=112507
TEST=none
Review URL: https://chromiumcodereview.appspot.com/9190018
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@120574 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/app/chrome_main_delegate.cc')
| -rw-r--r-- | chrome/app/chrome_main_delegate.cc | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/chrome/app/chrome_main_delegate.cc b/chrome/app/chrome_main_delegate.cc index ceabdb8..a742401 100644 --- a/chrome/app/chrome_main_delegate.cc +++ b/chrome/app/chrome_main_delegate.cc @@ -29,9 +29,7 @@ #include "chrome/plugin/chrome_content_plugin_client.h" #include "chrome/renderer/chrome_content_renderer_client.h" #include "chrome/utility/chrome_content_utility_client.h" -#include "content/app/content_main.h" #include "content/common/content_counters.h" -#include "content/public/app/content_main_delegate.h" #include "content/public/browser/render_process_host.h" #include "content/public/common/content_client.h" #include "content/public/common/content_paths.h" @@ -699,8 +697,7 @@ int ChromeMainDelegate::RunProcess( return kMainFunctions[i].function(main_function_params); } - NOTREACHED() << "Unknown process type: " << process_type; - return 1; + return -1; } void ChromeMainDelegate::ProcessExiting(const std::string& process_type) { |