Some possible optimizations in the JavaScript used on Browserscope

[GoogleCodeExporter]

Attached is a patch against revision 699 that gets rid of some unneeded 
JavaScript and does some minor optimizations here and there, like declaring 
multiple variables in one go instead of in separate `var` statements..

Note that setting `el.type = "text/javascript"` or `el.type = "text/css"` in JS 
is not needed since all browsers default to these values automatically. (This 
is also the reason why in HTML5 these attributes are optional in most cases.)

Original issue reported on code.google.com by mathias%...@gtempaccount.com on 3 Sep 2010 at 2:37

Attachments:

• javascript-optimizations.diff

[GoogleCodeExporter]

Hey Mathias, looks great! Would you mind to remove the optimizations that group 
var settings together though? We use the closure-compiler(see 
static/compile_js.sh) to produce the code for production, and it will do 
inlining for us. For debugging, I find it easier and more readable to have 
variable declarations independent of one another.
Then I'll just apply your patch and commit it!

Original comment by els...@gmail.com on 6 Sep 2010 at 8:31

• Changed state: Accepted
• Added labels: Type-Enhancement
• Removed labels: Type-Defect

[GoogleCodeExporter]

Original comment by mathias%...@gtempaccount.com on 7 Sep 2010 at 3:51

Attachments:

• js-optimizations.diff

[GoogleCodeExporter]

This issue was closed by revision r713.

Original comment by els...@gmail.com on 1 Oct 2010 at 3:23

• Changed state: Fixed