Closed Bug 552733 Opened 16 years ago Closed 16 years ago

Wrap all loose functions (e.g. in browser.js) in namespace object to prevent collisions

Categories

(Mozilla Labs Graveyard :: Test Pilot, defect, P1)

Product:
Mozilla Labs Graveyard
Component:
Test Pilot
Type:
defect

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: jono, Assigned: jono)

Details

Jorge: "You don't have proper namespacing in your main overlay code. The file browser.js has a number of loose functions with generic names that could cause conflicts with other add-ons or functions in Firefox itself. You should wrap all of these functions into a namespace object with a unique name, like 'TestPilot' for example."
Priority: -- → P1
Target Milestone: 0.1 → 1.0
Done in http://hg.mozilla.org/labs/testpilot/rev/3337df9495f0
Status: NEW → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Product: Mozilla Labs → Mozilla Labs Graveyard
You need to log in before you can comment on or make changes to this bug.