1172290 - don't validate JS source comments

Status: RESOLVED INVALID

(addons.mozilla.org Graveyard :: Add-on Validation, defect)

Description

[Dave Garrett]

I just ran the AMO addon validator on a test build and got this warning:

==================================================
Potentially unsafe preference branch referenced
Signing severity: low
Warning: Changing network preferences may be dangerous, and often leads to performance costs.

chrome/flagfox/content/flagfox.jsm
1328 // If "network.proxy.socks_remote_dns" is set to true or the proxy is otherwise set up to be the one to do all DNS resolution, then don't do ...
1329 if ( (proxyinfo != null) && (proxyinfo.flags &
==================================================

Yeah... it's giving me a warning because I mentioned the existence of a network preference in a source comment. :/

Please do a little bit to reduce the false-positives this thing gets by not validating the comments. (at least the "//" single line comments, if multi line comments using "/*" & "*/" are hard to parse)

Comment 1

[Mathieu Agopian [:magopian]]

:kmag, how feasible do you think this is?

Comment 2

[Kris Maglione [:kmag]]

Most of our tests use parse tree analysis, so they aren't triggered by code in comments. Some tests use regular expressions, which unfortunately are not able to distinguish comments from code. Unfortunately, the grammar of JavaScript is complicated enough that we really can't reliably filter out comments without going through full lexical analysis, which isn't really doable.

As for this particular warning, it should go away when I move preference tests to the regex test class which only checks the values of JS strings.