Add an additional unit-test for when PAC script is missing newline....

Add an additional unit-test for when PAC script is missing newline.
This variation uses a statement without semi-colon, rather than a comment, as the last line.

BUG=http://crbug.com/22864


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=27171

[eroman, 2009-09-24 19:54:43]

The contents of JS file don't show up in this review, because I marked it as
binary (so SVN doesn't mangle the line-ending).

This is what it contains:

// Ends with a statement, and no terminal newline.
function FindProxyForURL(url, host) { return "PROXY success:" + x; }
x = 3

[wtc, 2009-09-24 20:43:36]

LGTM.

http://codereview.chromium.org/238004/diff/1/2
File net/proxy/proxy_resolver_v8_unittest.cc (right):

http://codereview.chromium.org/238004/diff/1/2#newcode425
Line 425: // that we add to the script. http://crbug.com/22864
Nit: "that we add to the script" is no longer true in the
new code, because now we run the two scripts separately.

Why don't we break this into two scripts?  They only share
one line of code (line 427).

[eroman, 2009-09-24 20:51:41]

http://codereview.chromium.org/238004/diff/1/2
File net/proxy/proxy_resolver_v8_unittest.cc (right):

http://codereview.chromium.org/238004/diff/1/2#newcode425
Line 425: // that we add to the script. http://crbug.com/22864
On 2009/09/24 20:43:36, wtc wrote:
> Nit: "that we add to the script" is no longer true in the
> new code, because now we run the two scripts separately.

Reworded "add to the script" --> "add to the script's environment".

> 
> Why don't we break this into two scripts?  They only share
> one line of code (line 427).

Done, made as 2 separate TESTs.