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

Unified Diff: mojo/fetcher/base_application_fetcher.h

Issue 1342503003: Move fetching logic out of ApplicationManager, eliminate url mappings. (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/fetcher/about_fetcher_unittest.cc ('k') | mojo/fetcher/base_application_fetcher.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: mojo/fetcher/base_application_fetcher.h
diff --git a/mojo/fetcher/base_application_fetcher.h b/mojo/fetcher/base_application_fetcher.h
new file mode 100644
index 0000000000000000000000000000000000000000..5e39f29bf7f560126eec2a94a6a1fde2911001fe
--- /dev/null
+++ b/mojo/fetcher/base_application_fetcher.h
@@ -0,0 +1,41 @@
+// Copyright 2015 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "base/files/file_path.h"
+#include "mojo/fetcher/url_resolver.h"
+#include "mojo/services/network/public/interfaces/network_service.mojom.h"
+#include "mojo/services/network/public/interfaces/url_loader_factory.mojom.h"
+#include "mojo/shell/application_fetcher.h"
+
+namespace mojo {
+namespace fetcher {
+
+// This is the default implementation of shell::ApplicationFetcher. It loads
+// http/s urls off the network as well as providing special handling for mojo:
+// and about: urls.
+class BaseApplicationFetcher : public shell::ApplicationFetcher {
+ public:
+ // mojo: urls are only supported if |shell_file_root| is non-empty.
+ BaseApplicationFetcher(const base::FilePath& shell_file_root);
+ ~BaseApplicationFetcher() override;
+
+ private:
+ // Overridden from shell::ApplicationFetcher:
+ void SetApplicationManager(shell::ApplicationManager* manager) override;
+ GURL ResolveURL(const GURL& url) override;
+ void FetchRequest(
+ URLRequestPtr request,
+ const shell::Fetcher::FetchCallback& loader_callback) override;
+
+ shell::ApplicationManager* application_manager_;
+ scoped_ptr<URLResolver> url_resolver_;
+ const bool disable_cache_;
+ NetworkServicePtr network_service_;
+ URLLoaderFactoryPtr url_loader_factory_;
+
+ DISALLOW_COPY_AND_ASSIGN(BaseApplicationFetcher);
+};
+
+} // namespace fetcher
+} // namespace mojo
« no previous file with comments | « mojo/fetcher/about_fetcher_unittest.cc ('k') | mojo/fetcher/base_application_fetcher.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698