Initial implementation of Cronet Async API.

components/cronet/android/cronet_url_request_context_adapter.cc

 // 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.
 
-#include "components/cronet/android/url_request_context_adapter.h"
-
-#include <limits>
+#include "components/cronet/android/cronet_url_request_context_adapter.h"
 
 #include "base/bind.h"
 #include "base/files/file_util.h"
-#include "base/message_loop/message_loop.h"
 #include "base/single_thread_task_runner.h"
 #include "components/cronet/url_request_context_config.h"
-#include "net/android/network_change_notifier_factory_android.h"
 #include "net/base/net_errors.h"
 #include "net/base/net_log_logger.h"
-#include "net/base/net_util.h"
-#include "net/base/network_change_notifier.h"
 #include "net/cert/cert_verifier.h"
 #include "net/http/http_auth_handler_factory.h"
 #include "net/http/http_network_layer.h"
 #include "net/http/http_server_properties.h"
+#include "net/proxy/proxy_config_service_fixed.h"
 #include "net/proxy/proxy_service.h"
 #include "net/ssl/ssl_config_service_defaults.h"
 #include "net/url_request/static_http_user_agent_settings.h"
+#include "net/url_request/url_request_context.h"
 #include "net/url_request/url_request_context_builder.h"
 #include "net/url_request/url_request_context_storage.h"
 #include "net/url_request/url_request_job_factory_impl.h"
 
 namespace {
 
-// MessageLoop on the main thread, which is where objects that receive Java
-// notifications generally live.
-base::MessageLoop* g_main_message_loop = nullptr;
-
-net::NetworkChangeNotifier* g_network_change_notifier = nullptr;
-
 class BasicNetworkDelegate : public net::NetworkDelegate {
  public:
   BasicNetworkDelegate() {}
   virtual ~BasicNetworkDelegate() {}
 
  private:
   // net::NetworkDelegate implementation.
   int OnBeforeURLRequest(net::URLRequest* request,
                          const net::CompletionCallback& callback,
                          GURL* new_url) override {
@@ skipping 16 matching lines @@
       scoped_refptr<net::HttpResponseHeaders>* _response_headers,
       GURL* allowed_unsafe_redirect_url) override {
     return net::OK;
   }
 
   void OnBeforeRedirect(net::URLRequest* request,
                         const GURL& new_location) override {}
 
   void OnResponseStarted(net::URLRequest* request) override {}
 
-  void OnRawBytesRead(const net::URLRequest& request,
-                      int bytes_read) override {}
+  void OnRawBytesRead(const net::URLRequest& request, int bytes_read) override {
+  }
 
   void OnCompleted(net::URLRequest* request, bool started) override {}
 
   void OnURLRequestDestroyed(net::URLRequest* request) override {}
 
-  void OnPACScriptError(int line_number,
-                        const base::string16& error) override {}
+  void OnPACScriptError(int line_number, const base::string16& error) override {
+  }
 
   NetworkDelegate::AuthRequiredResponse OnAuthRequired(
       net::URLRequest* request,
       const net::AuthChallengeInfo& auth_info,
       const AuthCallback& callback,
       net::AuthCredentials* credentials) override {
     return net::NetworkDelegate::AUTH_REQUIRED_RESPONSE_NO_ACTION;
   }
 
   bool OnCanGetCookies(const net::URLRequest& request,
                        const net::CookieList& cookie_list) override {
     return false;
   }
 
   bool OnCanSetCookie(const net::URLRequest& request,
                       const std::string& cookie_line,
                       net::CookieOptions* options) override {
     return false;
   }
 
   bool OnCanAccessFile(const net::URLRequest& request,
                        const base::FilePath& path) const override {
     return false;
   }
 
-  bool OnCanThrottleRequest(
-      const net::URLRequest& request) const override {
+  bool OnCanThrottleRequest(const net::URLRequest& request) const override {
     return false;
   }
 
   int OnBeforeSocketStreamConnect(
       net::SocketStream* stream,
       const net::CompletionCallback& callback) override {
     return net::OK;
   }
 
   DISALLOW_COPY_AND_ASSIGN(BasicNetworkDelegate);
 };
 
 }  // namespace
 
 namespace cronet {
 
-URLRequestContextAdapter::URLRequestContextAdapter(
-    URLRequestContextAdapterDelegate* delegate,
-    std::string user_agent) {
-  delegate_ = delegate;
-  user_agent_ = user_agent;
+CronetURLRequestContextAdapter::CronetURLRequestContextAdapter() {
 }
 
-void URLRequestContextAdapter::Initialize(
-    scoped_ptr<URLRequestContextConfig> config) {
+void CronetURLRequestContextAdapter::Initialize(
+    scoped_ptr<URLRequestContextConfig> config,
+    const base::Closure& java_init_network_thread) {
   network_thread_ = new base::Thread("network");
   base::Thread::Options options;
   options.message_loop_type = base::MessageLoop::TYPE_IO;
   network_thread_->StartWithOptions(options);
-  config_ = config.Pass();
+
+  GetNetworkTaskRunner()->PostTask(
+      FROM_HERE,
+      base::Bind(&CronetURLRequestContextAdapter::InitializeOnNetworkThread,
+                 base::Unretained(this),
+                 Passed(&config),
+                 java_init_network_thread));
 }
 
-void URLRequestContextAdapter::InitRequestContextOnMainThread() {
-  if (!base::MessageLoop::current()) {
-    DCHECK(!g_main_message_loop);
-    g_main_message_loop = new base::MessageLoopForUI();
-    base::MessageLoopForUI::current()->Start();
-  }
-  DCHECK_EQ(g_main_message_loop, base::MessageLoop::current());
-  if (!g_network_change_notifier) {
-    net::NetworkChangeNotifier::SetFactory(
-        new net::NetworkChangeNotifierFactoryAndroid());
-    g_network_change_notifier = net::NetworkChangeNotifier::Create();
-  }
-  proxy_config_service_.reset(net::ProxyService::CreateSystemProxyConfigService(
-      GetNetworkTaskRunner(), NULL));
-  GetNetworkTaskRunner()->PostTask(
-      FROM_HERE,
-      base::Bind(&URLRequestContextAdapter::InitRequestContextOnNetworkThread,
-                 this));
-}
-
-void URLRequestContextAdapter::InitRequestContextOnNetworkThread() {
+void CronetURLRequestContextAdapter::InitializeOnNetworkThread(
+    scoped_ptr<URLRequestContextConfig> config,
+    const base::Closure& java_init_network_thread) {
   DCHECK(GetNetworkTaskRunner()->BelongsToCurrentThread());
-  DCHECK(config_);
   // TODO(mmenke):  Add method to have the builder enable SPDY.
   net::URLRequestContextBuilder context_builder;
   context_builder.set_network_delegate(new BasicNetworkDelegate());
-  context_builder.set_proxy_config_service(proxy_config_service_.get());
-  config_->ConfigureURLRequestContextBuilder(&context_builder);
+  context_builder.set_proxy_config_service(
+      new net::ProxyConfigServiceFixed(net::ProxyConfig()));
+  config->ConfigureURLRequestContextBuilder(&context_builder);
 
   context_.reset(context_builder.Build());
 
   // Currently (circa M39) enabling QUIC requires setting probability threshold.
-  if (config_->enable_quic) {
+  if (config->enable_quic) {
     context_->http_server_properties()
         ->SetAlternateProtocolProbabilityThreshold(0.0f);
-    for (size_t hint = 0; hint < config_->quic_hints.size(); ++hint) {
+    for (size_t hint = 0; hint < config->quic_hints.size(); ++hint) {

[mmenke, 2014/11/06 20:50:14]

nit:  Should be using a iterator (const_iterator, actually)

[mef, 2014/11/06 22:51:46]

Done.

       const URLRequestContextConfig::QuicHint& quic_hint =
-          *config_->quic_hints[hint];
+          *config->quic_hints[hint];
       if (quic_hint.host.empty()) {
         LOG(ERROR) << "Empty QUIC hint host: " << quic_hint.host;
         continue;
       }
 
-      url::CanonHostInfo host_info;
-      std::string canon_host(net::CanonicalizeHost(quic_hint.host, &host_info));
-      if (!host_info.IsIPAddress() &&
-          !net::IsCanonicalizedHostCompliant(canon_host)) {
-        LOG(ERROR) << "Invalid QUIC hint host: " << quic_hint.host;
+      if (quic_hint.port <= std::numeric_limits<uint16>::min() ||

[mmenke, 2014/11/06 20:50:15]

Why is the validation code different?  Just using an old version of it?

[mef, 2014/11/06 22:51:46]

Done.

+          quic_hint.port > std::numeric_limits<uint16>::max()) {
+        LOG(ERROR) << "Invalid QUIC hint port: " << quic_hint.port;
         continue;
       }
 
-      if (quic_hint.port <= std::numeric_limits<uint16>::min() ||
-          quic_hint.port > std::numeric_limits<uint16>::max()) {
-        LOG(ERROR) << "Invalid QUIC hint port: "
-                   << quic_hint.port;
-        continue;
-      }
-
       if (quic_hint.alternate_port <= std::numeric_limits<uint16>::min() ||
           quic_hint.alternate_port > std::numeric_limits<uint16>::max()) {
         LOG(ERROR) << "Invalid QUIC hint alternate port: "
                    << quic_hint.alternate_port;
         continue;
       }
 
-      net::HostPortPair quic_hint_host_port_pair(canon_host,
+      net::HostPortPair quic_hint_host_port_pair(quic_hint.host,
                                                  quic_hint.port);
       context_->http_server_properties()->SetAlternateProtocol(
           quic_hint_host_port_pair,
           static_cast<uint16>(quic_hint.alternate_port),
           net::AlternateProtocol::QUIC,
           1.0f);
     }
   }
-  config_.reset(NULL);
 
-  if (VLOG_IS_ON(2)) {
-    net_log_observer_.reset(new NetLogObserver());
-    context_->net_log()->AddThreadSafeObserver(net_log_observer_.get(),
-                                               net::NetLog::LOG_ALL_BUT_BYTES);
-  }
-
-  is_context_initialized_ = true;
-  while (!tasks_waiting_for_context_.empty()) {
-    tasks_waiting_for_context_.front().Run();
-    tasks_waiting_for_context_.pop();
-  }
-
-  delegate_->OnContextInitialized(this);
+  java_init_network_thread.Run();
 }
 
-void URLRequestContextAdapter::PostTaskToNetworkThread(
-    const tracked_objects::Location& posted_from,
-    const RunAfterContextInitTask& callback) {
-  GetNetworkTaskRunner()->PostTask(
-      posted_from,
-      base::Bind(
-          &URLRequestContextAdapter::RunTaskAfterContextInitOnNetworkThread,
-          this,
-          callback));
+void CronetURLRequestContextAdapter::Destroy() {
+  DCHECK(!GetNetworkTaskRunner()->BelongsToCurrentThread());
+  GetNetworkTaskRunner()->DeleteSoon(FROM_HERE, this);
+  // Deleting thread stops it after all tasks are completed.
+  delete network_thread_;
 }
 
-void URLRequestContextAdapter::RunTaskAfterContextInitOnNetworkThread(
-    const RunAfterContextInitTask& callback) {
+CronetURLRequestContextAdapter::~CronetURLRequestContextAdapter() {
   DCHECK(GetNetworkTaskRunner()->BelongsToCurrentThread());
-  if (is_context_initialized_) {
-    callback.Run();
-    return;
-  }
-  tasks_waiting_for_context_.push(callback);
+  StopNetLogOnNetworkThread();
 }
 
-URLRequestContextAdapter::~URLRequestContextAdapter() {
-  DCHECK(GetNetworkTaskRunner()->BelongsToCurrentThread());
-  if (net_log_observer_) {
-    context_->net_log()->RemoveThreadSafeObserver(net_log_observer_.get());
-    net_log_observer_.reset();
-  }
-  StopNetLogHelper();
-  // TODO(mef): Ensure that |network_thread_| is destroyed properly.
-}
-
-const std::string& URLRequestContextAdapter::GetUserAgent(
-    const GURL& url) const {
-  return user_agent_;
-}
-
-net::URLRequestContext* URLRequestContextAdapter::GetURLRequestContext() {
-  DCHECK(GetNetworkTaskRunner()->BelongsToCurrentThread());
+net::URLRequestContext* CronetURLRequestContextAdapter::GetURLRequestContext() {
   if (!context_) {
     LOG(ERROR) << "URLRequestContext is not set up";
   }
   return context_.get();
 }
 
 scoped_refptr<base::SingleThreadTaskRunner>
-URLRequestContextAdapter::GetNetworkTaskRunner() const {
-  return network_thread_->message_loop_proxy();
+CronetURLRequestContextAdapter::GetNetworkTaskRunner() const {
+  return network_thread_->task_runner();
 }
 
-void URLRequestContextAdapter::StartNetLogToFile(const std::string& file_name) {
-  PostTaskToNetworkThread(
+void CronetURLRequestContextAdapter::StartNetLogToFile(
+    const std::string& file_name) {
+  GetNetworkTaskRunner()->PostTask(
       FROM_HERE,
       base::Bind(
-          &URLRequestContextAdapter::StartNetLogToFileHelper, this, file_name));
+          &CronetURLRequestContextAdapter::StartNetLogToFileOnNetworkThread,
+          base::Unretained(this),
+          file_name));
 }
 
-void URLRequestContextAdapter::StopNetLog() {
-  PostTaskToNetworkThread(
-      FROM_HERE, base::Bind(&URLRequestContextAdapter::StopNetLogHelper, this));
+void CronetURLRequestContextAdapter::StopNetLog() {
+  GetNetworkTaskRunner()->PostTask(
+      FROM_HERE,
+      base::Bind(&CronetURLRequestContextAdapter::StopNetLogOnNetworkThread,
+                 base::Unretained(this)));
 }
 
-void URLRequestContextAdapter::StartNetLogToFileHelper(
+void CronetURLRequestContextAdapter::StartNetLogToFileOnNetworkThread(
     const std::string& file_name) {
   DCHECK(GetNetworkTaskRunner()->BelongsToCurrentThread());
   // Do nothing if already logging to a file.
   if (net_log_logger_)
     return;
 
   base::FilePath file_path(file_name);
   FILE* file = base::OpenFile(file_path, "w");
   if (!file)
     return;
 
   scoped_ptr<base::Value> constants(net::NetLogLogger::GetConstants());
   net_log_logger_.reset(new net::NetLogLogger(file, *constants));
   net_log_logger_->StartObserving(context_->net_log());
 }
 
-void URLRequestContextAdapter::StopNetLogHelper() {
+void CronetURLRequestContextAdapter::StopNetLogOnNetworkThread() {
   DCHECK(GetNetworkTaskRunner()->BelongsToCurrentThread());
   if (net_log_logger_) {
     net_log_logger_->StopObserving();
     net_log_logger_.reset();
   }
 }
 
-void NetLogObserver::OnAddEntry(const net::NetLog::Entry& entry) {
-  VLOG(2) << "Net log entry: type=" << entry.type()
-          << ", source=" << entry.source().type << ", phase=" << entry.phase();
-}
-
 }  // namespace cronet