Chromium Code Reviews
DescriptionFix NamingTest on all Frog configurations:
* throw the right exception even if the missing method was call$1
* fix a nasty escape bug in replaceAll ($ is a meta-char in the second argument of String.prototype.replace, so it must be escaped as "$$"). This bug is especially meta because the code that failed was str.replaceAll("$", "$$") ... but the bug could be triggered any time the "to" argument to "replaceAll" contained "$". See this link for more info:
https://developer.mozilla.org/en/JavaScript/Reference/Global_Objects/String/replace
Committed: https://code.google.com/p/dart/source/detail?r=1966
Patch Set 1 #
Total comments: 1
Messages
Total messages: 2 (0 generated)
|
||||||||||||||||||||||||||||||||||||||||||||||