Add trace points for the "dnsResolve()" and "myIpAddress()" PAC javascript bindings.

This makes note of the calls in the request's LoadLog.

Since the LoadLog is not thread-safe, SingleThreadedProxyResolver creates a private LoadLog for ProxyResolverV8 to write into, and then copies it into the mainLoadLog on completion (on the origin thread).

BUG=http://crbug.com/14478

Review URL: http://codereview.chromium.org/193037

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@25926 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 3 insertions, 5 deletions

diff --git a/net/proxy/proxy_service.cc b/net/proxy/proxy_service.cc
index 46509c2..637fbbd 100644
--- a/net/proxy/proxy_service.cc
+++ b/net/proxy/proxy_service.cc
@@ -54,7 +54,8 @@ class ProxyResolverNull : public ProxyResolver {
   virtual int GetProxyForURL(const GURL& url,
                              ProxyInfo* results,
                              CompletionCallback* callback,
-                             RequestHandle* request) {
+                             RequestHandle* request,
+                             LoadLog* load_log) {
     return ERR_NOT_IMPLEMENTED;
   }
 
@@ -99,11 +100,8 @@ class ProxyService::PacRequest
 
     config_id_ = service_->config_.id();
 
-    // TODO(eroman): ProxyResolver::GetProxyForURL should take LoadLog*.
-    // Then we can pass in |load_log_| and understand how much time is
-    // spent in bindings like dnsResolve()!
     return resolver()->GetProxyForURL(
-        url_, results_, &io_callback_, &resolve_job_/*, load_log_*/);
+        url_, results_, &io_callback_, &resolve_job_, load_log_);
   }
 
   bool is_started() const {