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

Unified Diff: content/network/network_service_impl.h

Issue 2968293002: Introduce SystemNetworkContextManager. (Closed)
Patch Set: Response to comments Created 3 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 | « content/network/network_service.cc ('k') | content/network/network_service_impl.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/network/network_service_impl.h
diff --git a/content/network/network_service.h b/content/network/network_service_impl.h
similarity index 67%
rename from content/network/network_service.h
rename to content/network/network_service_impl.h
index 703ffc29b13515b57d8c43cb812b1303fd0cfd0c..74e869109786571f368f1ece401aad35de9d49ff 100644
--- a/content/network/network_service.h
+++ b/content/network/network_service_impl.h
@@ -2,30 +2,43 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
-#ifndef CONTENT_NETWORK_NETWORK_SERVICE_H_
-#define CONTENT_NETWORK_NETWORK_SERVICE_H_
+#ifndef CONTENT_NETWORK_NETWORK_SERVICE_IMPL_H_
+#define CONTENT_NETWORK_NETWORK_SERVICE_IMPL_H_
#include <memory>
#include "base/macros.h"
#include "content/common/content_export.h"
#include "content/public/common/network_service.mojom.h"
+#include "content/public/network/network_service.h"
#include "mojo/public/cpp/bindings/binding.h"
#include "services/service_manager/public/cpp/binder_registry.h"
#include "services/service_manager/public/cpp/service.h"
+namespace net {
+class URLRequestContext;
+class URLRequestContextBuilder;
+} // namespace net
+
namespace content {
class NetworkContext;
-class NetworkService : public service_manager::Service,
- public mojom::NetworkService {
+class CONTENT_EXPORT NetworkServiceImpl : public service_manager::Service,
+ public NetworkService {
public:
- explicit NetworkService(
+ explicit NetworkServiceImpl(
std::unique_ptr<service_manager::BinderRegistry> registry);
- ~NetworkService() override;
- CONTENT_EXPORT static std::unique_ptr<NetworkService> CreateForTesting();
+ ~NetworkServiceImpl() override;
+
+ std::unique_ptr<mojom::NetworkContext> CreateNetworkContextWithBuilder(
+ content::mojom::NetworkContextRequest request,
+ content::mojom::NetworkContextParamsPtr params,
+ std::unique_ptr<net::URLRequestContextBuilder> builder,
+ net::URLRequestContext** url_request_context) override;
+
+ static std::unique_ptr<NetworkService> CreateForTesting();
// These are called by NetworkContexts as they are being created and
// destroyed.
@@ -39,9 +52,6 @@ class NetworkService : public service_manager::Service,
private:
class MojoNetLog;
- // Used for tests.
- NetworkService();
-
// service_manager::Service implementation.
void OnBindInterface(const service_manager::BindSourceInfo& source_info,
const std::string& interface_name,
@@ -62,9 +72,9 @@ class NetworkService : public service_manager::Service,
// destroyed first.
std::set<NetworkContext*> network_contexts_;
- DISALLOW_COPY_AND_ASSIGN(NetworkService);
+ DISALLOW_COPY_AND_ASSIGN(NetworkServiceImpl);
};
} // namespace content
-#endif // CONTENT_NETWORK_NETWORK_SERVICE_H_
+#endif // CONTENT_NETWORK_NETWORK_SERVICE_IMPL_H_
« no previous file with comments | « content/network/network_service.cc ('k') | content/network/network_service_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698