[Android WebView] Use NetLog for providing full request headers to DevTools

android_webview/browser/net/aw_url_request_context_getter.cc

Index: android_webview/browser/net/aw_url_request_context_getter.cc
diff --git a/android_webview/browser/net/aw_url_request_context_getter.cc b/android_webview/browser/net/aw_url_request_context_getter.cc
index 6e39cd79cce587003a430172276bbcd81324d69c..5ef1f941162dbd0c297ae5ae1e95a6ff5d1d755c 100644
--- a/android_webview/browser/net/aw_url_request_context_getter.cc
+++ b/android_webview/browser/net/aw_url_request_context_getter.cc
@@ -27,6 +27,7 @@
 #include "content/public/common/content_switches.h"
 #include "content/public/common/url_constants.h"
 #include "net/base/cache_type.h"
+#include "net/base/net_log.h"
 #include "net/cookies/cookie_store.h"
 #include "net/dns/mapped_host_resolver.h"
 #include "net/http/http_cache.h"
@@ -205,6 +206,8 @@ void AwURLRequestContextGetter::InitializeURLRequestContext() {
   }
   builder.set_accept_language(net::HttpUtil::GenerateAcceptLanguageHeader(
       AwContentBrowserClient::GetAcceptLangsImpl()));
+  net_log_.reset(new net::NetLog());
+  builder.set_net_log(net_log_.get());
   ApplyCmdlineOverridesToURLRequestContextBuilder(&builder);
 
   url_request_context_.reset(builder.Build());
@@ -276,4 +279,8 @@ AwURLRequestContextGetter::GetDataReductionProxyAuthRequestHandler() const {
   return data_reduction_proxy_auth_request_handler_.get();
 }
 
+net::NetLog* AwURLRequestContextGetter::GetNetLog() const {
+  return net_log_.get();

[boliu, 2014/09/09 15:41:40]

DCHECK(BrowserThread::CurrentlyOn(BrowserThread::IO));
if (!net_log_)
  InitializeURLRequestContext()

This is called on the IO thread, right?

[mnaganov (inactive), 2014/09/09 16:01:17]

Correct, this runs on the IO thread. Although, at least DevTools protects itself
from dealing with a NULL NetLog pointer, initializing still looks like a good
idea, thanks!

+}
+
 }  // namespace android_webview