Display the proxy PAC javascript errors in the NetLog....

net/proxy/proxy_resolver_v8_unittest.cc

 // Copyright (c) 2009 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 "base/file_util.h"
 #include "base/path_service.h"
 #include "base/string_util.h"
 #include "base/utf_string_conversions.h"
 #include "googleurl/src/gurl.h"
+#include "net/base/net_errors.h"
 #include "net/base/net_log_unittest.h"
-#include "net/base/net_errors.h"
 #include "net/proxy/proxy_info.h"
 #include "net/proxy/proxy_resolver_js_bindings.h"
 #include "net/proxy/proxy_resolver_v8.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 namespace net {
 namespace {
 
 // Javascript bindings for ProxyResolverV8, which returns mock values.
 // Each time one of the bindings is called into, we push the input into a
@@ skipping 429 matching lines @@
   EXPECT_EQ("127.0.0.1:80", proxy_info.proxy_server().ToURI());
 
   // Check that no other bindings were called.
   EXPECT_EQ(0U, bindings->errors.size());
   ASSERT_EQ(0U, bindings->alerts.size());
   ASSERT_EQ(0U, bindings->dns_resolves.size());
   EXPECT_EQ(0, bindings->my_ip_address_ex_count);
   ASSERT_EQ(0U, bindings->dns_resolves_ex.size());
 }
 
-// Test that calls to the myIpAddress() and dnsResolve() bindings get
-// logged to the NetLog parameter.
-TEST(ProxyResolverV8Test, NetLog) {
-  ProxyResolverV8WithMockBindings resolver;
-  int result = resolver.SetPacScriptFromDisk("simple.js");
-  EXPECT_EQ(OK, result);
-
-  ProxyInfo proxy_info;
-  CapturingBoundNetLog log(CapturingNetLog::kUnbounded);
-  result = resolver.GetProxyForURL(kQueryUrl, &proxy_info, NULL, NULL,
-                                   log.bound());
-
-  EXPECT_EQ(OK, result);
-  EXPECT_FALSE(proxy_info.is_direct());
-  EXPECT_EQ("c:100", proxy_info.proxy_server().ToURI());
-
-  // Note that dnsResolve() was never called directly, but it appears
-  // in the NetLog. This is because it gets called indirectly by
-  // isInNet() and isResolvable().
-
-  EXPECT_EQ(6u, log.entries().size());
-  EXPECT_TRUE(LogContainsBeginEvent(
-      log.entries(), 0, NetLog::TYPE_PROXY_RESOLVER_V8_MY_IP_ADDRESS));
-  EXPECT_TRUE(LogContainsEndEvent(
-      log.entries(), 1, NetLog::TYPE_PROXY_RESOLVER_V8_MY_IP_ADDRESS));
-  EXPECT_TRUE(LogContainsBeginEvent(
-      log.entries(), 2, NetLog::TYPE_PROXY_RESOLVER_V8_DNS_RESOLVE));
-  EXPECT_TRUE(LogContainsEndEvent(
-      log.entries(), 3, NetLog::TYPE_PROXY_RESOLVER_V8_DNS_RESOLVE));
-  EXPECT_TRUE(LogContainsBeginEvent(
-      log.entries(), 4, NetLog::TYPE_PROXY_RESOLVER_V8_DNS_RESOLVE));
-  EXPECT_TRUE(LogContainsEndEvent(
-      log.entries(), 5, NetLog::TYPE_PROXY_RESOLVER_V8_DNS_RESOLVE));
-}
-
 // Try loading a PAC script that ends with a comment and has no terminal
 // newline. This should not cause problems with the PAC utility functions
 // that we add to the script's environment.
 // http://crbug.com/22864
 TEST(ProxyResolverV8Test, EndsWithCommentNoNewline) {
   ProxyResolverV8WithMockBindings resolver;
   int result = resolver.SetPacScriptFromDisk("ends_with_comment.js");
   EXPECT_EQ(OK, result);
 
   ProxyInfo proxy_info;
@@ skipping 63 matching lines @@
 
   ASSERT_EQ(1u, bindings->dns_resolves.size());
   EXPECT_EQ("xn--bcher-kva.ch", bindings->dns_resolves[0]);
 
   ASSERT_EQ(1u, bindings->dns_resolves_ex.size());
   EXPECT_EQ("xn--bcher-kva.ch", bindings->dns_resolves_ex[0]);
 }
 
 }  // namespace
 }  // namespace net