diff options
Diffstat (limited to 'net/proxy')
| -rw-r--r-- | net/proxy/proxy_resolver_v8_unittest.cc | 27 |
1 files changed, 23 insertions, 4 deletions
diff --git a/net/proxy/proxy_resolver_v8_unittest.cc b/net/proxy/proxy_resolver_v8_unittest.cc index de6ea5c..e37e643 100644 --- a/net/proxy/proxy_resolver_v8_unittest.cc +++ b/net/proxy/proxy_resolver_v8_unittest.cc @@ -420,11 +420,11 @@ TEST(ProxyResolverV8Test, LoadLog) { LoadLog::PHASE_END); } -// Try loading a PAC script which ends with a trailing comment (no terminal -// newline). This should not cause problems with the PAC utility functions -// that we add to the script. +// 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, TrailingComment) { +TEST(ProxyResolverV8Test, EndsWithCommentNoNewline) { ProxyResolverV8WithMockBindings resolver; int result = resolver.SetPacScriptFromDisk("ends_with_comment.js"); EXPECT_EQ(OK, result); @@ -438,5 +438,24 @@ TEST(ProxyResolverV8Test, TrailingComment) { EXPECT_EQ("success:80", proxy_info.proxy_server().ToURI()); } +// Try loading a PAC script that ends with a statement 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, EndsWithStatementNoNewline) { + ProxyResolverV8WithMockBindings resolver; + int result = resolver.SetPacScriptFromDisk( + "ends_with_statement_no_semicolon.js"); + EXPECT_EQ(OK, result); + + ProxyInfo proxy_info; + scoped_refptr<LoadLog> log(new LoadLog); + result = resolver.GetProxyForURL(kQueryUrl, &proxy_info, NULL, NULL, log); + + EXPECT_EQ(OK, result); + EXPECT_FALSE(proxy_info.is_direct()); + EXPECT_EQ("success:3", proxy_info.proxy_server().ToURI()); +} + } // namespace } // namespace net |