586016 - Update JS_GetImplementationVersion for JS 1.8.5

Status: RESOLVED FIXED

(Core :: JavaScript Engine, defect)

Description

[Bob Clary [:bc] (inactive)]

Update js version to 1.8.5 and date in JS_GetImplementationVersion in jsapi.cpp for the Firefox 4 release.

Do we want to support type="text/javascript;version=1.8.5" ?

Comment 1

[Jason Orendorff [:jorendorff]]

(In reply to comment #0)
> Update js version to 1.8.5 and date in JS_GetImplementationVersion in jsapi.cpp
> for the Firefox 4 release.

I encourage Wes to take this bug and post the straightforward patch. I'll review. This is a zero-risk change (Gecko doesn't use the function) so I'll request approval to land it before FF4. If approval isn't granted, no big deal. You can apply the patch manually for the SM 1.8.5 source tarball, and we'll land it in the tracemonkey repo after FF4.

> Do we want to support type="text/javascript;version=1.8.5" ?

No.

Comment 2

[Wesley W. Garland]

Attachment: Trivial patch to tweak the version string

This affects only the jsapi.cpp function. No need to tweak jsversion.h, that was already done by jimb in October.

We should probably move toward having JS_GetImplementationVersion() and jsversion.h forcefully synchronized in the future.

Comment 3

[Jason Orendorff [:jorendorff]]

Comment on attachment 516596 [details] [diff] [review]
Trivial patch to tweak the version string

r=me, and feel free to adjust that date to whatever we end up releasing.

Comment 4

[Wesley W. Garland]

http://hg.mozilla.org/tracemonkey/rev/9f71009b8f8b