Wrong 'sourceMappingURL' prefix

[aar0nr]

The sourceMappingURL prefix is currently "//#" as shown on the following line:

https://github.com/gruntjs/grunt-contrib-uglify/blob/master/tasks/lib/uglify.js#L83

However, this doesn't work in Chrome (v27.0.1453.93). Chrome expects "//@", which is also referenced in the UglifyJS2 docs:

--source-map-url   The path to the source map to be added in //@ sourceMappingURL.  Defaults to the value passed with --source-map.

[jsoverson]

This has been changed in UglifyJS2 here although may be a bit premature given the state of the tracked issues in firefox and chrome

@mishoo, what was your timeline to publish the next version after UglifyJS2/213 was merged?

[mishoo]

@jsoverson Published v2.3.6 containing the change.

[aar0nr]

Ah I see. Thanks for the heads up.

[jsoverson]

@mishoo, didn't mean to rush you, was just wondering if you knew more about the status of the support in browsers.

@mattrobenolt, @acdha, any input as the creators of the respective PRs?

[acdha]

@jsoverson No clean fixes: it probably makes sense to hold off on shipping the patch until browser support ships and remove it after it's been in for a release or two.

[mattrobenolt]

@jsoverson I was following along the discussions about the spec, and it changed a few weeks ago. All of the browser vendors are on the same page with updating, but I suppose only time will tell as to when they adopt.

jQuery is going this direction, just not yet either. See: http://bugs.jquery.com/ticket/13793#comment:12

[jsoverson]

Reverted these changes and published 0.2.2 until ff/chrome gain support. Will open an issue but please feel free to ping on it if anyone is actively monitoring those browser issues.

[jamie-pate]

Why not include both? they are /comments/ right?
in chrome Version 27.0.1453.93 (linux) //# is not working for me

(i'm pretty confused about this whole thing...
reverting to grunt-cli@0.1.8 uglify-js@2.3.5 and grunt-contrib-uglify@0.2.0 seems to fix it for me)