Index: net/proxy/proxy_resolver_v8_unittest.cc |
=================================================================== |
--- net/proxy/proxy_resolver_v8_unittest.cc (revision 27049) |
+++ net/proxy/proxy_resolver_v8_unittest.cc (working copy) |
@@ -420,11 +420,11 @@ |
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 @@ |
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 |