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

Unified Diff: mojo/shell/context.cc

Issue 395163002: Bundle the network service in the shell on Android. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebase Created 6 years, 5 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
« no previous file with comments | « mojo/services/profile/profile_service_impl.cc ('k') | mojo/shell/network_service_loader.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: mojo/shell/context.cc
diff --git a/mojo/shell/context.cc b/mojo/shell/context.cc
index b4fe8297671d20e368151946a887f2aa4735f02d..a5e5f42666215deb17b372d5cbcdd737b926956c 100644
--- a/mojo/shell/context.cc
+++ b/mojo/shell/context.cc
@@ -18,7 +18,6 @@
#include "mojo/shell/dynamic_service_loader.h"
#include "mojo/shell/in_process_dynamic_service_runner.h"
#include "mojo/shell/out_of_process_dynamic_service_runner.h"
-#include "mojo/shell/profile_service_loader.h"
#include "mojo/shell/switches.h"
#include "mojo/spy/spy.h"
@@ -26,6 +25,10 @@
#include "mojo/shell/dbus_service_loader_linux.h"
#endif // defined(OS_LINUX)
+#if defined(OS_ANDROID)
+#include "mojo/shell/network_service_loader.h"
+#endif // defined(OS_ANDROID)
+
#if defined(USE_AURA)
#include "mojo/shell/view_manager_loader.h"
#endif
@@ -120,14 +123,22 @@ Context::Context()
"dbus");
#endif // defined(OS_LINUX)
- service_manager_.SetLoaderForURL(
- scoped_ptr<ServiceLoader>(new ProfileServiceLoader()),
- GURL("mojo:profile_service"));
-
if (cmdline->HasSwitch(switches::kSpy)) {
spy_.reset(new mojo::Spy(&service_manager_,
cmdline->GetSwitchValueASCII(switches::kSpy)));
}
+
+#if defined(OS_ANDROID)
+ // On android, the network service is bundled with the shell because the
+ // network stack depends on the android runtime.
+ service_manager_.SetLoaderForURL(
+ scoped_ptr<ServiceLoader>(
+ new BackgroundServiceLoader(
+ scoped_ptr<ServiceLoader>(new NetworkServiceLoader()),
+ "network_service",
+ base::MessageLoop::TYPE_IO)),
+ GURL("mojo:mojo_network_service"));
+#endif
}
Context::~Context() {
« no previous file with comments | « mojo/services/profile/profile_service_impl.cc ('k') | mojo/shell/network_service_loader.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698