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

Unified Diff: components/cronet/android/cronet_url_request_context_adapter.h

Issue 586143002: Initial implementation of Cronet Async API. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Address review comments. Created 6 years, 1 month 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
Index: components/cronet/android/cronet_url_request_context_adapter.h
diff --git a/components/cronet/android/cronet_url_request_context_adapter.h b/components/cronet/android/cronet_url_request_context_adapter.h
new file mode 100644
index 0000000000000000000000000000000000000000..b68a96bd6553af1c0b4a4c672472e448790fa38a
--- /dev/null
+++ b/components/cronet/android/cronet_url_request_context_adapter.h
@@ -0,0 +1,72 @@
+// Copyright 2014 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.
+
+#ifndef COMPONENTS_CRONET_ANDROID_CRONET_URL_REQUEST_CONTEXT_ADAPTER_H_
+#define COMPONENTS_CRONET_ANDROID_CRONET_URL_REQUEST_CONTEXT_ADAPTER_H_
+
+#include <string>
+
+#include "base/callback.h"
+#include "base/macros.h"
+#include "base/memory/ref_counted.h"
+#include "base/memory/scoped_ptr.h"
+#include "base/threading/thread.h"
+#include "net/base/net_log.h"
+#include "net/base/network_change_notifier.h"
+
+namespace base {
+class SingleThreadTaskRunner;
+} // namespace base
+
+namespace net {
+class NetLogLogger;
+class URLRequestContext;
+} // namespace net
+
+namespace cronet {
+
+struct URLRequestContextConfig;
+
+// Adapter between Java CronetUrlRequestContext and net::URLRequestContext.
+class CronetURLRequestContextAdapter {
+ public:
+ CronetURLRequestContextAdapter();
+
+ ~CronetURLRequestContextAdapter();
+
+ void Initialize(scoped_ptr<URLRequestContextConfig> config,
+ const base::Closure& java_init_network_thread);
+
+ // Releases all resources for the request context and deletes the object.
mmenke 2014/11/07 15:19:09 Worth mentioning it blocks until the thread is des
mef 2014/11/07 16:22:26 Done.
+ void Destroy();
+
+ net::URLRequestContext* GetURLRequestContext();
+
+ scoped_refptr<base::SingleThreadTaskRunner>
+ GetNetworkTaskRunner() const;
mmenke 2014/11/07 15:19:09 If this doesn't fit on one line, I think generally
mef 2014/11/07 16:22:26 Hmm, fits now. Not sure why was like that.
+
+ void StartNetLogToFile(const std::string& file_name);
+
+ void StopNetLog();
+
+ private:
+ // Network thread is owned by |this|, but is destroyed from java thread.
+ base::Thread* network_thread_;
+ // |net_log_logger_| and |context_| should only be accessed on network thread.
+ scoped_ptr<net::NetLogLogger> net_log_logger_;
+ scoped_ptr<net::URLRequestContext> context_;
+
+ // Initializes |context_| on the Network thread.
+ void InitializeOnNetworkThread(
+ scoped_ptr<URLRequestContextConfig> config,
+ const base::Closure& java_init_network_thread);
+ void StartNetLogToFileOnNetworkThread(const std::string& file_name);
+ void StopNetLogOnNetworkThread();
+
+ DISALLOW_COPY_AND_ASSIGN(CronetURLRequestContextAdapter);
+};
+
+} // namespace cronet
+
+#endif // COMPONENTS_CRONET_ANDROID_CRONET_URL_REQUEST_CONTEXT_ADAPTER_H_

Powered by Google App Engine
This is Rietveld 408576698