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

Unified Diff: mojo/shell/package_manager/package_manager_impl.cc

Issue 1563413002: Move fetchers into mojo/shell/fetcher (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 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
« no previous file with comments | « mojo/shell/package_manager/package_manager_impl.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: mojo/shell/package_manager/package_manager_impl.cc
diff --git a/mojo/shell/package_manager/package_manager_impl.cc b/mojo/shell/package_manager/package_manager_impl.cc
index b8cf8a70c95317e2d619fc50ed8af430d3114f61..9b95c3edee4326bbb10df5975d86e39350c074e9 100644
--- a/mojo/shell/package_manager/package_manager_impl.cc
+++ b/mojo/shell/package_manager/package_manager_impl.cc
@@ -10,14 +10,13 @@
#include "base/bind.h"
#include "mojo/application/public/interfaces/content_handler.mojom.h"
-#include "mojo/fetcher/about_fetcher.h"
-#include "mojo/fetcher/data_fetcher.h"
-#include "mojo/fetcher/local_fetcher.h"
-#include "mojo/fetcher/network_fetcher.h"
-#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/fetcher/about_fetcher.h"
+#include "mojo/shell/fetcher/data_fetcher.h"
+#include "mojo/shell/fetcher/local_fetcher.h"
+#include "mojo/shell/fetcher/network_fetcher.h"
+#include "mojo/shell/fetcher/switches.h"
#include "mojo/shell/package_manager/content_handler_connection.h"
#include "mojo/shell/query_util.h"
#include "mojo/shell/switches.h"
@@ -39,7 +38,7 @@ PackageManagerImpl::PackageManagerImpl(
if (!shell_file_root.empty()) {
GURL mojo_root_file_url =
util::FilePathToFileURL(shell_file_root).Resolve(std::string());
- url_resolver_.reset(new fetcher::URLResolver(mojo_root_file_url));
+ url_resolver_.reset(new URLResolver(mojo_root_file_url));
}
}
@@ -74,13 +73,13 @@ void PackageManagerImpl::FetchRequest(
URLRequestPtr request,
const Fetcher::FetchCallback& loader_callback) {
GURL url(request->url);
- if (url.SchemeIs(fetcher::AboutFetcher::kAboutScheme)) {
- fetcher::AboutFetcher::Start(url, loader_callback);
+ if (url.SchemeIs(AboutFetcher::kAboutScheme)) {
+ AboutFetcher::Start(url, loader_callback);
return;
}
if (url.SchemeIs(url::kDataScheme)) {
- fetcher::DataFetcher::Start(url, loader_callback);
+ DataFetcher::Start(url, loader_callback);
return;
}
@@ -94,10 +93,9 @@ void PackageManagerImpl::FetchRequest(
}
// Ownership of this object is transferred to |loader_callback|.
// TODO(beng): this is eff'n weird.
- new fetcher::LocalFetcher(
- network_service_.get(), resolved_url,
- GetBaseURLAndQuery(resolved_url, nullptr),
- shell_file_root_, loader_callback);
+ new LocalFetcher(network_service_.get(), resolved_url,
+ GetBaseURLAndQuery(resolved_url, nullptr),
+ shell_file_root_, loader_callback);
return;
}
@@ -108,8 +106,8 @@ void PackageManagerImpl::FetchRequest(
// Ownership of this object is transferred to |loader_callback|.
// TODO(beng): this is eff'n weird.
- new fetcher::NetworkFetcher(disable_cache_, std::move(request),
- url_loader_factory_.get(), loader_callback);
+ new NetworkFetcher(disable_cache_, std::move(request),
+ url_loader_factory_.get(), loader_callback);
}
uint32_t PackageManagerImpl::HandleWithContentHandler(
« no previous file with comments | « mojo/shell/package_manager/package_manager_impl.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698