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

Unified Diff: mojo/package_manager/package_manager_impl.cc

Issue 1354043002: Revert of Some more minor cleanup to ApplicationManager. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 3 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/mojo_shell.gyp ('k') | mojo/runner/context.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: mojo/package_manager/package_manager_impl.cc
diff --git a/mojo/package_manager/package_manager_impl.cc b/mojo/package_manager/package_manager_impl.cc
index 396b4d197fbbad272ca8b23005f279ac2279cccb..f1a1fc6bfb7be73f21c404c02923cc44b3d572c4 100644
--- a/mojo/package_manager/package_manager_impl.cc
+++ b/mojo/package_manager/package_manager_impl.cc
@@ -10,7 +10,6 @@
#include "mojo/fetcher/switches.h"
#include "mojo/fetcher/update_fetcher.h"
#include "mojo/shell/application_manager.h"
-#include "mojo/shell/connect_util.h"
#include "mojo/shell/query_util.h"
#include "mojo/shell/switches.h"
#include "mojo/util/filename_util.h"
@@ -74,8 +73,8 @@
// LocalFetcher uses the network service to infer MIME types from URLs.
// Skip this for mojo URLs to avoid recursively loading the network service.
if (!network_service_ && !url.SchemeIs("mojo")) {
- shell::ConnectToService(application_manager_,
- GURL("mojo:network_service"), &network_service_);
+ application_manager_->ConnectToService(GURL("mojo:network_service"),
+ &network_service_);
}
// Ownership of this object is transferred to |loader_callback|.
// TODO(beng): this is eff'n weird.
@@ -94,8 +93,7 @@
if (url.SchemeIs("mojo") &&
base::CommandLine::ForCurrentProcess()->HasSwitch(
switches::kUseUpdater)) {
- shell::ConnectToService(application_manager_, GURL("mojo:updater"),
- &updater_);
+ application_manager_->ConnectToService(GURL("mojo:updater"), &updater_);
// Ownership of this object is transferred to |loader_callback|.
// TODO(beng): this is eff'n weird.
new fetcher::UpdateFetcher(url, updater_.get(), loader_callback);
@@ -104,8 +102,8 @@
#endif
if (!url_loader_factory_) {
- shell::ConnectToService(application_manager_, GURL("mojo:network_service"),
- &url_loader_factory_);
+ application_manager_->ConnectToService(GURL("mojo:network_service"),
+ &url_loader_factory_);
}
// Ownership of this object is transferred to |loader_callback|.
« no previous file with comments | « mojo/mojo_shell.gyp ('k') | mojo/runner/context.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698