Comment #13 on issue 2764 by [email protected]: Implement custom PAC
support leveraging V8
http://code.google.com/p/chromium/issues/detail?id=2764
The following revision refers to this bug:
http://src.chromium.org/viewvc/chrome?view=rev&revision=10843
------------------------------------------------------------------------
r10843 | [email protected] | 2009-03-03 16:05:20 -0800 (Tue, 03 Mar
2009) | 16 lines
Changed paths:
M
http://src.chromium.org/viewvc/chrome/trunk/src/net/data/proxy_resolver_v8_unittest/pac_library_unittest.js?r1=10843&r2=10842
M
http://src.chromium.org/viewvc/chrome/trunk/src/net/proxy/proxy_resolver_script.h?r1=10843&r2=10842
Fix error being thrown in isInNet() PAC js.
Mozilla is using notation which treats regexp like a function:
var regex = /foo/;
var result = regex("str");
However v8 does not allow this (nor does IE), so we rewrite it in standard
form:
var regex = /foo/;
var result = regex.exec("str")
Also finishes off the tests in pac_library_unittest.js. I didn't test every
single permutation of dateRange() and timeRange() as there are way too
many; but did hit most common flavors.
BUG=2764
Review URL: http://codereview.chromium.org/40006
------------------------------------------------------------------------
--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings
--~--~---------~--~----~------------~-------~--~----~
Automated mail from issue updates at http://crbug.com/
Subscription options: http://groups.google.com/group/chromium-bugs
-~----------~----~----~----~------~----~------~--~---
