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

Unified Diff: mojo/fetcher/base_application_fetcher.h

Issue 1343823002: Revert of 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
deleted file mode 100644
index 5e39f29bf7f560126eec2a94a6a1fde2911001fe..0000000000000000000000000000000000000000
--- a/mojo/fetcher/base_application_fetcher.h
+++ /dev/null
@@ -1,41 +0,0 @@
-// 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