User:Ctalbert:QA Extension

From MozillaWiki
Jump to navigation Jump to search

Notes for Cleanup

I can help you configure Komodo to help with some of this. But here are the guidelines for the cleanup effort.

  • Be sure that all licenses conform to the standards in the boilerplate
  • Replace with 2 space indenting wherever possible
  • Fold lines at the 80 character mark where it does not impair functionality and readability.
  • File encodings should be UTF-8
  • Line endings should be LF
  • Remove all tabs, replace with 2 space indenting
  • All files end with LF
  • Two blank lines are usually unnecessary
  • No blank lines of spaces only.
  • No dangling spaces between the end of the text on a line and the LF


Files needing cleanup

  • jar-chrome.manifest - tabs
  • chrome.manifest - tabs
  • build.xml - tabs empty lines
  • chrome/content/accountcreate.js - license block, tabs
  • chrome/content/browserOverlays.xul - tabs
  • chrome/content/common.js - tabs, line folding
  • chrome/content/litmus.js - tabs, line folding
  • chrome/content/litmusReporter.js - tabs, license block, tabs
  • chrome/content/notifications.js - tabs, line folding
  • chrome/content/prefs.js tabs
  • chrome/content/qa.js - tabs, folding
  • chrome/content/settings.js - tabs, line folding
  • chrome/content/setup.xul - tabs, no ending line
  • chrome/content/tabs/bugAccess.js - email for David Hamp-CGonsalves, tabs
  • chrome/content/tabs/bugzilla.js - one tab
  • chrome/content/tabs/bugzilla.xul - tabs, folding
  • chrome/content/tabs/chat.xul - one tab
  • chrome/content/tabs/litmus.xul - tabs
  • chrome/content/tabs/qmo.js - tabs
  • chrome/content/tabs/qmo.xul - tabs
  • chrome/content/tabs/selecttests.js - tabs, folding
  • chrome/content/tabs/selecttests.xul - tabs
  • chrome/skin/browserOverlays.css - one tab, no license header
  • chrome/skin/qa.css.new -- needed?
  • chrome/skin/qa.css.old -- needed?
  • chrome/locale/en-US/qa.dtd - tabs
  • platform/Darwin/chrome/skin/platform.css - no license header
  • platform/WINNT/chrome/skin/platform.css - Why do we have to include a blank file? Why not just allow the standard one to win? Can we remove this file?