test_gjslint: Added flight-collection.js
test_gjslint: Disabled blank_lines_at_top_level check

Having two blank lines between the top level method makes sense, but
requiring three blank lines at the beginning of the file because the first
function is a @constructor does not make any sense.
Merge pull request #29 from Turbo87/gjslint

Closure lint improvements
baro.js: Converted module to functional inheritance
baro.js: Added placeholder parameters
baro.js: Added documentation
js: Reduced indentation

We don't need the additional scope
units.js: Converted module to functional inheritance
units.js: Reordered module members
units.js: Removed unnecessary @expose directives
map-click-handler.js: Converted module to functional inheritance
map-click-handler.js: Removed unnecessary @expose directives
map-click-handler.js: Added missing documentation to the trigger() method
phase-table.js: Converted module to functional inheritance
phase-table.js: Added documentation
phase-table.js: Removed unnecessary @expose directives
fix-table.js: Converted module to functional inheritance
fix-table: Adjusted/Added documentation
fix-table.js: Removed unnecessary @expose directives
flight-collection: Converted module to functional inheritance