Reformatting to MoFo coding style

34 files changed, 1250 insertions, 1075 deletions

diff --git a/data/lib/addAttachmentRow.js b/data/lib/addAttachmentRow.js
index 204192e..a2b67ee 100644
--- a/data/lib/addAttachmentRow.js
+++ b/data/lib/addAttachmentRow.js
@@ -3,39 +3,41 @@
 "use strict";
 
 function addAttachmentCallback(resp) {
-  var newAttachID = parseInt(resp.params.param.value.array.data.value.int, 10);
+  var newAttachID = parseInt(
+      resp.params.param.value.array.data.value.int, 10);
   console.log("attachID = " + newAttachID);
   // FIXME callback.call(param, newAttachID, data.length);
 }
 
 /**
- *
- * This has to stay in RHBugzillaPage because upstream doesn't have addAttachment
- * XML-RPC call yet.
+ * 
+ * This has to stay in RHBugzillaPage because upstream doesn't have
+ * addAttachment XML-RPC call yet.
  */
 function addAttachment(data, callback, param) {
   var params = [];
 
   if (!constantData.passwordState.passAvailable) {
-    console.error("addAttachment : No password, no XML-RPC calls; sorry");
+    console
+        .error("addAttachment : No password, no XML-RPC calls; sorry");
     return null;
   }
 
   params.push(getBugNo());
   params.push({
-    description: titleParsedAttachment,
-    filename: "parsed-backtrace.txt",
-    contenttype: "text/plain",
-    data: window.btoa(data),
-    nomail: true
+    description : titleParsedAttachment,
+    filename : "parsed-backtrace.txt",
+    contenttype : "text/plain",
+    data : window.btoa(data),
+    nomail : true
   });
 
   self.postMessage(new Message("MakeXMLRPCall", {
-    url: constantData.XMLRPCData[window.location.hostname].url,
-    login: getLogin(),
-    method: "bugzilla.addAttachment",
-    params: params,
-    callRPC: "AddAttachmentCallback"
+    url : constantData.XMLRPCData[window.location.hostname].url,
+    login : getLogin(),
+    method : "bugzilla.addAttachment",
+    params : params,
+    callRPC : "AddAttachmentCallback"
   }));
   reqCounter++;
 }
diff --git a/data/lib/addNewLinks.js b/data/lib/addNewLinks.js
index 02bd658..b8e7bd2 100644
--- a/data/lib/addNewLinks.js
+++ b/data/lib/addNewLinks.js
@@ -36,35 +36,42 @@
  * ***** END LICENSE BLOCK ***** */
 
 function addNewLinks(d) {
-  var product = d.querySelector("#field_container_product option[selected]");
+  var product = d
+      .querySelector("#field_container_product option[selected]");
   var component = d.querySelector("#component option[selected]");
 
   if (product) {
     var label = d.getElementById('field_container_product');
-    var url = 'enter_bug.cgi?product=' + encodeURIComponent(product.value);
+    var url = 'enter_bug.cgi?product='
+        + encodeURIComponent(product.value);
     if (label) {
-      createDeadLink("file_new_bug_product", "new", label,
-        url, [], "parens");
+      createDeadLink("file_new_bug_product", "new", label, url,
+          [], "parens");
     }
   }
 
   if (product && component) {
     var select = d.querySelector("select#component");
     var label = select.parentNode;
-    var url = 'enter_bug.cgi?product=' + encodeURIComponent(product.value) +
-      '&component=' + encodeURIComponent(component.value);
+    var url = 'enter_bug.cgi?product='
+        + encodeURIComponent(product.value) + '&component='
+        + encodeURIComponent(component.value);
     if (label) {
-      var componentElement = document.getElementById("bz_component_input");
+      var componentElement = document
+          .getElementById("bz_component_input");
       if (componentElement) { // We are in the Red Hat bugzilla
-      // do we have components list visible?
-        if (document.getElementById('bz_component_input').
-          classList.contains("bz_default_hidden")) {
-            label = document.getElementById("bz_component_edit_container");
-          }
-      } else {
+        // do we have components list visible?
+        if (document.getElementById('bz_component_input').classList
+            .contains("bz_default_hidden")) {
+          label = document
+              .getElementById("bz_component_edit_container");
+        }
+      }
+      else {
         label = document.getElementById('component').parentNode;
       }
-      createDeadLink("file_new_bug_component", "new", label, url, [], "parens");
+      createDeadLink("file_new_bug_component", "new", label,
+          url, [], "parens");
     }
   }
 }
diff --git a/data/lib/bug-page-mod.js b/data/lib/bug-page-mod.js
index e29b1bc..a405edb 100644
--- a/data/lib/bug-page-mod.js
+++ b/data/lib/bug-page-mod.js
@@ -43,7 +43,8 @@ function tweakBugzilla(d) {
     // Put the quicksearch text in the quicksearch boxes
     quicksearchHandler(d);
 
-    if (!d.getElementById("comments")) // don't process the mid-air collision pages
+    if (!d.getElementById("comments")) // don't process the mid-air collision
+                                        // pages
       return;
 
     // Make the comment box bigger
@@ -270,7 +271,8 @@ function tweakBugzilla(d) {
                     item = historyItems[++i].querySelectorAll("td")
                     ccPrefix = (trimContent(item[0]) == 'CC') ?
                         '<span class="bztw_cc bztw_historyitem">' : '<span class="bztw_historyitem">';
-                    // avoid showing a trailing semicolon if the previous entry wasn't a CC and this one is
+                    // avoid showing a trailing semicolon if the previous entry
+                    // wasn't a CC and this one is
                     var prefix = ccSuffix + ccPrefix;
                     // check to see if this is a flag setting
                     flagsFound = findFlag(item);
@@ -594,7 +596,8 @@ AttachmentFlagHandlerCtor.prototype = {
                             }
                         }
 
-                        // try to put the flag name and type part in a span which we will
+                        // try to put the flag name and type part in a span
+                        // which we will
                         // use in setupLinks to inject links into.
                         match = this._reLinkifyInterestingFlag.exec(text);
                         if (match) {
@@ -626,7 +629,8 @@ AttachmentFlagHandlerCtor.prototype = {
             if (!(id in this._db)) {
                 this._db[id] = [];
             }
-            name = name.split('@')[0]; // convert the name to the fraction before the @
+            name = name.split('@')[0]; // convert the name to the fraction
+                                        // before the @
             var added = this._parseData(name, trimContent(item[base + 2]));
             for (var i = 0; i < added.length; ++i) {
                 var flag = added[i];
diff --git a/data/lib/bugzillaDOMFunctions.js b/data/lib/bugzillaDOMFunctions.js
index 8392b87..f58f0d9 100644
--- a/data/lib/bugzillaDOMFunctions.js
+++ b/data/lib/bugzillaDOMFunctions.js
@@ -4,13 +4,13 @@
 
 /**
  * Select option with given value on the <SELECT> element with given id.
- *
+ * 
  * Also execute change HTMLEvent, so that the form behaves accordingly.
- *
+ * 
  * @param id
  * @param label
  * @return none
- *
+ * 
  */
 function selectOption (id, label, fireEvent) {
   if (!fireEvent) {
@@ -48,10 +48,12 @@ function selectOptionByLabel(id, label, fireEvent) {
 
 /**
  * Add object to the text box (comment box or status whiteboard)
- *
- * @param id String with the id of the element
- * @param stuff String/Array to be added to the comment box
- *
+ * 
+ * @param id
+ *          String with the id of the element
+ * @param stuff
+ *          String/Array to be added to the comment box
+ * 
  * @return none
  */
 function addStuffToTextBox (id, stuff) {
@@ -67,9 +69,11 @@ function addStuffToTextBox (id, stuff) {
 
 /**
  * Remove a keyword from the element if it is there
- *
- * @param id String with the id of the element
- * @param stuff String/Array with keyword(s) to be removed
+ * 
+ * @param id
+ *          String with the id of the element
+ * @param stuff
+ *          String/Array with keyword(s) to be removed
  */
 function removeStuffFromTextBox (id, stuff) {
   var changedElement = document.getElementById(id);
@@ -78,9 +82,11 @@ function removeStuffFromTextBox (id, stuff) {
 
 /**
  * generalized hasKeyword ... search in the value of the box with given id
- *
- * @param id String with ID of the element we want to check
- * @param str String to be searched for
+ * 
+ * @param id
+ *          String with ID of the element we want to check
+ * @param str
+ *          String to be searched for
  * @return Boolean found?
  */
 function idContainsWord (id, str) {
@@ -96,8 +102,9 @@ function idContainsWord (id, str) {
 
 /**
  * Check for the presence of a keyword
- *
- * @param str String with the keyword
+ * 
+ * @param str
+ *          String with the keyword
  * @return Boolean
  */
 function hasKeyword (str) {
@@ -106,11 +113,12 @@ function hasKeyword (str) {
 
 /**
  * Set the bug to NEEDINFO state
- *
+ * 
  * Working function.
+ * 
  * @return none
- * @todo TODO we may extend this to general setNeedinfo function
- * with parameter [reporter|assignee|general-email-address]
+ * @todo TODO we may extend this to general setNeedinfo function with parameter
+ *       [reporter|assignee|general-email-address]
  */
 function setNeedinfoReporter () {
   clickMouse("needinfo");
@@ -118,7 +126,7 @@ function setNeedinfoReporter () {
 }
 
 /**
- *
+ * 
  */
 function getOwner () {
   // TODO(maemo) doesn't work on maemo
@@ -127,9 +135,9 @@ function getOwner () {
 }
 
 /**
- * Return maintainer which is per default by bugzilla
- * (which is not necessarily the one who is default maintainer per component)
- *
+ * Return maintainer which is per default by bugzilla (which is not necessarily
+ * the one who is default maintainer per component)
+ * 
  * @return String with the maintainer's email address
  */
 function getDefaultBugzillaMaintainer (component) {
@@ -139,11 +147,15 @@ function getDefaultBugzillaMaintainer (component) {
 /**
  * Generic function to add new button to the page. Actually copies new button
  * from the old one (in order to have the same look-and-feel, etc.
- *
- * @param location Object around which the new button will be added
- * @param after Boolean before or after location ?
- * @param pkg String which package to take the command from
- * @param id String which command to take
+ * 
+ * @param location
+ *          Object around which the new button will be added
+ * @param after
+ *          Boolean before or after location ?
+ * @param pkg
+ *          String which package to take the command from
+ * @param id
+ *          String which command to take
  * @return none
  */
 function createNewButton (location, after, cmdObj) {
@@ -195,7 +207,7 @@ function createNewButton (location, after, cmdObj) {
 
 /**
  * Get the current title of the bug
- *
+ * 
  * @return string
  */
 function getSummary() {
@@ -204,7 +216,7 @@ function getSummary() {
 
 /**
  * Get the current title of the bug
- *
+ * 
  * @return string
  */
 function getSeverity() {
@@ -213,7 +225,7 @@ function getSeverity() {
 
 /**
  * Get the current email of the reporter of the bug.
- *
+ * 
  * @return string
  */
 function getReporter () {
@@ -222,7 +234,8 @@ function getReporter () {
   if (!reporterElement) {
     reporterElement = getOptionTableCell("bz_show_bug_column_1", "Reported", true);
   }
-  // Maemo calls the label "Reporter" and it doesn't have ids on table columns ... TODO(maemo)
+  // Maemo calls the label "Reporter" and it doesn't have ids on table columns
+  // ... TODO(maemo)
   return parseMailto(reporterElement);
 }
 
@@ -252,11 +265,10 @@ function commentsWalker (fce) {
 
 /**
  * collect the list of attachments in a structured format
- *
- * @return Array of arrays, one for each attachments;
- * each record has string name of the attachment, integer its id number,
- *     string of MIME type, integer of size in kilobytes, and the whole
- *     element itself
+ * 
+ * @return Array of arrays, one for each attachments; each record has string
+ *         name of the attachment, integer its id number, string of MIME type,
+ *         integer of size in kilobytes, and the whole element itself
  */
 function getAttachments () {
   var outAtts = [];
@@ -270,7 +282,7 @@ function getAttachments () {
 
 /**
  * Get login of the currently logged-in user.
- *
+ * 
  * @return String with the login name of the currently logged-in user
  */
 function getLogin () {
@@ -282,9 +294,10 @@ function getLogin () {
 
 /**
  * adds a person to the CC list, if it isn't already there
- *
- * @param who String with email address or "self" if the current user
- * of the bugzilla should be added
+ * 
+ * @param who
+ *          String with email address or "self" if the current user of the
+ *          bugzilla should be added
  */
 function addToCCList (who) {
   if (!who) {
@@ -303,7 +316,7 @@ function addToCCList (who) {
 
 /**
  * a collect a list of emails on CC list
- *
+ * 
  * @return Array with email addresses as Strings.
  */
 function getCCList () {
@@ -319,14 +332,16 @@ function getCCList () {
 
 /**
  * remove elements from the page based on their IDs
- *
- * @param doc Document object
- * @param target String/Array with ID(s)
- * @param remove Boolean indicating whether the node should be
- * actually removed or just hidden.
- * @return none
- * TODO remove parameter could be replaced by function which would
- * do actual activity.
+ * 
+ * @param doc
+ *          Document object
+ * @param target
+ *          String/Array with ID(s)
+ * @param remove
+ *          Boolean indicating whether the node should be actually removed or
+ *          just hidden.
+ * @return none TODO remove parameter could be replaced by function which would
+ *         do actual activity.
  */
 function killNodes(doc, target, remove) {
   var targetArr = target instanceof Array ? target : target.trim().split(/[,\s]+/);
@@ -343,7 +358,7 @@ function killNodes(doc, target, remove) {
 
 /**
  * Is this bug a RHEL bug?
- *
+ * 
  * @return Boolean true if it is a RHEL bug
  */
 function isEnterprise() {
@@ -355,7 +370,7 @@ function isEnterprise() {
 
 /**
  * Find out whether the bug is needed an attention of bugZappers
- *
+ * 
  * @return Boolean whether the bug has been triaged or not
  */
 function isTriaged() {
@@ -364,8 +379,9 @@ function isTriaged() {
 
 /**
  * Return string with the ID for the external_id SELECT for external bugzilla
- *
- * @param URLhostname String hostname of the external bugzilla
+ * 
+ * @param URLhostname
+ *          String hostname of the external bugzilla
  * @return String with the string for the external_id SELECT
  */
 function getBugzillaName(URLhostname, bzLabelNames) {
diff --git a/data/lib/bzpage.js b/data/lib/bzpage.js
index 3a832ea..9e65794 100644
--- a/data/lib/bzpage.js
+++ b/data/lib/bzpage.js
@@ -19,8 +19,8 @@ var constantData = {}; // This should be probably eliminated ASAP or
                        // or done by other means. TODO
 var equivalentComponents = null;
 /**
-* central handler processing messages from the main script.
-*/
+ * central handler processing messages from the main script.
+ */
 onMessage = function onMessage(msg) {
   console.log("onMessage - incoming : msg.cmd = " + msg.cmd);
   switch (msg.cmd) {
@@ -53,15 +53,17 @@ onMessage = function onMessage(msg) {
 };
 
 /**
- * @param cmd Object with all commands to be executed
- *
- * PROBLEM: according to https://developer.mozilla.org/en/Core_JavaScript_1.5_Reference\
- * /Statements/for...in there is no guaranteed order of execution of
- * commands (i.e., key, commentObj[key] pairs) in for..in cycle.
- * According to https://developer.mozilla.org/en/Core_JavaScript_1.5_Reference\
- * /Operators/Special_Operators/delete_Operator#Cross-browser_issues it seems that
- * everywhere except of Internet Explorer this should work well, but waiting
- * impatiently when this bite us.
+ * @param cmd
+ *          Object with all commands to be executed
+ * 
+ * PROBLEM: according to
+ * https://developer.mozilla.org/en/Core_JavaScript_1.5_Reference\
+ * /Statements/for...in there is no guaranteed order of execution of commands
+ * (i.e., key, commentObj[key] pairs) in for..in cycle. According to
+ * https://developer.mozilla.org/en/Core_JavaScript_1.5_Reference\
+ * /Operators/Special_Operators/delete_Operator#Cross-browser_issues it seems
+ * that everywhere except of Internet Explorer this should work well, but
+ * waiting impatiently when this bite us.
  */
 function executeCommand(cmdObj) {
   for (var key in cmdObj) {
@@ -71,9 +73,11 @@ function executeCommand(cmdObj) {
 
 /**
  * Actual execution function
- *
- * @param cmdLabel String with the name of the command to be executed
- * @param cmdParams Object with the appropriate parameters for the command
+ * 
+ * @param cmdLabel
+ *          String with the name of the command to be executed
+ * @param cmdParams
+ *          Object with the appropriate parameters for the command
  */
 function centralCommandDispatch (cmdLabel, cmdParams) {
   console.log("centralCommandDispatch : cmdLabel = " + cmdLabel);
@@ -180,10 +184,11 @@ function centralCommandDispatch (cmdLabel, cmdParams) {
 
 /**
  * Change assignee of the bug
- *
- * @param newAssignee String with the email address of new assigneeAElement
- *    or 'default' if the component's default assignee should be used.
- *    Value null clears "Reset Assignee to default for component" checkbox
+ * 
+ * @param newAssignee
+ *          String with the email address of new assigneeAElement or 'default'
+ *          if the component's default assignee should be used. Value null
+ *          clears "Reset Assignee to default for component" checkbox
  * @return none
  */
 function changeAssignee (newAssignee) {
@@ -223,11 +228,12 @@ function changeAssignee (newAssignee) {
 
 /**
  * Adds new option to the 'comment_action' scroll down box
- *
- * @param pkg String package name
- * @param cmd String with the name of the command
- * If the 'comment_action' scroll down box doesn't exist, this
- * function will set up new one.
+ * 
+ * @param pkg
+ *          String package name
+ * @param cmd
+ *          String with the name of the command If the 'comment_action' scroll
+ *          down box doesn't exist, this function will set up new one.
  */
 function addToCommentsDropdown (cmdObj) {
   var select = document.getElementById("comment_action");
@@ -302,7 +308,8 @@ function generateButtons (pkgs, kNodes) {
       }
     }
   }
-  // TODO This is weird in this place, but that's the place where all constantData etc.
+  // TODO This is weird in this place, but that's the place where all
+  // constantData etc.
   // are finally defined and available.
   if (RHBZinit) {
     RHBZinit();
@@ -326,7 +333,7 @@ function setConfigurationButton () {
 
 /**
  * dd
- *
+ * 
  * @return Element with the href attribute containng the information
  */
 function getOptionTableCell(tableId, label) {
@@ -349,12 +356,13 @@ function getOptionTableCell(tableId, label) {
 
 /**
  * Parse the row with the attachment
- *
- * @param DOM element to be parsed
+ * 
+ * @param DOM
+ *          element to be parsed
  * @return array with string name of the attachment, integer its id number,
- *     string of MIME type, integer of size in kilobytes, and the whole
- *     element itself
- *
+ *         string of MIME type, integer of size in kilobytes, and the whole
+ *         element itself
+ * 
  * TODO error handling is missing ... it's bleee
  */
 function parseAttachmentLine(inElem) {
@@ -399,7 +407,8 @@ function startup() {
   // Prepare for query buttons
   // element ID brElementPlace_location is later used in JSON files
   // Stay with this add_comment element even if RH BZ upgrades, this seems
-  // to be generally much more stable (even with other bugzillas, e.g. b.gnome.org)
+  // to be generally much more stable (even with other bugzillas, e.g.
+  // b.gnome.org)
   // then some getElementById.
   var commentArea = document.getElementsByName("add_comment")[0].parentNode;
   if (commentArea) {
@@ -412,7 +421,8 @@ function startup() {
     }
   }
 
-  // TODO Probably could be ignored ... used only once on line 973 of rhbzpage.js
+  // TODO Probably could be ignored ... used only once on line 973 of
+  // rhbzpage.js
   // if (parseAbrtBacktraces && this.RE.Abrt.test(getSummary())) {
   // title = document.getElementById("short_desc_nonedit_display").textContent;
 
diff --git a/data/lib/checkin-context.js b/data/lib/checkin-context.js
index 8ae482d..0ccec0c 100644
--- a/data/lib/checkin-context.js
+++ b/data/lib/checkin-context.js
@@ -1,11 +1,13 @@
 self.on('click', function(node, data) {
-  var message = document.getElementById("__bz_tw_checkin_comment");
+  var message = document
+      .getElementById("__bz_tw_checkin_comment");
   self.postMessage(message.textContent);
 });
 
 self.on('context', function(node) {
   if (!onBugzillaPage(document.URL))
     return false;
-  var message = document.getElementById("__bz_tw_checkin_comment");
+  var message = document
+      .getElementById("__bz_tw_checkin_comment");
   return !!message;
 });
diff --git a/data/lib/color.js b/data/lib/color.js
index 163071e..4d0c41f 100644
--- a/data/lib/color.js
+++ b/data/lib/color.js
@@ -51,10 +51,13 @@ Color.prototype.toString = function() {
  * Converts an RGB color value to HSL. Conversion formula adapted from
  * http://en.wikipedia.org/wiki/HSL_color_space. Assumes r, g, and b are
  * contained in the set [0, 255] and returns h, s, and l in the set [0, 1].4343
- *
- * @param Number r The red color value
- * @param Number g The green color value
- * @param Number b The blue color value
+ * 
+ * @param Number
+ *          r The red color value
+ * @param Number
+ *          g The green color value
+ * @param Number
+ *          b The blue color value
  * @return Array The HSL representation
  */
 Color.prototype.hsl = function() {
@@ -84,17 +87,22 @@ Color.prototype.hsl = function() {
     h /= 6;
   }
 
-  return [ h, s, l ];
+  return [
+      h, s, l
+  ];
 };
 
 /**
  * Converts an HSL color value to RGB. Conversion formula adapted from
  * http://en.wikipedia.org/wiki/HSL_color_space. Assumes h, s, and l are
  * contained in the set [0, 1] and returns r, g, and b in the set [0, 255].
- *
- * @param Number h The hue
- * @param Number s The saturation
- * @param Number l The lightness
+ * 
+ * @param Number
+ *          h The hue
+ * @param Number
+ *          s The saturation
+ * @param Number
+ *          l The lightness
  * @return Array The RGB representation
  */
 Color.prototype.hslToRgb = function(h, s, l) {
@@ -130,17 +138,22 @@ Color.prototype.hslToRgb = function(h, s, l) {
     b = hue2rgb(p, q, h - 1 / 3);
   }
 
-  return [ r * 255, g * 255, b * 255 ];
+  return [
+      r * 255, g * 255, b * 255
+  ];
 };
 
 /**
  * Converts an RGB color value to HSV. Conversion formula adapted from
  * http://en.wikipedia.org/wiki/HSV_color_space. Assumes r, g, and b are
  * contained in the set [0, 255] and returns h, s, and v in the set [0, 1].
- *
- * @param Number r The red color value
- * @param Number g The green color value
- * @param Number b The blue color value
+ * 
+ * @param Number
+ *          r The red color value
+ * @param Number
+ *          g The green color value
+ * @param Number
+ *          b The blue color value
  * @return Array The HSV representation
  */
 Color.prototype.hsv = function() {
@@ -171,17 +184,22 @@ Color.prototype.hsv = function() {
     h /= 6;
   }
 
-  return [ h, s, v ];
+  return [
+      h, s, v
+  ];
 };
 
 /**
  * Converts an HSV color value to RGB. Conversion formula adapted from
  * http://en.wikipedia.org/wiki/HSV_color_space. Assumes h, s, and v are
  * contained in the set [0, 1] and returns r, g, and b in the set [0, 255].
- *
- * @param Number h The hue
- * @param Number s The saturation
- * @param Number v The value
+ * 
+ * @param Number
+ *          h The hue
+ * @param Number
+ *          s The saturation
+ * @param Number
+ *          v The value
  * @return Array The RGB representation
  */
 Color.prototype.hsvToRgb = function(h, s, v) {
@@ -226,7 +244,9 @@ Color.prototype.hsvToRgb = function(h, s, v) {
     break;
   }
 
-  return [ r * 255, g * 255, b * 255 ];
+  return [
+      r * 255, g * 255, b * 255
+  ];
 };
 
 /**
diff --git a/data/lib/fixingAttMIME.js b/data/lib/fixingAttMIME.js
index a016fe6..365cfae 100644
--- a/data/lib/fixingAttMIME.js
+++ b/data/lib/fixingAttMIME.js
@@ -5,17 +5,15 @@ var reqCounter = 0; // TODO should be probably a dict indexed by called method
 
 /**
  * Callback function for the XMLRPC request
- *
- * @param ret Object with xmlhttprequest response with attributes:
- * + status -- int return code
- * + statusText
- * + responseHeaders
- * + responseText
+ * 
+ * @param ret
+ *          Object with xmlhttprequest response with attributes: + status -- int
+ *          return code + statusText + responseHeaders + responseText
  */
 function XMLRPCcallback() {
   reqCounter--;
   if (reqCounter <= 0) {
-    setTimeout(function () {
+    setTimeout(function() {
       window.location.reload(true);
     }, 1000);
   }
@@ -24,24 +22,27 @@ function XMLRPCcallback() {
 /**
  * The worker function -- call XMLRPC to fix MIME type of the particular
  * attachment
- *
- * @param id Integer with the attachment id to be fixed
- * @param type String with the new MIME type, optional defaults to "text/plain"
- * @param email Boolean whether email should be sent to appropriate person;
- *      option, defaults to false
- *
+ * 
+ * @param id
+ *          Integer with the attachment id to be fixed
+ * @param type
+ *          String with the new MIME type, optional defaults to "text/plain"
+ * @param email
+ *          Boolean whether email should be sent to appropriate person; option,
+ *          defaults to false
+ * 
  * updateAttachMimeType($data_ref, $username, $password)
- *
+ * 
  * Update the attachment mime type of an attachment. The first argument is a
  * data hash containing information on the new MIME type and the attachment id
  * that you want to act on.
- *
+ * 
  * $data_ref = { "attach_id" => "<Attachment ID>", # Attachment ID to perform
  * MIME type change on. "mime_type" => "<New MIME Type Value>", # Legal MIME
  * type value that you want to change the attachment to. "nomail" => 0, #
  * OPTIONAL Flag that is either 1 or 0 if you want email to be sent or not for
  * this change };
- *
+ * 
  */
 function fixAttachById(id, XMLRPCURL, type, email) {
   var params = [];
@@ -63,18 +64,18 @@ function fixAttachById(id, XMLRPCURL, type, email) {
   });
 
   self.postMessage(new Message("MakeXMLRPCall", {
-    url: XMLRPCURL,
-    login: getLogin(),
-    method: "bugzilla.updateAttachMimeType",
-    params: params,
-    callRPC: "FixAttachmentMIMECallback"
+    url : XMLRPCURL,
+    login : getLogin(),
+    method : "bugzilla.updateAttachMimeType",
+    params : params,
+    callRPC : "FixAttachmentMIMECallback"
   }));
   reqCounter++;
 }
 
 /**
  * Add a link to the bad attachment for fixing it.
- *
+ * 
  * @param
  * <TR> DOM jQuery element with a bad attachment
  * @return none
@@ -82,6 +83,8 @@ function fixAttachById(id, XMLRPCURL, type, email) {
 function addTextLink(row, xmlRpcUrl) {
   var elemS = row[4].getElementsByTagName("td");
   var elem = elemS[elemS.length - 1];
-  createDeadLink("addFix2TextLink", "text", elem,
-      fixAttachById, [row[1], xmlRpcUrl], "br");
+  createDeadLink("addFix2TextLink", "text", elem, fixAttachById,
+      [
+          row[1], xmlRpcUrl
+      ], "br");
 }
diff --git a/data/lib/jumpNextBug.js b/data/lib/jumpNextBug.js
index 76e6657..b777fe8 100644
--- a/data/lib/jumpNextBug.js
+++ b/data/lib/jumpNextBug.js
@@ -4,40 +4,46 @@
 
 (function createRelationElements() {
   var relation = {};
-  var linkLabels = ["First", "Last", "Prev", "Next"];
+  var linkLabels = [
+      "First", "Last", "Prev", "Next"
+  ];
   var labelToRelation = {
-    "First": {
-      rel: "start"
+    "First" : {
+      rel : "start"
     },
-    "Last": {
-      rel: "last"
+    "Last" : {
+      rel : "last"
     },
-    "Prev": {
-      rel: "prev"
+    "Prev" : {
+      rel : "prev"
     },
-    "Next": {
-      rel: "next"
+    "Next" : {
+      rel : "next"
     }
   };
 
-  function createLinkRel (rel, href) {
+  function createLinkRel(rel, href) {
     var newLinkElement = document.createElement("link");
     newLinkElement.setAttribute("rel", rel);
     newLinkElement.setAttribute("href", href);
-    document.getElementsByTagName("head")[0].
-      appendChild(newLinkElement);
+    document.getElementsByTagName("head")[0]
+        .appendChild(newLinkElement);
   }
 
-  var aNavigElements = document.querySelectorAll("#bugzilla-body .navigation a");
+  var aNavigElements = document
+      .querySelectorAll("#bugzilla-body .navigation a");
   Array.forEach(aNavigElements, function(elem) {
-      var labelText = elem.textContent.trim();
-      if (isInList(labelText, linkLabels)) {
-        labelToRelation[labelText].href = elem.getAttribute("href");
-      };
+    var labelText = elem.textContent.trim();
+    if (isInList(labelText, linkLabels)) {
+      labelToRelation[labelText].href = elem
+          .getAttribute("href");
+    }
+    ;
   });
-  console.myDebug("labelToRelation = " + labelToRelation.toSource());
+  console.myDebug("labelToRelation = "
+      + labelToRelation.toSource());
 
-  for (var key in labelToRelation) {
+  for ( var key in labelToRelation) {
     if (labelToRelation.hasOwnProperty(key)) {
       relation = labelToRelation[key];
       if (relation.href) {
diff --git a/data/lib/logging-front.js b/data/lib/logging-front.js
index b8343a4..2f85a51 100644
--- a/data/lib/logging-front.js
+++ b/data/lib/logging-front.js
@@ -12,8 +12,7 @@ function addLogRecord() {
   rec.date = new Date();
   rec.url = document.location.toString();
   rec.title = document.title;
-  var comment = window.prompt(
-      "Enter comments for this comment");
+  var comment = window.prompt("Enter comments for this comment");
   if (comment !== null) {
     comment = comment.trim();
     if (comment.length > 0) {
@@ -22,8 +21,7 @@ function addLogRecord() {
       var dateStr = getISODate(rec.date);
       var urlStr = window.location.hostname;
       var bugNo = getBugNoFromURL(window.location.href);
-      rec.key = dateStr + "+" +
-        urlStr + "+" + bugNo;
+      rec.key = dateStr + "+" + urlStr + "+" + bugNo;
       self.postMessage(new Message("AddLogRecord", rec));
     }
     return rec;
@@ -33,52 +31,50 @@ function addLogRecord() {
 
 /**
  */
-function setUpLogging () {
+function setUpLogging() {
   // Protection against double-call
   if (document.getElementById("generateTSButton")) {
-    return ;
+    return;
   }
 
   // For adding additional buttons to the top toolbar
-  var additionalButtons = document.querySelector("#bugzilla-body *.related_actions");
+  var additionalButtons = document
+      .querySelector("#bugzilla-body *.related_actions");
   var that = this;
 
   // logging all submits for timesheet
   if (!submitHandlerInstalled) {
-    document.forms.namedItem("changeform").addEventListener("submit",function (evt) {
-      if (addLogRecord() === null) {
-        evt.stopPropagation();
-        evt.preventDefault();
-      }
-    }, false);
+    document.forms.namedItem("changeform").addEventListener(
+        "submit", function(evt) {
+          if (addLogRecord() === null) {
+            evt.stopPropagation();
+            evt.preventDefault();
+          }
+        }, false);
     submitHandlerInstalled = true;
   }
 
   // (id, text, parent, callback, params, before, covered, accesskey)
-  createDeadLink("generateTSButton", "Generate TS", additionalButtons,
-    function(evt) {
-      self.postMessage(new Message("GenerateTS"));
-    }, [], "dash", "li");
+  createDeadLink("generateTSButton", "Generate TS",
+      additionalButtons, function(evt) {
+        self.postMessage(new Message("GenerateTS"));
+      }, [], "dash", "li");
 
   createDeadLink("clearLogs", "Clear TS", additionalButtons,
-    function(evt) {
-      self.postMessage(new Message("ClearTS"));
-    }, [], "dash", "li");
+      function(evt) {
+        self.postMessage(new Message("ClearTS"));
+      }, [], "dash", "li");
 
-  createDeadLink("importTSButton", "Import TS", additionalButtons,
-    function(evt) {
-      self.postMessage(new Message("ImportTS"));
-    }, [], "dash", "li");
+  createDeadLink("importTSButton", "Import TS",
+      additionalButtons, function(evt) {
+        self.postMessage(new Message("ImportTS"));
+      }, [], "dash", "li");
 
-  /* TODO
-  var clearLogAElem = document.getElementById("clearLogs");
-  if (this.log.isEmpty()) {
-    clearLogAElem.style.color = this.log.EmptyLogsColor;
-    clearLogAElem.style.fontWeight = "normal";
-  }
-  else {
-    clearLogAElem.style.color = this.log.FullLogsColor;
-    clearLogAElem.style.fontWeight = "bolder";
-  }
-  */
+  /*
+   * TODO var clearLogAElem = document.getElementById("clearLogs"); if
+   * (this.log.isEmpty()) { clearLogAElem.style.color = this.log.EmptyLogsColor;
+   * clearLogAElem.style.fontWeight = "normal"; } else {
+   * clearLogAElem.style.color = this.log.FullLogsColor;
+   * clearLogAElem.style.fontWeight = "bolder"; }
+   */
 }
diff --git a/data/lib/makeBacktraceAttachment.js b/data/lib/makeBacktraceAttachment.js
index a6b6ab0..6011c06 100644
--- a/data/lib/makeBacktraceAttachment.js
+++ b/data/lib/makeBacktraceAttachment.js
@@ -3,69 +3,68 @@
 var titleParsedAttachment = "Part of the thread where crash happened";
 
 /**
- *
+ * 
  */
 function pasteBacktraceInComments(atts) {
   /*
-  Let's comment it out, and we'll see if anything breaks.
-  TODO This paragraph looks suspicous ... what is it?
-  Does it belong to this function?
-  var notedLabel = document.querySelector("label[for='newcc']");
-  while (notedLabel.firstChild) {
-    var node = notedLabel.removeChild(notedLabel.firstChild);
-    notedLabel.parentNode.insertBefore(node, notedLabel);
-  }
-  notedLabel.parentNode.removeChild(notedLabel);
-  */
+   * Let's comment it out, and we'll see if anything breaks. TODO This paragraph
+   * looks suspicous ... what is it? Does it belong to this function? var
+   * notedLabel = document.querySelector("label[for='newcc']"); while
+   * (notedLabel.firstChild) { var node =
+   * notedLabel.removeChild(notedLabel.firstChild);
+   * notedLabel.parentNode.insertBefore(node, notedLabel); }
+   * notedLabel.parentNode.removeChild(notedLabel);
+   */
 
   // FIXME BROKEN and its depending functions are even more broken
   return null;
 
-  var abrtQueryURL = "https://bugzilla.redhat.com/buglist.cgi?" +
-    "cmdtype=dorem&remaction=run&namedcmd=all%20NEW%20abrt%20crashes&"+
-		  "sharer_id=74116";
+  var abrtQueryURL = "https://bugzilla.redhat.com/buglist.cgi?"
+      + "cmdtype=dorem&remaction=run&namedcmd=all%20NEW%20abrt%20crashes&"
+      + "sharer_id=74116";
 
   var mainTitle = document
       .getElementsByClassName("bz_alias_short_desc_container")[0];
 
-  createDeadLink ("callAbrtQuery_link",
-    "Abrt bugs", mainTitle, abrtQueryURL, [], false, null, "a");
+  createDeadLink("callAbrtQuery_link", "Abrt bugs", mainTitle,
+      abrtQueryURL, [], false, null, "a");
 
   if (idContainsWord("cf_devel_whiteboard", 'btparsed')) {
     addStuffToTextBox('status_whiteboard', 'btparsed');
   }
 
-  if (!(isTriaged() || idContainsWord("status_whiteboard",
-      'btparsed') || (atts.length > 0))) {
-    var btAttachments = atts
-        .filter(function(att) {
-          return (/File: backtrace/.test(att[0]));
-      });
+  if (!(isTriaged()
+      || idContainsWord("status_whiteboard", 'btparsed') || (atts.length > 0))) {
+    var btAttachments = atts.filter(function(att) {
+      return (/File: backtrace/.test(att[0]));
+    });
     // TODO we need to go through all backtrace attachments, but
     // just the first one will do for now, we would need to do async
     //
-    btAttachments.forEach(function(x) {
-      var attURL = "https://bugzilla.redhat.com/attachment.cgi?id="
-          + x[1];
-      if ((!btSnippet) && // ???? FIXME
-          (!idContainsWord("status_whiteboard", 'btparsed'))) {
-        Request({
-          url: attURL,
-          onComplete: function(response) {
-            if (response.status == 200) {
-              btSnippet = parseBacktrace(response.text);
-              if (btSnippet) {
-                addCheckShowLink(x,btSnippet);
+    btAttachments
+        .forEach(
+            function(x) {
+              var attURL = "https://bugzilla.redhat.com/attachment.cgi?id="
+                  + x[1];
+              if ((!btSnippet) && // ???? FIXME
+              (!idContainsWord("status_whiteboard", 'btparsed'))) {
+                Request({
+                  url : attURL,
+                  onComplete : function(response) {
+                    if (response.status == 200) {
+                      btSnippet = parseBacktrace(response.text);
+                      if (btSnippet) {
+                        addCheckShowLink(x, btSnippet);
+                      }
+                    }
+                  }
+                }).get();
               }
-            }
-          }
-        }).get();
-      }
-    }, this);
+            }, this);
   }
   // Add "show BT" links
   if (parsedAttachments.length > 0) {
-    this.parsedAttachments.forEach(function (att) {
+    this.parsedAttachments.forEach(function(att) {
       addShowParsedBTLink(att);
     });
   }
@@ -73,85 +72,95 @@ function pasteBacktraceInComments(atts) {
 
 /**
  * Open new window with the content of the attachment.
- *
- * @param id Number of the attachment id
+ * 
+ * @param id
+ *          Number of the attachment id
  * @return none
  */
 function showAttachment(id) {
-  self.postMessage(new Message("OpenURLinPanel",
-    "https://" + window.location.hostname + "/attachment.cgi?id=" + id));
+  self.postMessage(new Message("OpenURLinPanel", "https://"
+      + window.location.hostname + "/attachment.cgi?id=" + id));
 }
 
 /**
  * add a link opening a window with the parsed backtrace
- *
- * @param att Attachment object
+ * 
+ * @param att
+ *          Attachment object
  */
 function addShowParsedBTLink(att) {
   var elem = att[4].querySelector("td:last-of-type");
-  createDeadLink("showParsedBacktraceWindow-" + att[1], "showBT",
-    elem, showAttachment, att[1], true);
+  createDeadLink("showParsedBacktraceWindow-" + att[1],
+      "showBT", elem, showAttachment, att[1], true);
 }
 
 /**
  * Unfinished ... see above FIXME BROKEN AND DOESN'T WORK
  */
-function addNewAttachmentRow(origAtt,
-    newAttId, newAttSize) {
+function addNewAttachmentRow(origAtt, newAttId, newAttSize) {
   var that = this;
-  var oldAddBTLink = document.getElementById("attachBacktraceActivator");
+  var oldAddBTLink = document
+      .getElementById("attachBacktraceActivator");
   oldAddBTLink.parentNode.removeChild(oldAddBTLink);
   var newTRElem = origAtt[4].cloneNode(true);
 
   // fix number of the attachment
-  Array.forEach(newTRElem.getElementsByTagName("a"), function (aEl) {
-    aEl.setAttribute("href",
-      aEl.getAttribute("href").replace(origAtt[1], newAttId));
+  Array.forEach(newTRElem.getElementsByTagName("a"), function(
+      aEl) {
+    aEl.setAttribute("href", aEl.getAttribute("href").replace(
+        origAtt[1], newAttId));
   });
 
   var aElements = newTRElem.getElementsByTagName("a");
-  aElements[0].setAttribute("name","parsed-backtrace.txt");
+  aElements[0].setAttribute("name", "parsed-backtrace.txt");
   aElements[0].getElementsByTagName("b")[0].textContent = titleParsedAttachment;
 
-  var sizeSpan = newTRElem.getElementsByClassName("bz_attach_extra_info")[0];
-  sizeSpan.textContent = "(" + (newAttSize / 1024).toFixed(2) + " KB, text/plain)";
+  var sizeSpan = newTRElem
+      .getElementsByClassName("bz_attach_extra_info")[0];
+  sizeSpan.textContent = "(" + (newAttSize / 1024).toFixed(2)
+      + " KB, text/plain)";
 
-  // aElements[1].textContent = new Date().toString(); TODO we should add eventually, but not pressing
+  // aElements[1].textContent = new Date().toString(); TODO we should add
+  // eventually, but not pressing
 
   var vcardSpan = newTRElem.getElementsByClassName("vcard")[0];
   if (vcardSpan !== undefined) {
     var vcardSpanClassList = vcardSpan.classList;
-    if (/@redhat\.com/.test(this.login) && !vcardSpanClassList.contains("redhat_user")) {
+    if (/@redhat\.com/.test(this.login)
+        && !vcardSpanClassList.contains("redhat_user")) {
       vcardSpanClassList.add("redhat_user");
     }
     var vcardAElem = vcardSpan.getElementsByTagName("a")[0];
     vcardAElem.setAttribute("title", this.login);
     vcardAElem.setAttribute("href", "mailto:" + this.login);
     vcardAElem.className = "email";
-    vcardAElem.innerHTML="<span class='fn'>" + this.login + "</span>";
+    vcardAElem.innerHTML = "<span class='fn'>" + this.login
+        + "</span>";
   }
 
   var elem = newTRElem.querySelector("td:last-of-type");
   this.createDeadLink("showBacktrace", "show BT", elem,
-    this.showAttachment, newAttId, false);
+      this.showAttachment, newAttId, false);
 
-  origAtt[4].parentNode.insertBefore(newTRElem, origAtt[4].nextSibling);
+  origAtt[4].parentNode.insertBefore(newTRElem,
+      origAtt[4].nextSibling);
 }
 
 /**
  * Add a link to create a new attachment with a parsed backtrace
- *
- * @param oldAtt Object with an attachment row
- * @param snippet String with parsed backtrace
+ * 
+ * @param oldAtt
+ *          Object with an attachment row
+ * @param snippet
+ *          String with parsed backtrace
  * @return none
  */
 function addCheckShowLink(oldAtt, snippet) {
   var elem = oldAtt[4].querySelector("td:last-of-type");
-/*
-  createDeadLink("attachBacktraceActivator", "add parsed BT", elem, function(x) {
-    // pass function and parameters as two separate parameters, the function to be called from
-    // addAttachment
-    addAttachment(snippet, addNewAttachmentRow, oldAtt);
-    }, [], true);
-*/
+  /*
+   * createDeadLink("attachBacktraceActivator", "add parsed BT", elem,
+   * function(x) { // pass function and parameters as two separate parameters,
+   * the function to be called from // addAttachment addAttachment(snippet,
+   * addNewAttachmentRow, oldAtt); }, [], true);
+   */
 }
diff --git a/data/lib/otherButtons.js b/data/lib/otherButtons.js
index 66b5a9b..ab8ec35 100644
--- a/data/lib/otherButtons.js
+++ b/data/lib/otherButtons.js
@@ -4,9 +4,9 @@
 
 /**
  * Set background color of all comments made by reporter in ReporterColor color
- *
+ * 
  */
-function checkComments () {
+function checkComments() {
   var reporter = getReporter();
   commentsWalker(function(x) {
     var email = parseMailto(x.getElementsByClassName("vcard")[0]
@@ -17,39 +17,39 @@ function checkComments () {
   });
 }
 
-function collectComments () {
+function collectComments() {
   var outStr = "";
   commentsWalker(function(x) {
-    outStr += x.getElementsByTagName("pre")[0].textContent + "\n";
+    outStr += x.getElementsByTagName("pre")[0].textContent
+        + "\n";
   });
   return outStr.trim();
 }
 
 /**
  * Find default assignee based on the current component
- *
- * @return String what would be a default assignee if
- * we haven't set it up.
+ * 
+ * @return String what would be a default assignee if we haven't set it up.
  */
 function getDefaultAssignee() {
   return filterByRegexp(constantData.defaultAssignee,
-    getComponent());
+      getComponent());
 }
 
 /**
  * Set default assignee
- *
- * @return none
- * sets this.defaultAssignee property according to defaultAssignee list
+ * 
+ * @return none sets this.defaultAssignee property according to defaultAssignee
+ *         list
  */
 function setDefaultAssignee() {
   var defAss = getDefaultAssignee();
 
   // Add setting default assignee
   if (defAss && (defAss !== getOwner())) {
-    createNewButton("bz_assignee_edit_container",true, {
-      "name": "Def. Assignee",
-      "assignee": "default"
+    createNewButton("bz_assignee_edit_container", true, {
+      "name" : "Def. Assignee",
+      "assignee" : "default"
     });
   }
 }
@@ -67,7 +67,7 @@ function markBugTriaged() {
     alert("This won't do! Specify some severity!");
   }
   else {
-    addStuffToTextBox("keywords","Triaged");
+    addStuffToTextBox("keywords", "Triaged");
   }
 }
 
@@ -78,12 +78,13 @@ function addingEmbelishments(list) {
   var maintCCAddr = "";
   if (constantData.CCmaintainer) {
     maintCCAddr = filterByRegexp(constantData.CCmaintainer,
-      getComponent())[0]; // filterByRegexp returns array, not string
+        getComponent())[0]; // filterByRegexp returns array, not string
   }
 
   // we should make visible whether maintCCAddr is in CCList
   if (maintCCAddr && isInList(maintCCAddr, getCCList())) {
-    var ccEditBoxElem = document.getElementById("cc_edit_area_showhide");
+    var ccEditBoxElem = document
+        .getElementById("cc_edit_area_showhide");
     ccEditBoxElem.style.color = "navy";
     ccEditBoxElem.style.fontWeight = "bolder";
     ccEditBoxElem.style.textDecoration = "underline";
@@ -93,8 +94,8 @@ function addingEmbelishments(list) {
   if (config.signature && config.signature.length > 0) {
     var signaturesCounter = 0;
     var signatureFedoraString = config.signature;
-    document.forms.namedItem("changeform").addEventListener("submit",
-        function(aEvt) {
+    document.forms.namedItem("changeform").addEventListener(
+        "submit", function(aEvt) {
           if (signaturesCounter < 1) {
             addStuffToTextBox("comment", signatureFedoraString);
             signaturesCounter += 1;
@@ -105,19 +106,19 @@ function addingEmbelishments(list) {
   // set default assignee on change of the component
   var compElement = document.getElementById("component");
   if (compElement && (compElement.options)) {
-    document.getElementById("component").addEventListener("change",
-        function() {
+    document.getElementById("component").addEventListener(
+        "change", function() {
           changeAssignee("default");
         }, false);
   }
 
   // TODO Get compiz bugs as well
-  if ((constantData.chipNames) &&
-      (list[0]) &&
-      (!FillMagicDoneRE.test(getSummary())) &&
-      (maintCCAddr === "xgl-maint@redhat.com")) {
+  if ((constantData.chipNames) && (list[0])
+      && (!FillMagicDoneRE.test(getSummary()))
+      && (maintCCAddr === "xgl-maint@redhat.com")) {
     // Add find chip magic button
-    var whiteboard_string = document.getElementById("status_whiteboard").value;
+    var whiteboard_string = document
+        .getElementById("status_whiteboard").value;
     if (!/card_/.test(whiteboard_string)) {
       fillInChipMagic(list[0][1]);
     }
@@ -126,10 +127,13 @@ function addingEmbelishments(list) {
 
 /**
  * Set branding colours to easily distinguish between Fedora and RHEL bugs
- *
- * @param brand String with product of the current bug
- * @param version String with the version of the bug
- * @param its String with the IsueTracker numbers
+ * 
+ * @param brand
+ *          String with product of the current bug
+ * @param version
+ *          String with the version of the bug
+ * @param its
+ *          String with the IsueTracker numbers
  * @return none
  */
 function setBranding(xLogAtts) {
@@ -147,8 +151,8 @@ function setBranding(xLogAtts) {
       brandColor = RHColor;
     }
   }
-  else if (new RegExp("Fedora").test(document.
-      getElementById("product").value)) {
+  else if (new RegExp("Fedora").test(document
+      .getElementById("product").value)) {
     if (document.getElementById("version").value === "rawhide") {
       brandColor = RawhideColor;
     }
@@ -161,7 +165,8 @@ function setBranding(xLogAtts) {
   document.getElementsByTagName("body")[0].style.background = brandColor
       .toString()
       + " none";
-  document.getElementById("titles").style.background = brandColor.toString()
+  document.getElementById("titles").style.background = brandColor
+      .toString()
       + " none";
 
   // Remove "Bug" from the title of the bug page, so we have more space with
@@ -169,7 +174,8 @@ function setBranding(xLogAtts) {
   var titleElem = document.getElementsByTagName("title")[0];
 
   titleElem.textContent = titleElem.textContent.slice(4);
-  var bodyTitleParent = document.getElementById("summary_alias_container").parentNode;
+  var bodyTitleParent = document
+      .getElementById("summary_alias_container").parentNode;
   var bodyTitleElem = bodyTitleParent.getElementsByTagName("b")[0];
   bodyTitleElem.textContent = bodyTitleElem.textContent.slice(4);
 
@@ -177,7 +183,8 @@ function setBranding(xLogAtts) {
   // for security bugs.
   if (hasKeyword("Security")) {
     document.getElementById("bugzilla-body").style.background = SalmonPink
-        .toString() + ' none';
+        .toString()
+        + ' none';
   }
 
   // Make it visible whether the bug has been triaged
@@ -194,6 +201,6 @@ function setBranding(xLogAtts) {
           .getElementById("bz_component_edit_container"))) {
     compElems.style.background = "red none";
   }
-  
+
   addingEmbelishments(xLogAtts);
 }
diff --git a/data/lib/preprocessDuplicates.js b/data/lib/preprocessDuplicates.js
index 44ea5ca..d312fb9 100644
--- a/data/lib/preprocessDuplicates.js
+++ b/data/lib/preprocessDuplicates.js
@@ -36,85 +36,97 @@
  * ***** END LICENSE BLOCK ***** */
 
 function preprocessDuplicateMarkers(mainDoc, histDoc) {
-    var comments = mainDoc.querySelectorAll(".bz_comment");
-    var reDuplicate = /^\s*\*\*\*\s+Bug\s+(\d+)\s+has\s+been\s+marked\s+as\s+a\s+duplicate\s+of\s+this\s+bug.\s+\*\*\*\s*$/i;
-    var row = 0;
-    var rows = histDoc.querySelectorAll("#bugzilla-body tr");
-    for (var i = 1 /* comment 0 can never be a duplicate marker */;
-         i < comments.length; ++i) {
-        var textHolder = comments[i].querySelector(".bz_comment_text");
-        var match = reDuplicate.exec(trimContent(textHolder));
-        if (match) {
-            // construct the table row to be injected in histDoc
-            var bugID = match[1];
-            var email = comments[i].querySelector(".bz_comment_user .email")
-                                   .href
-                                   .substr(7);
-            var link = textHolder.querySelector("a");
-            var title = link.title;
-            var time = trimContent(comments[i].querySelector(".bz_comment_time"));
-            var what = 'Duplicate';
-            var removed = '';
-            var number = trimContent(comments[i].querySelector(".bz_comment_number")).
-                         replace(/[^\d]+/g, '');
-            var class_ = '';
-            if (/bz_closed/i.test(link.className + " " + link.parentNode.className)) {
-                class_ += 'bz_closed ';
-            }
-            if (link.parentNode.tagName.toLowerCase() == 'i') {
-                class_ += 'bztw_unconfirmed ';
-            }
-            var added = '<a href="show_bug.cgi?id=' + bugID + '" title="' +
-                        htmlEncode(title) + '" name="c' + number + '" class="' + class_ +
-                        '">' + bugID + '</a>';
+  var comments = mainDoc.querySelectorAll(".bz_comment");
+  var reDuplicate = /^\s*\*\*\*\s+Bug\s+(\d+)\s+has\s+been\s+marked\s+as\s+a\s+duplicate\s+of\s+this\s+bug.\s+\*\*\*\s*$/i;
+  var row = 0;
+  var rows = histDoc.querySelectorAll("#bugzilla-body tr");
+  for ( var i = 1 /* comment 0 can never be a duplicate marker */; i < comments.length; ++i) {
+    var textHolder = comments[i]
+        .querySelector(".bz_comment_text");
+    var match = reDuplicate.exec(trimContent(textHolder));
+    if (match) {
+      // construct the table row to be injected in histDoc
+      var bugID = match[1];
+      var email = comments[i]
+          .querySelector(".bz_comment_user .email").href
+          .substr(7);
+      var link = textHolder.querySelector("a");
+      var title = link.title;
+      var time = trimContent(comments[i]
+          .querySelector(".bz_comment_time"));
+      var what = 'Duplicate';
+      var removed = '';
+      var number = trimContent(
+          comments[i].querySelector(".bz_comment_number"))
+          .replace(/[^\d]+/g, '');
+      var class_ = '';
+      if (/bz_closed/i.test(link.className + " "
+          + link.parentNode.className)) {
+        class_ += 'bz_closed ';
+      }
+      if (link.parentNode.tagName.toLowerCase() == 'i') {
+        class_ += 'bztw_unconfirmed ';
+      }
+      var added = '<a href="show_bug.cgi?id=' + bugID
+          + '" title="' + htmlEncode(title) + '" name="c'
+          + number + '" class="' + class_ + '">' + bugID
+          + '</a>';
 
-            // inject the table row
-            var reachedEnd = false;
-            for (; row < rows.length; ++row) {
-                var cells = rows[row].querySelectorAll("td");
-                if (cells.length != 5)
-                    continue;
-                if (time > trimContent(cells[1])) {
-                    if (row < rows.length - 1) {
-                        continue;
-                    } else {
-                        reachedEnd = true;
-                    }
-                }
-                if (time == trimContent(cells[1])) {
-                    cells[0].rowSpan++;
-                    cells[1].rowSpan++;
-                    var rowContents = [what, removed, added];
-                    var tr =  histDoc.createElement("tr");
-                    rowContents.forEach(function (cellContents) {
-                        var td = histDoc.createElement("td");
-                        td.innerHTML = cellContents;
-                        tr.appendChild(td);
-                    });
-                    if (row != rows.length - 1) {
-                        rows[row].parentNode.insertBefore(tr, rows[row+1]);
-                    } else {
-                        rows[row].parentNode.appendChild(tr);
-                    }
-                } else {
-                    var rowContents = [email, time, what, removed, added];
-                    var tr =  histDoc.createElement("tr");
-                    rowContents.forEach(function (cellContents) {
-                        var td = histDoc.createElement("td");
-                        td.innerHTML = cellContents;
-                        tr.appendChild(td);
-                    });
-                    if (reachedEnd) {
-                        rows[row].parentNode.appendChild(tr);
-                    } else {
-                        rows[row].parentNode.insertBefore(tr, rows[row]);
-                    }
-                }
-                break;
-            }
-
-            // remove the comment from the main doc
-            comments[i].parentNode.removeChild(comments[i]);
+      // inject the table row
+      var reachedEnd = false;
+      for (; row < rows.length; ++row) {
+        var cells = rows[row].querySelectorAll("td");
+        if (cells.length != 5)
+          continue;
+        if (time > trimContent(cells[1])) {
+          if (row < rows.length - 1) {
+            continue;
+          }
+          else {
+            reachedEnd = true;
+          }
+        }
+        if (time == trimContent(cells[1])) {
+          cells[0].rowSpan++;
+          cells[1].rowSpan++;
+          var rowContents = [
+              what, removed, added
+          ];
+          var tr = histDoc.createElement("tr");
+          rowContents.forEach(function(cellContents) {
+            var td = histDoc.createElement("td");
+            td.innerHTML = cellContents;
+            tr.appendChild(td);
+          });
+          if (row != rows.length - 1) {
+            rows[row].parentNode.insertBefore(tr, rows[row + 1]);
+          }
+          else {
+            rows[row].parentNode.appendChild(tr);
+          }
         }
+        else {
+          var rowContents = [
+              email, time, what, removed, added
+          ];
+          var tr = histDoc.createElement("tr");
+          rowContents.forEach(function(cellContents) {
+            var td = histDoc.createElement("td");
+            td.innerHTML = cellContents;
+            tr.appendChild(td);
+          });
+          if (reachedEnd) {
+            rows[row].parentNode.appendChild(tr);
+          }
+          else {
+            rows[row].parentNode.insertBefore(tr, rows[row]);
+          }
+        }
+        break;
+      }
+
+      // remove the comment from the main doc
+      comments[i].parentNode.removeChild(comments[i]);
     }
+  }
 }
diff --git a/data/lib/queries.js b/data/lib/queries.js
index 4bf959a..52c6ce6 100644
--- a/data/lib/queries.js
+++ b/data/lib/queries.js
@@ -2,57 +2,61 @@
 // http://www.opensource.org/licenses/mit-license.php
 "use strict";
 
-function getSelection () {
-    var text = "";
-    var selectionObject = window.getSelection();
-    if (selectionObject.rangeCount > 0) {
-        text = selectionObject.getRangeAt(0).toString().trim();
-    }
-    return text;
+function getSelection() {
+  var text = "";
+  var selectionObject = window.getSelection();
+  if (selectionObject.rangeCount > 0) {
+    text = selectionObject.getRangeAt(0).toString().trim();
+  }
+  return text;
 }
 
 /**
  * Opens a new tab with a query for the given text in the selected component
- *
- * @param text to be searched for
- * @param component String with the component name (maybe latter regexp?)
- * @param product (optional) string with the product name, if undefined,
- *        search in all products
+ * 
+ * @param text
+ *          to be searched for
+ * @param component
+ *          String with the component name (maybe latter regexp?)
+ * @param product
+ *          (optional) string with the product name, if undefined, search in all
+ *          products
  * @return None
  * 
  */
 function queryInNewTab(text, component, product, equivComps) {
-    var urlStr = "https://" + window.location.hostname + "/buglist.cgi?query_format=advanced";
-    if (product) {
-        urlStr += "&product=" + product.trim();
-    }
-    if (component) {
-        if (equivComps) {
-            var equivCompsArr = equivComps.
-                filter(function (REstr) {
-                    return new RegExp(REstr).test(component);
-            }, this);
-            if (equivCompsArr.length > 0) {
-                component = equivCompsArr[0];
-            }
-        }
-        urlStr += "&component=" + component.trim();
+  var urlStr = "https://" + window.location.hostname
+      + "/buglist.cgi?query_format=advanced";
+  if (product) {
+    urlStr += "&product=" + product.trim();
+  }
+  if (component) {
+    if (equivComps) {
+      var equivCompsArr = equivComps.filter(function(REstr) {
+        return new RegExp(REstr).test(component);
+      }, this);
+      if (equivCompsArr.length > 0) {
+        component = equivCompsArr[0];
+      }
     }
+    urlStr += "&component=" + component.trim();
+  }
 
-    // using more complicated query tables here, because they can be more easily
-    // edited
-    // for further investigative searches
-    if (text) {
-        text = encodeURIComponent(text.trim());
-        var searchText = "&field0-0-0=longdesc&type0-0-0=substring&value0-0-0="
-                + text
-                + "&field0-0-1=attach_data.thedata&type0-0-1=substring&value0-0-1="
-                + text
-                + "&field0-0-2=status_whiteboard&type0-0-2=substring&value0-0-2="
-                + text;
-        urlStr += searchText;
-        self.postMessage(new Message("OpenURLinTab", urlStr)); // utils.js is always avaiulable
-    }
+  // using more complicated query tables here, because they can be more easily
+  // edited
+  // for further investigative searches
+  if (text) {
+    text = encodeURIComponent(text.trim());
+    var searchText = "&field0-0-0=longdesc&type0-0-0=substring&value0-0-0="
+        + text
+        + "&field0-0-1=attach_data.thedata&type0-0-1=substring&value0-0-1="
+        + text
+        + "&field0-0-2=status_whiteboard&type0-0-2=substring&value0-0-2="
+        + text;
+    urlStr += searchText;
+    self.postMessage(new Message("OpenURLinTab", urlStr)); // utils.js is
+                                                            // always avaiulable
+  }
 }
 
 /**
@@ -60,39 +64,42 @@ function queryInNewTab(text, component, product, equivComps) {
  * function this.queryInNewTab, and run it.
  */
 function queryForSelection() {
-    var text = getSelection();
-    if (!text) {
-        self.postMessage(new Message("GetClipboard", "queryLocal"));
-    } else {
-        if (equivalentComponents) {
-            queryInNewTab(text, getComponent(), getProduct(), equivalentComponents);
-        }
-        else {
-            queryInNewTab(text, getComponent(), getProduct());
-        }
+  var text = getSelection();
+  if (!text) {
+    self.postMessage(new Message("GetClipboard", "queryLocal"));
+  }
+  else {
+    if (equivalentComponents) {
+      queryInNewTab(text, getComponent(), getProduct(),
+          equivalentComponents);
     }
+    else {
+      queryInNewTab(text, getComponent(), getProduct());
+    }
+  }
 }
 
 /**
- *
+ * 
  */
 function queryUpstreamCallback(text, queryUpBug) {
   var searchData = filterByRegexp(queryUpBug, getComponent());
   if (!searchData) {
-    return ; // not sure why it should happen, but certainly better
+    return; // not sure why it should happen, but certainly better
   }
   var urlBase = searchData.url;
-  text = searchData.searchBy+":"+searchData.fillIn+" "+text.trim();
+  text = searchData.searchBy + ":" + searchData.fillIn + " "
+      + text.trim();
   if (searchData.fillIn == "$$$") {
     text = text.replace("$$$", getComponent());
   }
-  text = encodeURIComponent(text).replace("%20","+");
+  text = encodeURIComponent(text).replace("%20", "+");
   self.postMessage(new Message("OpenURLinTab", urlBase + text));
 }
 
 /**
  * Search simple query in the upstream bugzilla appropriate for the component
- *
+ * 
  * @return none
  */
 function queryUpstream(qUpBug) {
@@ -102,7 +109,8 @@ function queryUpstream(qUpBug) {
   }
   var text = getSelection();
   if (!text) {
-    self.postMessage(new Message("GetClipboard", "queryUpstream"));
+    self
+        .postMessage(new Message("GetClipboard", "queryUpstream"));
   }
   else {
     queryUpstreamCallback(text, qUpBug);
diff --git a/data/lib/rhbzpage.js b/data/lib/rhbzpage.js
index ec562df..14fe19d 100644
--- a/data/lib/rhbzpage.js
+++ b/data/lib/rhbzpage.js
@@ -16,13 +16,14 @@ var RawhideColor = new Color(0, 119, 0); // or "green", or RGB 0, 119, 0, or
 // 120, 0, 23
 var RHITColor = new Color(102, 0, 102); // RGB 102, 0, 102; HSL 300, 0, 20
 
-// [   126.386] (--) NOUVEAU(0): Chipset: "NVIDIA NVaf"
+// [ 126.386] (--) NOUVEAU(0): Chipset: "NVIDIA NVaf"
 var logAnalyzeLogic = {
   "AnalyzeInterestingLine": {
   /*
-  [   126.378] (--) PCI:*(0:4:0:0) 10de:08a0:106b:00c2 rev 162, Mem @ 0xd2000000/16777216, \
-  0xc0000000/268435456, 0xd0000000/33554432, I/O @ 0x00001000/128, BIOS @ 0x????????/131072
-  */
+   * [ 126.378] (--) PCI:*(0:4:0:0) 10de:08a0:106b:00c2 rev 162, Mem @
+   * 0xd2000000/16777216, \ 0xc0000000/268435456, 0xd0000000/33554432, I/O @
+   * 0x00001000/128, BIOS @ 0x????????/131072
+   */
     re: [
       "^(\\[[ .0-9]+\\])?\\s*\\(--\\) PCI:\\*\\([0-9:]+\\)\\s*" +
         "([0-9a-f:]+).*$",
@@ -32,14 +33,13 @@ var logAnalyzeLogic = {
     func: chipsetMagic
   },
   /*
-  [   126.385] (WW) Falling back to old probe method for vesa
-  [   126.385] (WW) Falling back to old probe method for fbdev
-  [   126.386] (--) NOUVEAU(0): Chipset: "NVIDIA NVaf"
-  Backtrace:
-
-  [  33.158] Kernel command line: ro root=LABEL=root rd_NO_LUKS rd_NO_LVM rd_NO_MD rd_NO_DM LANG=en_US.UTF-8 SYSFONT=latarcyrheb-sun16 KEYTABLE=us drm.debug=0x04
-
-  */
+   * [ 126.385] (WW) Falling back to old probe method for vesa [ 126.385] (WW)
+   * Falling back to old probe method for fbdev [ 126.386] (--) NOUVEAU(0):
+   * Chipset: "NVIDIA NVaf" Backtrace: [ 33.158] Kernel command line: ro
+   * root=LABEL=root rd_NO_LUKS rd_NO_LVM rd_NO_MD rd_NO_DM LANG=en_US.UTF-8
+   * SYSFONT=latarcyrheb-sun16 KEYTABLE=us drm.debug=0x04
+   * 
+   */
   "AnalyzeXorgLogBacktrace": {
     re:  "^\\s*(\\[[0-9 .]*\\])?\\s*(\\((EE|WW)\\)|.* [cC]hipset:.*)|\\s*(Backtrace|Kernel command line)",
     func: analyzeXorg
@@ -143,8 +143,9 @@ function RHcentralCommandDispatch(cmdLabel, cmdParams) {
 
 /**
  * Make it sailent that the some attachments with bad MIME type are present
- *
- * @param atts Array of attachments subarrays
+ * 
+ * @param atts
+ *          Array of attachments subarrays
  * @return none
  */
 function markBadAttachments(atts) {
@@ -176,7 +177,7 @@ function markBadAttachments(atts) {
 
 /**
  * Open a tab in the upstream bugzilla to create a new bug
- *
+ * 
  * @return none
  */
 function sendBugUpstream() {
@@ -195,7 +196,7 @@ function sendBugUpstream() {
 
 /**
  * Add a link opening selected lines of Xorg.0.log
- *
+ * 
  * @return none
  */
 function addCheckXorgLogLink(attList) {
@@ -212,13 +213,14 @@ function addCheckXorgLogLink(attList) {
 /**
  * Given line to be parsed, find out which chipset it is and fill in the
  * whiteboard
- *
- * @param PCIidArrObj object with two fields
- *    id Array manufacturer-ID and product-ID (PCI IDs)
- *    chipsetLine whole line containing PCI ID.
- * @param driverStr String with the driver name
+ * 
+ * @param PCIidArrObj
+ *          object with two fields id Array manufacturer-ID and product-ID (PCI
+ *          IDs) chipsetLine whole line containing PCI ID.
+ * @param driverStr
+ *          String with the driver name
  * @return None
- *
+ * 
  */
 function fillInWhiteBoard(cardName) {
   console.myDebug("fillInWhiteBoard: cardName = " + cardName);
@@ -232,8 +234,9 @@ function fillInWhiteBoard(cardName) {
  * Get attached Xorg.0.log, parse it and find the value of chip. Does not fill
  * the whiteboard itself, just adds button to do so,paramList so that slow
  * XMLHttpRequest is done in advance.
- *
- * @param log array of XorgLogAttList
+ * 
+ * @param log
+ *          array of XorgLogAttList
  * @return None
  */
 function fillInChipMagic(XlogID) {
@@ -242,8 +245,9 @@ function fillInChipMagic(XlogID) {
 
 /**
  * Creates a button to change summary by adding a graphic chip label
- *
- * @param Array with matching results of re.exec()
+ * 
+ * @param Array
+ *          with matching results of re.exec()
  */
 function chipsetMagic (interestingLineArr) {
   // parse Xorg.0.log
@@ -285,7 +289,7 @@ function chipsetMagic (interestingLineArr) {
       if (cardStr) {
         createNewButton("short_desc_nonedit_display", false, {
           "name": "Fill In",
-          "chipMagic": cardStr,
+          "chipMagic": cardStr
         });
       }
     }
@@ -356,8 +360,9 @@ function findInterestingLine(wholeLog, backMsg) {
 
 /**
  * Add information about the upstream bug upstream, and closing it.
- *
- * @param evt Event which called this handler
+ * 
+ * @param evt
+ *          Event which called this handler
  * @return none
  */
 function addClosingUpstream() {
@@ -407,7 +412,7 @@ function addClosingUpstream() {
 }
 
 /**
- *
+ * 
  */
 function parseBacktrace (ret) {
   var signalHandler = new RegExp("^\\s*#[0-9]*\\s*<signal handler called>");
diff --git a/data/lib/skip-bug.js b/data/lib/skip-bug.js
index 3639448..c909a3e 100644
--- a/data/lib/skip-bug.js
+++ b/data/lib/skip-bug.js
@@ -6,12 +6,12 @@ function reloadPage() {
   var stemURL = 'https://HOSTNAME/show_bug.cgi?id=';
   var titleElems = document.getElementsByTagName('title');
   if (titleElems) {
-    var REArr = new RegExp('[bB]ug\\s+([0-9]+)').
-      exec(titleElems[0].textContent);
+    var REArr = new RegExp('[bB]ug\\s+([0-9]+)')
+        .exec(titleElems[0].textContent);
     var hostname = document.location.hostname;
     if (REArr) {
-      document.location = stemURL.
-        replace('HOSTNAME',hostname) + REArr[1];
+      document.location = stemURL.replace('HOSTNAME', hostname)
+          + REArr[1];
     }
   }
 }
diff --git a/data/lib/urltest.js b/data/lib/urltest.js
index a0dbabf..609e77b 100644
--- a/data/lib/urltest.js
+++ b/data/lib/urltest.js
@@ -1,4 +1,5 @@
 function onBugzillaPage(url) {
-  return /https:\/\/bugzilla(-[a-zA-Z]+)*\.mozilla\.org/.test(url)
+  return /https:\/\/bugzilla(-[a-zA-Z]+)*\.mozilla\.org/
+      .test(url)
       || /https:\/\/landfill.*\.bugzilla\.org/.test(url);
-}
\ No newline at end of file
+}
diff --git a/data/lib/util.js b/data/lib/util.js
index 0e3dac4..77e7bfa 100644
--- a/data/lib/util.js
+++ b/data/lib/util.js
@@ -7,16 +7,17 @@
 
 /**
  * parse URL to get its parts.
- *
+ * 
  * @param url
  * @return object with all parsed parts of URL as properties
- *
- * Originally from http://james.padolsey.com/javascript/parsing-urls-with-the-dom/
- * Copyright February 19th, 2009, James Padolsey, <license undeclared>
- *
- * This function creates a new anchor element and uses location
- * properties (inherent) to get the desired URL data. Some String
- * operations are used (to normalize results across browsers).
+ * 
+ * Originally from
+ * http://james.padolsey.com/javascript/parsing-urls-with-the-dom/ Copyright
+ * February 19th, 2009, James Padolsey, <license undeclared>
+ * 
+ * This function creates a new anchor element and uses location properties
+ * (inherent) to get the desired URL data. Some String operations are used (to
+ * normalize results across browsers).
  */
 function parseURL(url) {
   var a =  document.createElement('a');
@@ -47,16 +48,19 @@ function parseURL(url) {
 }
 
 /**
- * parse XML object out of string working around various bugs in Gecko implementation
- * see https://developer.mozilla.org/en/E4X for more information
- *
- * @param inStr String with unparsed XML string
+ * parse XML object out of string working around various bugs in Gecko
+ * implementation see https://developer.mozilla.org/en/E4X for more information
+ * 
+ * @param inStr
+ *          String with unparsed XML string
  * @return XML object
  */
 function parseXMLfromString (inStuff) {
-  // if (typeof inStuff !== 'string') In future we should recognize this.response
+  // if (typeof inStuff !== 'string') In future we should recognize
+  // this.response
   // and get just .text property out of it. TODO
-  var respStr = inStuff.replace(/^<\?xml\s+version\s*=\s*(["'])[^\1]+\1[^?]*\?>/, ""); // bug 336551
+  var respStr = inStuff.replace(/^<\?xml\s+version\s*=\s*(["'])[^\1]+\1[^?]*\?>/, ""); // bug
+                                                                                        // 336551
   return new XML(respStr);
 }
 
@@ -70,7 +74,9 @@ function getBugNo() {
 
 /**
  * Get a bug no from URL ... fails with aliases
- * @param url String with URL to be analyzed
+ * 
+ * @param url
+ *          String with URL to be analyzed
  * @return String with the bug ID
  */
 function getBugNoFromURL(url) {
@@ -82,8 +88,9 @@ function getBugNoFromURL(url) {
 
 /**
  * Send mouse click to the specified element
- *
- * @param String ID of the element to send mouseclick to
+ * 
+ * @param String
+ *          ID of the element to send mouseclick to
  * @return None
  */
 function clickMouse (targetID) {
@@ -94,14 +101,22 @@ function clickMouse (targetID) {
 }
 
 /**
- * Create a A element leadink nowhere, but with listener running a callback on the click
- *
- * @param id String with a id to be added to the element
- * @param text String with a string to be added as a textContent of the element
- * @param parent Node which is a parent of the object
- * @param callback Function to be called after clicking on the link
- * @param params Array with parameters of the callback
- * @param Boolean before if there should be a <br> element before.
+ * Create a A element leadink nowhere, but with listener running a callback on
+ * the click
+ * 
+ * @param id
+ *          String with a id to be added to the element
+ * @param text
+ *          String with a string to be added as a textContent of the element
+ * @param parent
+ *          Node which is a parent of the object
+ * @param callback
+ *          Function to be called after clicking on the link
+ * @param params
+ *          Array with parameters of the callback
+ * @param Boolean
+ *          before if there should be a <br>
+ *          element before.
  * @return none
  */
 function createDeadLink (id, text, parent, callback, params, before, covered, accesskey) {
@@ -152,13 +167,13 @@ function createDeadLink (id, text, parent, callback, params, before, covered, ac
 }
 
 /*
- * From <a> element diggs out just plain email address
- * Note that bugzilla.gnome.org doesn't have mailto: url but
+ * From <a> element diggs out just plain email address Note that
+ * bugzilla.gnome.org doesn't have mailto: url but
  * https://bugzilla.gnome.org/page.cgi?id=describeuser.html&login=mcepl%40redhat.com
- *
- * @param aElement Element with href attribute or something else
- * @return String with the address or null
- *
+ * 
+ * @param aElement Element with href attribute or something else @return String
+ * with the address or null
+ * 
  */
 function parseMailto(aElement) {
   var emailStr = "", hrefStr = "";
@@ -178,7 +193,7 @@ function parseMailto(aElement) {
 
 /**
  * format date to be in ISO format (just day part)
- *
+ * 
  * @param date
  * @return string with the formatted date
  */
@@ -194,9 +209,11 @@ function getISODate(dateStr) {
 /**
  * Check whether an item is member of the list. Idea is just to make long if
  * commands slightly more readable.
- *
- * @param mbr string to be searched in the list
- * @param list list
+ * 
+ * @param mbr
+ *          string to be searched in the list
+ * @param list
+ *          list
  * @return position of the string in the list, or -1 if none found.
  */
 function isInList(mbr, list) {
@@ -208,12 +225,12 @@ function isInList(mbr, list) {
 
 /**
  * Make sure value returned is Array
- *
+ * 
  * @param Array/String
  * @return Array
- *
- * If something else than Array or String is passed to the function
- * the result will be untouched actual argument of the call.
+ * 
+ * If something else than Array or String is passed to the function the result
+ * will be untouched actual argument of the call.
  */
 function valToArray(val) {
   var isArr = val &&
@@ -224,9 +241,11 @@ function valToArray(val) {
 
 /**
  * Merges two comma separated string as a list and returns new string
- *
- * @param str String with old values
- * @param value String/Array with other values
+ * 
+ * @param str
+ *          String with old values
+ * @param value
+ *          String/Array with other values
  * @return String with merged lists
  */
 function addCSVValue(str, value) {
@@ -246,9 +265,11 @@ function addCSVValue(str, value) {
 
 /**
  * Treats comma separated string as a list and removes one item from it
- *
- * @param str String treated as a list
- * @param value String with the value to be removed from str
+ * 
+ * @param str
+ *          String treated as a list
+ * @param value
+ *          String with the value to be removed from str
  * @return String with the resulting list comma separated
  */
 function removeCSVValue(str, value) {
@@ -264,9 +285,11 @@ function removeCSVValue(str, value) {
 /**
  * select element of the array where regexp in the first element matches second
  * parameter of this function
- *
- * @param list Array with regexps and return values
- * @param chosingMark String by which the element of array is to be matched
+ * 
+ * @param list
+ *          Array with regexps and return values
+ * @param chosingMark
+ *          String by which the element of array is to be matched
  * @return Object chosen element
  */
 function filterByRegexp(list, chosingMark) {
@@ -286,14 +309,16 @@ function filterByRegexp(list, chosingMark) {
 
 /**
  * remove elements from the page based on their IDs
- *
- * @param doc Document object
- * @param target String/Array with ID(s)
- * @param remove Boolean indicating whether the node should be
- * actually removed or just hidden.
- * @return none
- * TODO remove parameter could be replaced by function which would
- * do actual activity.
+ * 
+ * @param doc
+ *          Document object
+ * @param target
+ *          String/Array with ID(s)
+ * @param remove
+ *          Boolean indicating whether the node should be actually removed or
+ *          just hidden.
+ * @return none TODO remove parameter could be replaced by function which would
+ *         do actual activity.
  */
 function killNodes(doc, target, remove) {
   target = target.trim();
@@ -311,8 +336,9 @@ function killNodes(doc, target, remove) {
 
 /**
  * Remove duplicate elements from array
- *
- * @param arr Array which needs to be cleaned up
+ * 
+ * @param arr
+ *          Array which needs to be cleaned up
  * @return cleaned up array
  */
 function removeDuplicates (arr) {
@@ -328,11 +354,8 @@ function removeDuplicates (arr) {
 
 // ============================================
 /**
- * object to pack messaging. Use as in
-    self.postMessage(new Message("GetPassword", {
-      login: login,
-      hostname: location.hostname
-    }));
+ * object to pack messaging. Use as in self.postMessage(new
+ * Message("GetPassword", { login: login, hostname: location.hostname }));
  */
 function Message(cmd, data) {
 	this.cmd = cmd;
diff --git a/data/lib/viewSource.js b/data/lib/viewSource.js
index 25cba58..fd47cec 100644
--- a/data/lib/viewSource.js
+++ b/data/lib/viewSource.js
@@ -51,7 +51,7 @@ function viewAttachmentSource(doc) {
   if (!table)
     return;
   var rows = table.querySelectorAll("tr");
-  for (var i = 0; i < rows.length; ++i) {
+  for ( var i = 0; i < rows.length; ++i) {
     var items = rows[i].querySelectorAll("td");
     if (items.length != 3)
       continue;
@@ -68,32 +68,36 @@ function viewAttachmentSource(doc) {
       // Match mime type followed by ";" (charset) or ")" (no charset)
       typeName = span.textContent.match(reAttachmentType)[1];
       typeName = typeName.split(";")[0]; // ignore charset following type
-    } catch (e) {}
-    if (typeName == "application/java-archive" ||
-        typeName == "application/x-jar") {
+    }
+    catch (e) {
+    }
+    if (typeName == "application/java-archive"
+        || typeName == "application/x-jar") {
       // Due to the fix for bug 369814, only zip files with this special
       // mime type can be used with the jar: protocol.
       // http://hg.mozilla.org/mozilla-central/rev/be54f6bb9e1e
-      addLink(items[2], "JAR Contents", "jar:" + attachHref + "!/");
-    // https://bugzilla.mozilla.org/show_bug.cgi?id=369814#c5 has more possible mime types for zips?
-    } else if (typeName == "application/zip" ||
-               typeName == "application/x-zip-compressed" ||
-               typeName == "application/x-xpinstall") {
-      addLink(items[2], "Static ZIP Contents", "jar:" + attachHref + "!/");
-    } else if (typeName != "text/plain" &&
-               typeName != "patch" &&
-               // Other types that Gecko displays like text/plain
-               // http://mxr.mozilla.org/mozilla-central/source/parser/htmlparser/public/nsIParser.h
-               typeName != "text/css" &&
-               typeName != "text/javascript" &&
-               typeName != "text/ecmascript" &&
-               typeName != "application/javascript" &&
-               typeName != "application/ecmascript" &&
-               typeName != "application/x-javascript" &&
-               // Binary image types for which the "source" is not useful
-               typeName != "image/gif" &&
-               typeName != "image/png" &&
-               typeName != "image/jpeg") {
+      addLink(items[2], "JAR Contents", "jar:" + attachHref
+          + "!/");
+      // https://bugzilla.mozilla.org/show_bug.cgi?id=369814#c5 has more
+      // possible mime types for zips?
+    }
+    else if (typeName == "application/zip"
+        || typeName == "application/x-zip-compressed"
+        || typeName == "application/x-xpinstall") {
+      addLink(items[2], "Static ZIP Contents", "jar:"
+          + attachHref + "!/");
+    }
+    else if (typeName != "text/plain" && typeName != "patch" &&
+    // Other types that Gecko displays like text/plain
+    // http://mxr.mozilla.org/mozilla-central/source/parser/htmlparser/public/nsIParser.h
+    typeName != "text/css" && typeName != "text/javascript"
+        && typeName != "text/ecmascript"
+        && typeName != "application/javascript"
+        && typeName != "application/ecmascript"
+        && typeName != "application/x-javascript" &&
+        // Binary image types for which the "source" is not useful
+        typeName != "image/gif" && typeName != "image/png"
+        && typeName != "image/jpeg") {
       addLink(items[2], "Source", "view-source:" + attachHref);
     }
   }
diff --git a/data/lib/xorgBugCategories.js b/data/lib/xorgBugCategories.js
index c2c9c9b..3357ed7 100644
--- a/data/lib/xorgBugCategories.js
+++ b/data/lib/xorgBugCategories.js
@@ -4,25 +4,26 @@
 
 /**
  * Returns true if the bug is in a good shape
- *
- * @return Boolean if the bug is either not in the category
- * where we care about it (i.e., we don't have set up categories for
- * this component) or if it is in the concerned categories, then it has
- * a category recorded in the whiteboard input box.
- *
+ * 
+ * @return Boolean if the bug is either not in the category where we care about
+ *         it (i.e., we don't have set up categories for this component) or if
+ *         it is in the concerned categories, then it has a category recorded in
+ *         the whiteboard input box.
+ * 
  */
 function hasXorgBugsCategory() {
   var catRE = /\s*\[cat:.*?\]\s*/; // RE for testing whether
   // there is already category tag in the Whiteboard
 
-  var isXOrgBug = filterByRegexp(constantData.
-      xorgBugsCategories, getComponent());
-  var whiteboardContent = document.
-    getElementById("status_whiteboard").value;
+  var isXOrgBug = filterByRegexp(
+      constantData.xorgBugsCategories, getComponent());
+  var whiteboardContent = document
+      .getElementById("status_whiteboard").value;
 
   if (isXOrgBug) { // is it XOR?
     return catRE.test(whiteboardContent);
-  } else {
+  }
+  else {
     return true;
   }
 }
@@ -47,23 +48,24 @@ function makeBugCategoriesList(catList) {
 
   // Fill-in <select> with <options>s for each category one
   if (catList) {
-    catList.forEach(function (cat) {
+    catList.forEach(function(cat) {
       optionElement = document.createElement("option");
       optionElement.value = cat;
-      optionElement.setAttribute("id", "catId_" +
-        cat.replace(" ","").toLowerCase());
+      optionElement.setAttribute("id", "catId_"
+          + cat.replace(" ", "").toLowerCase());
       optionElement.appendChild(document.createTextNode(cat));
       categoryList.appendChild(optionElement);
     });
   }
 
-  categoryList.addEventListener("change", function (evt) {
+  categoryList.addEventListener("change", function(evt) {
     var selectedCategory = "[cat:" + this.value + "]";
-    var whiteboardElement = document.getElementById("status_whiteboard");
+    var whiteboardElement = document
+        .getElementById("status_whiteboard");
 
     if (hasXorgBugsCategory()) {
-      whiteboardElement.value = whiteboardElement.
-        value.replace(catRE, "");
+      whiteboardElement.value = whiteboardElement.value.replace(
+          catRE, "");
     }
     addStuffToTextBox("status_whiteboard", selectedCategory);
   }, false);
diff --git a/lib/cookiemanager.js b/lib/cookiemanager.js
index 087d6ab..b777ab9 100644
--- a/lib/cookiemanager.js
+++ b/lib/cookiemanager.js
@@ -46,10 +46,10 @@ exports.getHostCookies = function getHostCookies(host) {
     if (cookie.rawHost == host) {
       result.push({
         name: cookie.name,
-        value: cookie.value,
+        value: cookie.value
       });
     }
   }
   return result;
-}
+};
 
diff --git a/lib/libbugzilla.js b/lib/libbugzilla.js
index 7e2398b..c85d331 100644
--- a/lib/libbugzilla.js
+++ b/lib/libbugzilla.js
@@ -38,23 +38,26 @@ function log(msg) {
 }
 
 /**
- * parse XML object out of string working around various bugs in Gecko implementation
- * see https://developer.mozilla.org/en/E4X for more information
- *
- * @param inStr String with unparsed XML string
+ * parse XML object out of string working around various bugs in Gecko
+ * implementation see https://developer.mozilla.org/en/E4X for more information
+ * 
+ * @param inStr
+ *          String with unparsed XML string
  * @return XML object
  */
 function parseXMLfromString (inStuff) {
-  // if (typeof inStuff !== 'string') In future we should recognize this.response
+  // if (typeof inStuff !== 'string') In future we should recognize
+  // this.response
   // and get just .text property out of it. TODO
-  var respStr = inStuff.replace(/^<\?xml\s+version\s*=\s*(["'])[^\1]+\1[^?]*\?>/, ""); // bug 336551
+  var respStr = inStuff.replace(/^<\?xml\s+version\s*=\s*(["'])[^\1]+\1[^?]*\?>/, ""); // bug
+                                                                                        // 336551
   return new XML(respStr);
 }
 
 /**
- * In case URL contains alias, not the real bug number, get the real bug no
- * from the XML representation. Sets correct value to this.bugNo.
- *
+ * In case URL contains alias, not the real bug number, get the real bug no from
+ * the XML representation. Sets correct value to this.bugNo.
+ * 
  * This is a slow variant for bugs other than actual window
  */
 function getRealBugNoSlow(bugNo, location, callback) {
@@ -141,14 +144,11 @@ exports.changeJSONURL = function changeJSONURL() {
 };
 
 /**
- *
-   libbz.getInstalledPackages(msg.data, function (pkgsMsg) {
-        worker.postMessage(pkgsMsg);
-
-   locationLoginObj: {
-    location: window.location.href,
-    login: getLogin()
-  }
+ * 
+ * libbz.getInstalledPackages(msg.data, function (pkgsMsg) {
+ * worker.postMessage(pkgsMsg);
+ * 
+ * locationLoginObj: { location: window.location.href, login: getLogin() }
  */
 exports.getInstalledPackages = function getInstalledPackages(locationLoginObj, callback) {
   var installedPackages = {};
@@ -283,14 +283,16 @@ exports.createUpstreamBug = function createUpstreamBug(urlStr, subject, comment)
   });
 };
 
-// Make a XML-RPC call ... most of the business logic should stay in the content script
+// Make a XML-RPC call ... most of the business logic should stay in the content
+// script
 exports.makeXMLRPCCall = function makeXMLRPCCall(url, login, method, params, callback) {
   var urlObj = urlMod.URL(url);
   getPassword(login,
     urlObj.schema + "://" + urlObj.host,
     function (passwObj) {
       if (!passwObj.password) {
-        // TODO this should happen, only when user presses Escape in password prompt
+        // TODO this should happen, only when user presses Escape in password
+        // prompt
         return null;
       }
 
diff --git a/lib/logger.js b/lib/logger.js
index ce96f69..ed9ac47 100644
--- a/lib/logger.js
+++ b/lib/logger.js
@@ -24,8 +24,10 @@ exports.initialize = function initialize(aMap) {
 };
 
 exports.addLogRecord = function addLogRecord(rec) {
-  if (myStorage.storage.logs[rec.key] && myStorage.storage.logs[rec.key].comment) {
-    myStorage.storage.logs[rec.key].comment += "<br/>\n" + rec.comment;
+  if (myStorage.storage.logs[rec.key]
+      && myStorage.storage.logs[rec.key].comment) {
+    myStorage.storage.logs[rec.key].comment += "<br/>\n"
+        + rec.comment;
   }
   else {
     myStorage.storage.logs[rec.key] = rec;
@@ -54,7 +56,7 @@ exports.importTimeSheet = function importTimeSheet() {
   if (fileMod.exists(filename)) {
     var otherTS = JSON.parse(fileMod.read(filename));
     if (otherTS.logs) {
-      for (var rec in otherTS.logs) {
+      for ( var rec in otherTS.logs) {
         myStorage.storage.logs[rec] = otherTS.logs[rec];
       }
     }
@@ -74,27 +76,30 @@ function getBugzillaAbbr(url) {
 }
 
 exports.generateTimeSheet = function generateTimeSheet() {
-  var docHTML = timeSheetRecordsPrinter(myStorage.storage.logs, new Date());
-  libbz.openURLInNewTab("data:text/html;charset=utf-8," + docHTML);
-}; 
+  var docHTML = timeSheetRecordsPrinter(myStorage.storage.logs,
+      new Date());
+  libbz.openURLInNewTab("data:text/html;charset=utf-8,"
+      + docHTML);
+};
 
-var timeSheetRecordsPrinter = exports.timeSheetRecordsPrinter =
-    function timeSheetRecordsPrinter(records, date) {
-  var commentBugRE = new RegExp("[bB]ug\\s+([0-9]+)","g");
+var timeSheetRecordsPrinter = exports.timeSheetRecordsPrinter = function timeSheetRecordsPrinter(
+    records, date) {
+  var commentBugRE = new RegExp("[bB]ug\\s+([0-9]+)", "g");
   // sort the records into temporary array
   var tmpArr = [];
-  var dateStr = date.getFullYear() + "-" +
-    xrpc.leadingZero(date.getMonth()+1) + "-" +
-    xrpc.leadingZero(date.getDate());
-  var outStr = '<!DOCTYPE html>' +
-    "<html><head>\n"+
-    "<meta charset='utf-8'/>\n"+
-    "<title>TimeSheet-"+ dateStr + "</title>\n</head>\n<body>\n" +
-    "<h1>TimeSheet</h1>\n";
+  var dateStr = date.getFullYear() + "-"
+      + xrpc.leadingZero(date.getMonth() + 1) + "-"
+      + xrpc.leadingZero(date.getDate());
+  var outStr = '<!DOCTYPE html>' + "<html><head>\n"
+      + "<meta charset='utf-8'/>\n" + "<title>TimeSheet-"
+      + dateStr + "</title>\n</head>\n<body>\n"
+      + "<h1>TimeSheet</h1>\n";
 
-  for (var i in records) {
+  for ( var i in records) {
     if (records.hasOwnProperty(i)) {
-      tmpArr.push( [ i, records[i] ]);
+      tmpArr.push([
+          i, records[i]
+      ]);
     }
   }
   tmpArr.sort(function(a, b) {
@@ -112,27 +117,24 @@ var timeSheetRecordsPrinter = exports.timeSheetRecordsPrinter =
   var currentDay = "";
   // now print the array
   tmpArr.forEach(function(rec) {
-      var x = rec[1];
-      var dayStr = utilMod.getISODate(x.date);
-      var host = urlMod.URL(x.url).host;
-      var BZName = getBugzillaAbbr(x.url);
-      var bugNo = utilMod.getBugNo(x.url);
-      if (dayStr != currentDay) {
-        currentDay = dayStr;
-        outStr += "<hr/><p><strong>" + currentDay
-            + "</strong></p>\n";
-      }
-      // replace "bug ####" with a hyperlink to the current bugzilla
-      var comment = x.comment.replace(commentBugRE,
-        "<a href='http://"+host+"/show_bug.cgi?id=$1'>$&</a>");
-      outStr += "<p><em><a href='"
-          + x.url
-          + "'>Bug "
-          + BZName + "/" + bugNo + ": "
-          + x.title
-          + "</a>"
-          + " </em>\n<br/>" + comment + "</p>\n";
+    var x = rec[1];
+    var dayStr = utilMod.getISODate(x.date);
+    var host = urlMod.URL(x.url).host;
+    var BZName = getBugzillaAbbr(x.url);
+    var bugNo = utilMod.getBugNo(x.url);
+    if (dayStr != currentDay) {
+      currentDay = dayStr;
+      outStr += "<hr/><p><strong>" + currentDay
+          + "</strong></p>\n";
+    }
+    // replace "bug ####" with a hyperlink to the current bugzilla
+    var comment = x.comment.replace(commentBugRE,
+        "<a href='http://" + host
+            + "/show_bug.cgi?id=$1'>$&</a>");
+    outStr += "<p><em><a href='" + x.url + "'>Bug " + BZName
+        + "/" + bugNo + ": " + x.title + "</a>"
+        + " </em>\n<br/>" + comment + "</p>\n";
   });
   outStr += "</body></html>";
   return outStr;
-}
+};
diff --git a/lib/main.js b/lib/main.js
index 849da68..f3e9458 100644
--- a/lib/main.js
+++ b/lib/main.js
@@ -23,13 +23,13 @@ function isOurPage(window, matchingURLs) {
   var url = window.location.href;
 
   // like ["regexp-url1", "regexp-url2"]
-  return matchingURLs.some(function (element,i,a) {
+  return matchingURLs.some(function(element, i, a) {
     return new RegExp(element).test(url);
   });
 }
 
 /**
- *
+ * 
  */
 function skipThisPage(doc) {
   var stemURL = "https://HOSTNAME/show_bug.cgi?id=";
@@ -39,20 +39,21 @@ function skipThisPage(doc) {
   var hostname = doc.location.hostname;
   if (REArr) {
     var bugNo = REArr[1];
-    doc.location = stemURL.replace("HOSTNAME",hostname) + bugNo;
+    doc.location = stemURL.replace("HOSTNAME", hostname) + bugNo;
   }
 }
 
-var messageHandler = exports.messageHandler = function messageHandler(worker, msg) {
+var messageHandler = exports.messageHandler = function messageHandler(
+    worker, msg) {
   switch (msg.cmd) {
   case "LogMessage":
     console.log(msg.data);
     break;
   case "ExecCmd":
-    libbz.executeCommand(msg.data); 
+    libbz.executeCommand(msg.data);
     break;
   case "AddLogRecord":
-    logger.addLogRecord(msg.data); 
+    logger.addLogRecord(msg.data);
     break;
   case "GenerateTS":
     logger.generateTimeSheet();
@@ -65,12 +66,12 @@ var messageHandler = exports.messageHandler = function messageHandler(worker, ms
     break;
   case "GetInstalledPackages":
     // send message with packages back
-    libbz.getInstalledPackages(msg.data, function (pkgsMsg) {
+    libbz.getInstalledPackages(msg.data, function(pkgsMsg) {
       worker.postMessage(pkgsMsg);
     });
     break;
   case "GetClipboard":
-    libbz.getClipboard(function (clipboard) {
+    libbz.getClipboard(function(clipboard) {
       worker.postMessage(new Message(msg.data, clipboard));
     });
     break;
@@ -91,18 +92,23 @@ var messageHandler = exports.messageHandler = function messageHandler(worker, ms
     break;
   case "MakeXMLRPCall":
     // url, login, method, params, callback
-    libbz.makeXMLRPCCall(msg.data.url, msg.data.login, msg.data.method,
-      msg.data.params, function(ret) {
-        worker.postMessage(new Message(msg.data.callRPC, ret));
-    });
+    libbz
+        .makeXMLRPCCall(msg.data.url, msg.data.login,
+            msg.data.method, msg.data.params, function(ret) {
+              worker.postMessage(new Message(msg.data.callRPC,
+                  ret));
+            });
     break;
   case "GetURL":
-    libbz.getURL(msg.data.url, function(stuff) {
-      worker.postMessage(new Message(msg.data.backMessage, stuff));
-    });
+    libbz.getURL(msg.data.url,
+        function(stuff) {
+          worker.postMessage(new Message(msg.data.backMessage,
+              stuff));
+        });
     break;
   case "OpenBugUpstream":
-    libbz.createUpstreamBug(msg.data.url, msg.data.subject, msg.data.comment);
+    libbz.createUpstreamBug(msg.data.url, msg.data.subject,
+        msg.data.comment);
     break;
   case "testReady":
     // we ignore it here, interesting only in unit test
@@ -113,36 +119,36 @@ var messageHandler = exports.messageHandler = function messageHandler(worker, ms
 };
 
 var contentScriptLibraries = [
-  self.data.url('lib/urltest.js'),
-  self.data.url("lib/util.js"),
-  self.data.url("lib/jumpNextBug.js"),
-  self.data.url("lib/queries.js"),
-  self.data.url("lib/preprocessDuplicates.js"),
-  self.data.url("lib/viewSource.js"),
-  self.data.url("lib/color.js"),
-  self.data.url("lib/addNewLinks.js"),
-  self.data.url("lib/bugzillaDOMFunctions.js"),
-  self.data.url("lib/xorgBugCategories.js"),
-  self.data.url("lib/otherButtons.js"),
-  self.data.url("lib/makeBacktraceAttachment.js"),
-  self.data.url("lib/fixingAttMIME.js"),
-  self.data.url("lib/logging-front.js"),
-  self.data.url('lib/bug-page-mod.js'),
-  self.data.url("lib/rhbzpage.js"),
-  self.data.url("lib/bzpage.js")
+    self.data.url('lib/urltest.js'),
+    self.data.url("lib/util.js"),
+    self.data.url("lib/jumpNextBug.js"),
+    self.data.url("lib/queries.js"),
+    self.data.url("lib/preprocessDuplicates.js"),
+    self.data.url("lib/viewSource.js"),
+    self.data.url("lib/color.js"),
+    self.data.url("lib/addNewLinks.js"),
+    self.data.url("lib/bugzillaDOMFunctions.js"),
+    self.data.url("lib/xorgBugCategories.js"),
+    self.data.url("lib/otherButtons.js"),
+    self.data.url("lib/makeBacktraceAttachment.js"),
+    self.data.url("lib/fixingAttMIME.js"),
+    self.data.url("lib/logging-front.js"),
+    self.data.url('lib/bug-page-mod.js'),
+    self.data.url("lib/rhbzpage.js"),
+    self.data.url("lib/bzpage.js")
 ];
 
-libbz.initialize(libbz.config, function () {
+libbz.initialize(libbz.config, function() {
   pageMod.PageMod({
-    include: [
-      "https://bugzilla.redhat.com/show_bug.cgi?id=*",
-      "https://bugzilla.mozilla.org/show_bug.cgi?id=*",
-      "https://bugzilla.gnome.org/show_bug.cgi?id=*"
+    include : [
+        "https://bugzilla.redhat.com/show_bug.cgi?id=*",
+        "https://bugzilla.mozilla.org/show_bug.cgi?id=*",
+        "https://bugzilla.gnome.org/show_bug.cgi?id=*"
     ],
-    contentScriptWhen: 'ready',
-    contentScriptFile: contentScriptLibraries,
-    onAttach: function onAttach(worker, msg) {
-      worker.on('message', function (msg) {
+    contentScriptWhen : 'ready',
+    contentScriptFile : contentScriptLibraries,
+    onAttach : function onAttach(worker, msg) {
+      worker.on('message', function(msg) {
         messageHandler(worker, msg);
       });
     }
@@ -150,34 +156,37 @@ libbz.initialize(libbz.config, function () {
 });
 
 pageMod.PageMod({
-  include: [
-    "https://bugzilla.redhat.com/process_bug.cgi",
-    "https://bugzilla.redhat.com/post_bug.cgi",
-    "https://bugzilla.redhat.com/attachment.cgi",
-    "https://bugzilla.mozilla.org/process_bug.cgi",
-    "https://bugzilla.mozilla.org/post_bug.cgi",
-    "https://bugzilla.mozilla.org/attachment.cgi",
-    "https://bugzilla.gnome.org/process_bug.cgi",
-    "https://bugzilla.gnome.org/post_bug.cgi",
-    "https://bugzilla.gnome.org/attachment.cgi"
+  include : [
+      "https://bugzilla.redhat.com/process_bug.cgi",
+      "https://bugzilla.redhat.com/post_bug.cgi",
+      "https://bugzilla.redhat.com/attachment.cgi",
+      "https://bugzilla.mozilla.org/process_bug.cgi",
+      "https://bugzilla.mozilla.org/post_bug.cgi",
+      "https://bugzilla.mozilla.org/attachment.cgi",
+      "https://bugzilla.gnome.org/process_bug.cgi",
+      "https://bugzilla.gnome.org/post_bug.cgi",
+      "https://bugzilla.gnome.org/attachment.cgi"
   ],
-  contentScriptWhen: 'ready',
-  contentScriptFile: self.data.url("lib/skip-bug.js")
+  contentScriptWhen : 'ready',
+  contentScriptFile : self.data.url("lib/skip-bug.js")
 });
 
-
 // Allow toggling of CC event displays using a context menu entry
 contextMenu.Item({
-  label: "Toggle CC History",
-  contentScriptFile: [self.data.url('lib/urltest.js'),
-                      self.data.url('lib/cc-context.js')]
+  label : "Toggle CC History",
+  contentScriptFile : [
+      self.data.url('lib/urltest.js'),
+      self.data.url('lib/cc-context.js')
+  ]
 });
 
 contextMenu.Item({
-  label: "Copy Check-in Comment",
-  contentScriptFile: [self.data.url('lib/urltest.js'),
-                      self.data.url('lib/checkin-context.js')],
-  onMessage: function (comment) {
-      require("clipboard").set(comment);
+  label : "Copy Check-in Comment",
+  contentScriptFile : [
+      self.data.url('lib/urltest.js'),
+      self.data.url('lib/checkin-context.js')
+  ],
+  onMessage : function(comment) {
+    require("clipboard").set(comment);
   }
 });
diff --git a/lib/prompts.js b/lib/prompts.js
index 7cfd3d1..3f34e0d 100644
--- a/lib/prompts.js
+++ b/lib/prompts.js
@@ -11,23 +11,23 @@ var promptTitle = "Bugzilla Triage Script";
 
 /**
  * shows the text in a simple window
- *
+ * 
  * @return none
  */
 exports.alert = function alert(msg) {
   var prompts = Cc["@mozilla.org/embedcomp/prompt-service;1"]
-    .getService(Ci.nsIPromptService);
+      .getService(Ci.nsIPromptService);
   prompts.alert(null, promptTitle, msg);
 };
 
 /**
  * general prompts for a string method
- *
+ * 
  * @return String with the password
  */
 exports.prompt = function prompt(prompt, defaultValue) {
   var stringValue = {
-    value: defaultValue ? defaultValue : ""
+    value : defaultValue ? defaultValue : ""
   };
 
   var prompts = Cc["@mozilla.org/embedcomp/prompt-service;1"]
@@ -44,7 +44,7 @@ exports.prompt = function prompt(prompt, defaultValue) {
 
 /**
  * returns password with a special password
- *
+ * 
  * @return String with the password
  */
 exports.promptPassword = function promptPassword(prompt) {
@@ -59,8 +59,8 @@ exports.promptPassword = function promptPassword(prompt) {
   var check = {
     value : true
   }; // default the checkbox to true
-  var result = prompts.promptPassword(null, "Bugzilla Triage Script", prompt,
-      password, null, check);
+  var result = prompts.promptPassword(null,
+      "Bugzilla Triage Script", prompt, password, null, check);
   // result is true if OK was pressed, false if cancel was pressed.
   // password.value is set if OK was pressed.
   // The checkbox is not displayed.
@@ -83,8 +83,9 @@ exports.promptYesNoCancel = function promptOKNoCancel(prompt) {
   var prompts = Cc["@mozilla.org/embedcomp/prompt-service;1"]
       .getService(Ci.nsIPromptService);
 
-  var result = prompts.confirmEx(null, "Bugzilla Triage Script", prompt,
-      prompts.STD_YES_NO_BUTTONS, null, null, null, null, {});
+  var result = prompts.confirmEx(null, "Bugzilla Triage Script",
+      prompt, prompts.STD_YES_NO_BUTTONS, null, null, null,
+      null, {});
   if (result === 0) {
     return true;
   }
@@ -97,22 +98,22 @@ exports.promptYesNoCancel = function promptOKNoCancel(prompt) {
 };
 
 /**
- *
+ * 
  * documentation is https://developer.mozilla.org/en/NsIFilePicker
  */
-exports.promptFileOpenPicker = function promptFilePicker (win) {
+exports.promptFileOpenPicker = function promptFilePicker(win) {
   var window = require("window-utils").activeWindow;
   var fp = Cc["@mozilla.org/filepicker;1"]
-    .createInstance(Ci.nsIFilePicker);
+      .createInstance(Ci.nsIFilePicker);
   fp.init(window, "JSON File Open", Ci.nsIFilePicker.modeOpen);
   fp.appendFilter("JSON files", "*.json");
   fp.appendFilters(Ci.nsIFilePicker.filterAll);
   fp.filterIndex = 0;
   var res = fp.show();
 
-  if (res === Ci.nsIFilePicker.returnOK ||
-    res === Ci.nsIFilePicker.returnReplace ) {
-      return fp.file.path;
+  if (res === Ci.nsIFilePicker.returnOK
+      || res === Ci.nsIFilePicker.returnReplace) {
+    return fp.file.path;
   }
   return null;
 };
diff --git a/lib/util.js b/lib/util.js
index d28f001..008d6fb 100644
--- a/lib/util.js
+++ b/lib/util.js
@@ -11,7 +11,7 @@ var urlMod = require("url");
 /**
  * get parameters of URL as an object (name, value)
  */
-function getParamsFromURL (url, base) {
+function getParamsFromURL(url, base) {
   if (!url || (url.toString().length === 0)) {
     throw new Error("Missing URL value!");
   }
@@ -23,7 +23,7 @@ function getParamsFromURL (url, base) {
   var paramsArr = url.path.split("?");
   if (paramsArr.length === 1) {
     return {};
-   }
+  }
 
   // get convert URL parameters to an Object
   var params = {}, s = [];
@@ -35,11 +35,11 @@ function getParamsFromURL (url, base) {
 }
 
 /**
- * Get a bug no from URL ... fails with aliases
- * It should theoretically belong to bzpage.js, but we don't have
- * unit tests there yet, so keeping here.
- *
- * @param url String with URL to be analyzed
+ * Get a bug no from URL ... fails with aliases It should theoretically belong
+ * to bzpage.js, but we don't have unit tests there yet, so keeping here.
+ * 
+ * @param url
+ *          String with URL to be analyzed
  * @return String with the bug ID (hopefully number, but not for aliases)
  */
 exports.getBugNo = function getBugNo(url) {
@@ -51,7 +51,7 @@ exports.getBugNo = function getBugNo(url) {
 
 /**
  * format date to be in ISO format (just day part)
- *
+ * 
  * @param date
  * @return string with the formatted date
  */
@@ -60,18 +60,15 @@ exports.getISODate = function getISODate(dateStr) {
     return n < 10 ? '0' + n : n;
   }
   var date = new Date(dateStr);
-  return date.getFullYear() + '-' + pad(date.getMonth() + 1) + '-' +
-      pad(date.getDate());
+  return date.getFullYear() + '-' + pad(date.getMonth() + 1)
+      + '-' + pad(date.getDate());
 };
 
 /**
- * object to pack messaging. Use as in
-    postMessage(new Message("GetPassword", {
-      login: login,
-      hostname: location.hostname
-    }));
+ * object to pack messaging. Use as in postMessage(new Message("GetPassword", {
+ * login: login, hostname: location.hostname }));
  */
 exports.Message = function Message(cmd, data) {
-	this.cmd = cmd;
-	this.data = data;
+  this.cmd = cmd;
+  this.data = data;
 };
diff --git a/lib/xmlrpc.js b/lib/xmlrpc.js
index e038c4e..3991b04 100644
--- a/lib/xmlrpc.js
+++ b/lib/xmlrpc.js
@@ -3,41 +3,44 @@
 // Modification of Matěj Cepl released under the MIT/X11 license
 // http://www.opensource.org/licenses/mit-license.php
 /*
- *
+ * 
  * xmlrpc.js beta version 1 Tool for creating XML-RPC formatted requests in
  * JavaScript
- *
+ * 
  * Copyright 2001 Scott Andrew LePera scott@scottandrew.com
- *
- *
+ * 
+ * 
  * License: You are granted the right to use and/or redistribute this code only
  * if this license and the copyright notice are included and you accept that no
  * warranty of any kind is made or implied by the author.
- *
+ * 
  */
 
 /**
  * checks whether parameter is an array
- *
- * @param obj Object
+ * 
+ * @param obj
+ *          Object
  * @return Boolean true if obj is array
- *
+ * 
  * The problem is that in different contexts, Array is not same, and so obj is
  * not an instance of SAME Array.
  */
 
 /**
  * pads a single number with a leading zero. Heh.
- *
- * @param n String or Number
+ * 
+ * @param n
+ *          String or Number
  * @return String with leading zero added if necessary
- *
+ * 
  * If the real parameter is not numerical, it is just returned as it is.
  */
 var leadingZero = exports.leadingZero = function leadingZero(n) {
   if (isNaN(Number(n))) {
     return n;
-  };
+  }
+  ;
 
   var s = "0" + n;
   if (s.length > 2) {
@@ -46,22 +49,24 @@ var leadingZero = exports.leadingZero = function leadingZero(n) {
   return s;
 };
 
-var dateToISO8601 = exports.dateToISO8601 = function dateToISO8601(date) {
+var dateToISO8601 = exports.dateToISO8601 = function dateToISO8601(
+    date) {
   // wow I hate working with the Date object
   console.log("date = " + date);
   var year = date.getYear();
   var month = leadingZero(date.getMonth());
   var day = leadingZero(date.getDate());
-  var time = leadingZero(date.getHours()) +
-    ":" + leadingZero(date.getMinutes()) +
-    ":" + leadingZero(date.getSeconds());
+  var time = leadingZero(date.getHours()) + ":"
+      + leadingZero(date.getMinutes()) + ":"
+      + leadingZero(date.getSeconds());
 
   var converted = year + month + day + "T" + time;
   console.log("date = " + converted);
   return converted;
 };
 
-var XMLRPCMessage = exports.XMLRPCMessage = function XMLRPCMessage(methodname) {
+var XMLRPCMessage = exports.XMLRPCMessage = function XMLRPCMessage(
+    methodname) {
   this.method = methodname || "system.listMethods";
   this.params = [];
   return this;
@@ -71,20 +76,19 @@ XMLRPCMessage.prototype.myIsArray = function myIsArray(obj) {
   return (typeof obj.sort === 'function');
 };
 
-
-XMLRPCMessage.prototype.setMethod = function (methodName) {
+XMLRPCMessage.prototype.setMethod = function(methodName) {
   if (methodName !== undefined) {
     this.method = methodName;
   }
 };
 
-XMLRPCMessage.prototype.addParameter = function (data) {
+XMLRPCMessage.prototype.addParameter = function(data) {
   if (data !== undefined) {
     this.params.push(data);
   }
 };
 
-XMLRPCMessage.prototype.xml = function () {
+XMLRPCMessage.prototype.xml = function() {
 
   var method = this.method;
 
@@ -97,11 +101,11 @@ XMLRPCMessage.prototype.xml = function () {
   xml += "<params>\n";
 
   // do individual parameters
-  this.params.forEach(function (data) {
+  this.params.forEach(function(data) {
     xml += "<param>\n";
-    xml += "<value>" +
-      this.getParamXML(this.dataTypeOf(data),
-        data) + "</value>\n";
+    xml += "<value>"
+        + this.getParamXML(this.dataTypeOf(data), data)
+        + "</value>\n";
     xml += "</param>\n";
   }, this);
   xml += "</params>\n";
@@ -110,7 +114,7 @@ XMLRPCMessage.prototype.xml = function () {
   return xml; // for now
 };
 
-XMLRPCMessage.prototype.dataTypeOf = function (o) {
+XMLRPCMessage.prototype.dataTypeOf = function(o) {
   // identifies the data type
   var type = typeof (o);
   type = type.toLowerCase();
@@ -138,49 +142,50 @@ XMLRPCMessage.prototype.dataTypeOf = function (o) {
   return type;
 };
 
-XMLRPCMessage.prototype.doValueXML = function (type, data) {
+XMLRPCMessage.prototype.doValueXML = function(type, data) {
   var xml = "<" + type + ">" + data + "</" + type + ">";
   return xml;
 };
 
-XMLRPCMessage.prototype.doBooleanXML = function (data) {
+XMLRPCMessage.prototype.doBooleanXML = function(data) {
   var value = (data === true) ? 1 : 0;
   var xml = "<boolean>" + value + "</boolean>";
   return xml;
 };
 
-XMLRPCMessage.prototype.doDateXML = function (data) {
+XMLRPCMessage.prototype.doDateXML = function(data) {
   var xml = "<dateTime.iso8601>";
   xml += dateToISO8601(data);
   xml += "</dateTime.iso8601>";
   return xml;
 };
 
-XMLRPCMessage.prototype.doArrayXML = function (data) {
+XMLRPCMessage.prototype.doArrayXML = function(data) {
   var xml = "<array><data>\n";
-  for (var i = 0; i < data.length; i++) {
-    xml += "<value>" +
-      this.getParamXML(this.dataTypeOf(data[i]),
-        data[i]) + "</value>\n";
+  for ( var i = 0; i < data.length; i++) {
+    xml += "<value>"
+        + this.getParamXML(this.dataTypeOf(data[i]), data[i])
+        + "</value>\n";
   }
   xml += "</data></array>\n";
   return xml;
 };
 
-XMLRPCMessage.prototype.doStructXML = function (data) {
+XMLRPCMessage.prototype.doStructXML = function(data) {
   var xml = "<struct>\n";
-  for (var i in data) {
+  for ( var i in data) {
     xml += "<member>\n";
     xml += "<name>" + i + "</name>\n";
-    xml += "<value>" + this.getParamXML(this.dataTypeOf(data[i]),
-            data[i]) + "</value>\n";
+    xml += "<value>"
+        + this.getParamXML(this.dataTypeOf(data[i]), data[i])
+        + "</value>\n";
     xml += "</member>\n";
   }
   xml += "</struct>\n";
   return xml;
 };
 
-XMLRPCMessage.prototype.getParamXML = function (type, data) {
+XMLRPCMessage.prototype.getParamXML = function(type, data) {
   var xml;
   switch (type) {
   case "date":
diff --git a/tests/pagemod-test-helpers.js b/tests/pagemod-test-helpers.js
index 8621421..9980715 100644
--- a/tests/pagemod-test-helpers.js
+++ b/tests/pagemod-test-helpers.js
@@ -1,10 +1,10 @@
 "use strict";
 
-const {Cc,Ci} = require("chrome");
-
+const Cc = require("chrome").Cc;
+const Ci = require("chrome").Ci;
 /**
- * A helper function that creates a PageMod, then opens the specified URL
- * and checks the effect of the page mod on 'onload' event via testCallback.
+ * A helper function that creates a PageMod, then opens the specified URL and
+ * checks the effect of the page mod on 'onload' event via testCallback.
  */
 exports.testPageMod = function testPageMod(test, testURL, pageModOptions,
                        testCallback, timeout) {
@@ -20,7 +20,7 @@ exports.testPageMod = function testPageMod(test, testURL, pageModOptions,
   var browserWindow = wm.getMostRecentWindow("navigator:browser");
   if (!browserWindow) {
   test.pass("page-mod tests: could not find the browser window, so " +
-        "will not run. Use -a firefox to run the pagemod tests.")
+        "will not run. Use -a firefox to run the pagemod tests.");
   return null;
   }
 
@@ -44,7 +44,7 @@ exports.testPageMod = function testPageMod(test, testURL, pageModOptions,
   function onPageLoad() {
   b.removeEventListener("load", onPageLoad, true);
   testCallback(b.contentWindow.wrappedJSObject, function done() {
-    pageMods.forEach(function(mod) mod.destroy());
+    pageMods.forEach(function(mod) {mod.destroy()});
     // XXX leaks reported if we don't close the tab?
     tabBrowser.removeTab(newTab);
     test.done();
diff --git a/tests/test-color.js b/tests/test-color.js
index eefb97d..95d3e16 100644
--- a/tests/test-color.js
+++ b/tests/test-color.js
@@ -1,31 +1,31 @@
 /*global exports: false, require: false */
 // TODO: add some failing tests as well
 "use strict";
-//var util = require("color");
+// var util = require("color");
 
 // testing Color object
-var ensureColorNew = function (test) {
+var ensureColorNew = function(test) {
   var col = new util.Color(255, 255, 166);
   test.assertEqual(col.toString(), "#ffffa6",
-    "creation of new RGB Color object");
+      "creation of new RGB Color object");
 };
 
-var ensureColorUpdate = function (test) {
+var ensureColorUpdate = function(test) {
   var col = new util.Color(255, 255, 166);
   col.update(255, 224, 176);
   test.assertEqual(col.toString(), "#ffe0b0",
-    "updating Color object");
+      "updating Color object");
 };
 
-var ensureColorHSL = function (test) {
+var ensureColorHSL = function(test) {
   var col = new util.Color(255, 224, 176);
   test.assertEqual(col.hsl().toSource(),
-    "[0.10126582278481013, 1, 0.8450980392156863]",
-    "converting to HSL model");
+      "[0.10126582278481013, 1, 0.8450980392156863]",
+      "converting to HSL model");
 };
 
-var ensureColorLight = function (test) {
+var ensureColorLight = function(test) {
   var col = new util.Color(255, 224, 176);
   test.assertEqual(col.lightColor().toString(), "#e8dcc9",
-    "getting a light color");
+      "getting a light color");
 };
diff --git a/tests/test-logger.js b/tests/test-logger.js
index 2b087f8..d814838 100644
--- a/tests/test-logger.js
+++ b/tests/test-logger.js
@@ -6,34 +6,34 @@ var logMod = require("logger");
 var selfMod = require("self");
 
 var testGenerateTimeSheetDataLogs = {
-  "2010-07-27+bugzilla.redhat.com+533567": {
-      "date": "2010-07-27T21:28:47.103Z",
-      "url": "https://bugzilla.redhat.com/show_bug.cgi?id=533567",
-      "title": "KMS:RS480:X200M - GPU lockup (screen goes black)",
-      "comment": "removing filled in chip type for Guadec"
+  "2010-07-27+bugzilla.redhat.com+533567" : {
+    "date" : "2010-07-27T21:28:47.103Z",
+    "url" : "https://bugzilla.redhat.com/show_bug.cgi?id=533567",
+    "title" : "KMS:RS480:X200M - GPU lockup (screen goes black)",
+    "comment" : "removing filled in chip type for Guadec"
   },
-  "2010-07-27+bugzilla.redhat.com+618769": {
-      "date": "2010-07-27T21:30:18.845Z",
-      "url": "https://bugzilla.redhat.com/show_bug.cgi?id=618769",
-      "title": "gdm and display unstable with ATI FirePro V3700 graphics card",
-      "comment": "asking for logs"
+  "2010-07-27+bugzilla.redhat.com+618769" : {
+    "date" : "2010-07-27T21:30:18.845Z",
+    "url" : "https://bugzilla.redhat.com/show_bug.cgi?id=618769",
+    "title" : "gdm and display unstable with ATI FirePro V3700 graphics card",
+    "comment" : "asking for logs"
   }
 };
-var testGenerateTimeSheetResultStr = "<!DOCTYPE html><html><head>\n<meta charset='utf-8'/>\n"+
-  "<title>TimeSheet-2011-04-17</title>\n</head>\n<body>\n<h1>TimeSheet</h1>\n"+
-  "<hr/><p><strong>2010-07-27</strong></p>\n<p><em>"+
-  "<a href='https://bugzilla.redhat.com/show_bug.cgi?id=533567'>Bug RH/533567: "+
-  "KMS:RS480:X200M - GPU lockup (screen goes black)</a> </em>\n<br/>removing filled "+
-  "in chip type for Guadec</p>\n<p><em><a "+
-  "href='https://bugzilla.redhat.com/show_bug.cgi?id=618769'>Bug RH/618769: "+
-  "gdm and display unstable with ATI FirePro V3700 graphics card</a> </em>\n"+
-  "<br/>asking for logs</p>\n</body></html>";
+var testGenerateTimeSheetResultStr = "<!DOCTYPE html><html><head>\n<meta charset='utf-8'/>\n"
+    + "<title>TimeSheet-2011-04-17</title>\n</head>\n<body>\n<h1>TimeSheet</h1>\n"
+    + "<hr/><p><strong>2010-07-27</strong></p>\n<p><em>"
+    + "<a href='https://bugzilla.redhat.com/show_bug.cgi?id=533567'>Bug RH/533567: "
+    + "KMS:RS480:X200M - GPU lockup (screen goes black)</a> </em>\n<br/>removing filled "
+    + "in chip type for Guadec</p>\n<p><em><a "
+    + "href='https://bugzilla.redhat.com/show_bug.cgi?id=618769'>Bug RH/618769: "
+    + "gdm and display unstable with ATI FirePro V3700 graphics card</a> </em>\n"
+    + "<br/>asking for logs</p>\n</body></html>";
 
-exports.ensureTimeSheetRecordsPrinter = function (test) {
-  logMod.initialize(JSON.parse(selfMod.data.load(
-              "bugzillalabelAbbreviations.json")))
-  test.assertEqual(logMod.timeSheetRecordsPrinter(testGenerateTimeSheetDataLogs,
-    new Date("2011-04-17")), testGenerateTimeSheetResultStr,
-    "Generates correct log report from given data.");
+exports.ensureTimeSheetRecordsPrinter = function(test) {
+  logMod.initialize(JSON.parse(selfMod.data
+      .load("bugzillalabelAbbreviations.json")));
+  test.assertEqual(logMod.timeSheetRecordsPrinter(
+      testGenerateTimeSheetDataLogs, new Date("2011-04-17")),
+      testGenerateTimeSheetResultStr,
+      "Generates correct log report from given data.");
 };
-
diff --git a/tests/test-match-pattern.js b/tests/test-match-pattern.js
index 9e3d2bd..106937a 100644
--- a/tests/test-match-pattern.js
+++ b/tests/test-match-pattern.js
@@ -1,11 +1,15 @@
-var { MatchPattern } = require("match-pattern");
+var MatchPattern = require("match-pattern").MatchPattern;
 
-exports.ensureMatchPattern = function (test) {
-  var pattern = new MatchPattern("https://bugzilla.redhat.com/attachment.cgi");
+exports.ensureMatchPattern = function(test) {
+  var pattern = new MatchPattern(
+      "https://bugzilla.redhat.com/attachment.cgi");
   console.log("pattern = " + pattern);
-  test.assert(pattern.
-    test("https://bugzilla.redhat.com/attachment.cgi"), "testing match pattern");
-  test.assert(!pattern.
-    test("https://bugzilla.redhat.com/attachment.cgi?bugid=676538&action=enter"),
-    "testing failing match pattern");
+  test.assert(pattern
+      .test("https://bugzilla.redhat.com/attachment.cgi"),
+      "testing match pattern");
+  test
+      .assert(
+          !pattern
+              .test("https://bugzilla.redhat.com/attachment.cgi?bugid=676538&action=enter"),
+          "testing failing match pattern");
 };
diff --git a/tests/test-pageMod.js b/tests/test-pageMod.js
index 5434bbd..31cb3dd 100644
--- a/tests/test-pageMod.js
+++ b/tests/test-pageMod.js
@@ -13,126 +13,121 @@ var testURL = self.data.url('tests/change-more-bugs01.html');
 var JSONifiedMessage = '{"cmd":"testMessage","data":{"a":"first","b":"second"}}';
 
 exports.ensureMessagesWork = function(test) {
-  var msg = new utilMod.Message("testMessage", { a: "first", b: "second" } );
+  var msg = new utilMod.Message("testMessage", {
+    a : "first",
+    b : "second"
+  });
   test.assertEqual(msg.cmd, "testMessage",
-    "msg.cmd comes over well");
+      "msg.cmd comes over well");
   test.assertEqual(msg.data.a, "first",
-    "msg.data.a comes over well");
+      "msg.data.a comes over well");
   test.assertEqual(msg.data.b, "second",
-    "msg.data.b comes over well");
+      "msg.data.b comes over well");
   test.assertEqual(JSON.stringify(msg), JSONifiedMessage,
-    "JSONification of Message works as well");
+      "JSONification of Message works as well");
 };
 
 /*
+ * 
+ * var theURL = main.theURL; var testURL =
+ * self.data.url('tests/change-more-bugs01.html');
+ * 
+ * var contentScriptLibraries = { "bugzilla.redhat.com": [
+ * self.data.url("util.js"), self.data.url("color.js"),
+ * self.data.url("rhbzpage.js"), self.data.url("bzpage.js") ] };
+ * 
+ * libbz.initialize(libbz.config, function () { pageMod.PageMod({ include: [
+ * "https://bugzilla.redhat.com/show_bug.cgi?id=*" ], contentScriptWhen:
+ * 'ready', contentScriptFile: contentScriptLibraries["bugzilla.redhat.com"],
+ * onAttach: function onAttach(worker, msg) { console.log("worker: " + worker);
+ * worker.on('message', function (msg) { messageHandler(worker, msg); }); } });
+ * });
+ * 
+ * pageMod.PageMod({ include: [ "https://bugzilla.redhat.com/process_bug.cgi" ],
+ * contentScriptWhen: 'ready', contentScriptFile: self.data.url("skip-bug.js")
+ * });
+ */
 
-var theURL = main.theURL;
-var testURL = self.data.url('tests/change-more-bugs01.html');
-
-var contentScriptLibraries = {
-  "bugzilla.redhat.com": [
-    self.data.url("util.js"),
-    self.data.url("color.js"),
-     self.data.url("rhbzpage.js"),
-    self.data.url("bzpage.js")
-  ]
-};
-
-libbz.initialize(libbz.config, function () {
-  pageMod.PageMod({
-    include: [
-      "https://bugzilla.redhat.com/show_bug.cgi?id=*"
-    ],
-    contentScriptWhen: 'ready',
-    contentScriptFile: contentScriptLibraries["bugzilla.redhat.com"],
-    onAttach: function onAttach(worker, msg) {
-      console.log("worker: " + worker);
-      worker.on('message', function (msg) {
-        messageHandler(worker, msg);
-      });
+var ensureSimplePageLoad = function(test) {
+  console.log("testURL = " + testURL);
+  testPageMod(test, testURL, [
+    {
+      include : [
+        "*"
+      ],
+      contentScriptWhen : 'ready',
+      contentScriptFile : [
+          self.data.url("libPW.js"),
+          self.data.url("simplePageWorker.js")
+      ],
+      onAttach : function onAttach(worker) {
+        worker.on('message', function(msg) {
+          switch (msg.cmd) {
+          case "LogMessage":
+            log(msg.data);
+            break;
+          case "CallBack":
+            worker
+                .postMessage(new utilMod.Message("Main", null));
+            break;
+          default:
+            console.error(msg);
+          }
+        });
+      }
     }
+  ], function(win, done) {
+    test.assertNotEqual(win.document
+        .getElementsByTagName("form")[0], null,
+        "test of loading the page");
+    done();
   });
-});
-
-pageMod.PageMod({
-  include: [
-    "https://bugzilla.redhat.com/process_bug.cgi"
-  ],
-  contentScriptWhen: 'ready',
-  contentScriptFile: self.data.url("skip-bug.js")
-});
- */
-
-var ensureSimplePageLoad = function (test) {
-	console.log("testURL = " + testURL);
-	testPageMod(test, testURL, [{
-		include: ["*"],
-		contentScriptWhen: 'ready',
-		contentScriptFile: [
-			self.data.url("libPW.js"),
-			self.data.url("simplePageWorker.js")
-		],
-		onAttach: function onAttach(worker) {
-			worker.on('message', function (msg) {
-				switch (msg.cmd) {
-					case "LogMessage":
-						log(msg.data);
-						break;
-					case "CallBack":
-						worker.postMessage(new utilMod.Message("Main", null));
-						break;
-					default:
-						console.error(msg);
-				}
-			});
-		}
-	}],
-	function (win, done) {
-		test.assertNotEqual(win.document.getElementsByTagName("form")[0],
-			null, "test of loading the page");
-		done();
-	});
 };
 
-var ensurePageLoadsWell = function (test) {
+var ensurePageLoadsWell = function(test) {
   var wm = Cc['@mozilla.org/appshell/window-mediator;1']
-       .getService(Ci.nsIWindowMediator);
-  var browserWindow = wm.getMostRecentWindow("navigator:browser");
+      .getService(Ci.nsIWindowMediator);
+  var browserWindow = wm
+      .getMostRecentWindow("navigator:browser");
   if (!browserWindow) {
-  test.fail("page-mod tests: could not find the browser window, so " +
-        "will not run. Use -a firefox to run the pagemod tests.");
-  return null;
+    test
+        .fail("page-mod tests: could not find the browser window, so "
+            + "will not run. Use -a firefox to run the pagemod tests.");
+    return null;
   }
 
   var loader = test.makeSandboxedLoader();
   var pageMod = loader.require("page-mod");
-  var testDoc = {}, b = {}, tabBrowser = {}, newTab  = {};
+  var testDoc = {}, b = {}, tabBrowser = {}, newTab = {};
 
   pageMod.PageMod({
-		include: ["*"],
-		contentScriptWhen: 'ready',
-		contentScriptFile: [
-			self.data.url("libPW.js"),
-			self.data.url("pageWorker.js")
-		],
-		onAttach: function onAttach(worker) {
-			worker.on('message', function (msg) {
-				switch (msg.cmd) {
-					case "testReady":
-						testDoc = b.contentWindow.wrappedJSObject.document;
-						test.assertNotEqual(testDoc.getElementById("dupeid_action"),
-							null, "test of DOM modifications of the page");
-						pageMod.destroy();
-						tabBrowser.removeTab(newTab);
-						test.done();
-						// the test itself
-						break;
-					default:
-						main.messageHandler(worker, msg);
-				}
-			});
-		}
-	});
+    include : [
+      "*"
+    ],
+    contentScriptWhen : 'ready',
+    contentScriptFile : [
+        self.data.url("libPW.js"),
+        self.data.url("pageWorker.js")
+    ],
+    onAttach : function onAttach(worker) {
+      worker.on('message', function(msg) {
+        switch (msg.cmd) {
+        case "testReady":
+          testDoc = b.contentWindow.wrappedJSObject.document;
+          test.assertNotEqual(testDoc
+              .getElementById("dupeid_action"), null,
+              "test of DOM modifications of the page");
+          pageMod.destroy();
+          tabBrowser.removeTab(newTab);
+          test.done();
+          // the test itself
+          break;
+        default:
+          main.messageHandler(worker, msg);
+        }
+      });
+    }
+  });
 
   tabBrowser = browserWindow.gBrowser;
   newTab = tabBrowser.addTab(testURL);
diff --git a/tests/test-util.js b/tests/test-util.js
index 26c5f6d..7ab6280 100644
--- a/tests/test-util.js
+++ b/tests/test-util.js
@@ -6,13 +6,13 @@ var util = require("util");
 var urlMod = require("url");
 
 // testing util.heir
-var ensureHeir = function (test) {
+var ensureHeir = function(test) {
   var fedlimid = {}, naoise = {};
 
   function Father(x) {
     this.family = x;
   }
-  
+
   Father.prototype.getFamily = function getFamily() {
     return this.family;
   };
@@ -21,194 +21,216 @@ var ensureHeir = function (test) {
     Father.call(this, x);
     this.wife = w;
   }
-    
+
   Son.prototype = util.heir(Father);
   Son.prototype.constructor = Son;
-  
+
   Son.prototype.getWife = function getWife() {
     return this.wife;
   };
-  
+
   Son.prototype.getFamily = function getFamily() {
-    var upFamily =
-      Father.prototype.getFamily.call(this);
+    var upFamily = Father.prototype.getFamily.call(this);
     return upFamily + ", " + this.wife;
   };
-  
+
   // for curious and non-Celtic
   // http://en.wikipedia.org/wiki/Deirdre :)
   fedlimid = new Father("mac Daill");
   naoise = new Son("Usnech", "Deirdre");
-  
+
   test.assertEqual(fedlimid.getFamily(), "mac Daill",
-    "checking creation of new simple object");
-  
+      "checking creation of new simple object");
+
   test.assertEqual(naoise.getWife(), "Deirdre",
-    "checking creation of new daughter object");
+      "checking creation of new daughter object");
 
   test.assertEqual(naoise.getFamily(), "Usnech, Deirdre",
-    "checking creation of new overloaded method");
+      "checking creation of new overloaded method");
 };
 
 // testing util.isInList
-var ensureIsInListTrue = function (test) {
-  test.assert(util.isInList("a", ["a"]),
-    "conversion of a string to an array");
+var ensureIsInListTrue = function(test) {
+  test.assert(util.isInList("a", [
+    "a"
+  ]), "conversion of a string to an array");
 };
 
-var ensureIsInListFalse = function (test) {
-  test.assert(!util.isInList("b", ["a"]),
-    "conversion of a string to an array");
+var ensureIsInListFalse = function(test) {
+  test.assert(!util.isInList("b", [
+    "a"
+  ]), "conversion of a string to an array");
 };
 
-var ensureIsInListEmpty = function (test) {
+var ensureIsInListEmpty = function(test) {
   test.assert(!util.isInList("b", []),
-    "conversion of a string to an array");
+      "conversion of a string to an array");
 };
 
-var ensureIsInListNoMember = function (test) {
-  test.assert(!util.isInList("", ["x"]),
-    "conversion of a string to an array");
+var ensureIsInListNoMember = function(test) {
+  test.assert(!util.isInList("", [
+    "x"
+  ]), "conversion of a string to an array");
 };
 
 // testing util.filterByRegexp
-var ensureFilterByRegexp = function (test) {
+var ensureFilterByRegexp = function(test) {
   var list = [
-    {
-      "regexp": "test(ing|ed)",
-      "addr": "correct" 
-    },
-    {
-      "regexp": "ba.*d",
-      "addr": true
-    }
+      {
+        "regexp" : "test(ing|ed)",
+        "addr" : "correct"
+      }, {
+        "regexp" : "ba.*d",
+        "addr" : true
+      }
   ];
-  
-  test.assertEqual(util.filterByRegexp(list, "testing"), "correct",
-    "simple testing of filterByRegexp");
-  test.assertEqual(util.filterByRegexp(list, "unknown value"), "",
-    "simple testing of filterByRegexp with non-found address");
-  test.assert(util.filterByRegexp(list, "baaad"),
-    "simple testing of filterByRegexp with non-string return value");
+
+  test.assertEqual(util.filterByRegexp(list, "testing"),
+      "correct", "simple testing of filterByRegexp");
+  test.assertEqual(util.filterByRegexp(list, "unknown value"),
+      "",
+      "simple testing of filterByRegexp with non-found address");
+  test
+      .assert(util.filterByRegexp(list, "baaad"),
+          "simple testing of filterByRegexp with non-string return value");
 };
 
-var ensureFilterByRegexpEmpty = function (test) {
-  test.assertRaises(function () {
-      util.filterByRegexp(undefined, "tralala");
-    },
-    "list is undefined",
-    "filterByRegexp throws an exception with empty list");
+var ensureFilterByRegexpEmpty = function(test) {
+  test.assertRaises(function() {
+    util.filterByRegexp(undefined, "tralala");
+  }, "list is undefined",
+      "filterByRegexp throws an exception with empty list");
 };
 
 // testing util.getISODate
-var ensureGetISODate = function (test) {
-  test.assertEqual(util.getISODate("Mon May 31 2010 23:29:09 GMT+0200 (CET)"),
-    "2010-05-31", "conversion of a Date to ISO-formatted String");
+var ensureGetISODate = function(test) {
+  test.assertEqual(util
+      .getISODate("Mon May 31 2010 23:29:09 GMT+0200 (CET)"),
+      "2010-05-31",
+      "conversion of a Date to ISO-formatted String");
 };
 
 // testing util.valToArray
-var ensureValToArrayString = function (test) {
-  test.assertEqual(JSON.stringify(util.valToArray("a")),
-      JSON.stringify(["a"]),
-    "conversion of a string to an array");
+var ensureValToArrayString = function(test) {
+  test.assertEqual(JSON.stringify(util.valToArray("a")), JSON
+      .stringify([
+        "a"
+      ]), "conversion of a string to an array");
 };
 
-var ensureValToArrayEmpty = function (test) {
-  test.assertEqual(JSON.stringify(util.valToArray("")),
-         JSON.stringify([""]),
-    "conversion of an empty string to an array");
+var ensureValToArrayEmpty = function(test) {
+  test.assertEqual(JSON.stringify(util.valToArray("")), JSON
+      .stringify([
+        ""
+      ]), "conversion of an empty string to an array");
 };
 
-var ensureValToArrayArray = function (test) {
-  test.assertEqual(JSON.stringify(util.valToArray(["a"])),
-      JSON.stringify(["a"]),
-    "non-conversion of an array");
+var ensureValToArrayArray = function(test) {
+  test.assertEqual(JSON.stringify(util.valToArray([
+    "a"
+  ])), JSON.stringify([
+    "a"
+  ]), "non-conversion of an array");
 };
 
 // testing util.addCSVValue
-var ensureCSVAddedToNull = function (test) {
+var ensureCSVAddedToNull = function(test) {
   test.assertEqual(util.addCSVValue("", "b"), "b",
-    "adding a string to empty string");
+      "adding a string to empty string");
 };
 
-var ensureCSVAddedNull = function (test) {
+var ensureCSVAddedNull = function(test) {
   test.assertEqual(util.addCSVValue("a", ""), "a",
-    "adding nothing to a string");
+      "adding nothing to a string");
 };
 
-var ensureCSVAddedString = function (test) {
+var ensureCSVAddedString = function(test) {
   test.assertEqual(util.addCSVValue("a", "b"), "a, b",
-    "adding one string to another one");
+      "adding one string to another one");
 };
 
-var ensureCSVAddedArray = function (test) {
-  test.assertEqual(util.addCSVValue("a", ["b", "c"]), "a, b, c",
-    "adding array to a string");
+var ensureCSVAddedArray = function(test) {
+  test.assertEqual(util.addCSVValue("a", [
+      "b", "c"
+  ]), "a, b, c", "adding array to a string");
 };
 
-var ensureCSVAddedArray2Array = function (test) {
-  test.assertEqual(util.addCSVValue("a, b", ["c", "d"]), "a, b, c, d",
-    "adding one array to another");
+var ensureCSVAddedArray2Array = function(test) {
+  test.assertEqual(util.addCSVValue("a, b", [
+      "c", "d"
+  ]), "a, b, c, d", "adding one array to another");
 };
 
 // testing util.removeCSVValue
-var ensureCSVRemoveSimple = function (test) {
+var ensureCSVRemoveSimple = function(test) {
   test.assertEqual(util.removeCSVValue("a, b", "b"), "a",
-    "removing one string from an array");
+      "removing one string from an array");
 
 };
 
 // also checking a tolerancy against different ways of writing arrays
-var ensureCSVRemoveNonMember = function (test) {
-  test.assertEqual(util.removeCSVValue("a,b", "c"), "a, b",
-    "removing a string from an array of which it isn't a member");
+var ensureCSVRemoveNonMember = function(test) {
+  test
+      .assertEqual(util.removeCSVValue("a,b", "c"), "a, b",
+          "removing a string from an array of which it isn't a member");
 
 };
 
-var ensureCSVRemoveEmpty = function (test) {
+var ensureCSVRemoveEmpty = function(test) {
   test.assertEqual(util.removeCSVValue("", "c"), "",
-    "removing a string from an empty array");
+      "removing a string from an empty array");
 
 };
 
 // testing util.getObjectKeys
-var ensureGetObjectKeys = function (test) {
+var ensureGetObjectKeys = function(test) {
   var testObj = {
-    a: 1,
-    b: 2
+    a : 1,
+    b : 2
   };
   test.assertEqual(JSON.stringify(util.getObjectKeys(testObj)),
-      JSON.stringify(["a", "b"]),
-    "getting keys from a object");
+      JSON.stringify([
+          "a", "b"
+      ]), "getting keys from a object");
 };
 
 // testing util.getParamsFromURL
-var ensureGetParamsFromURL = function (test) {
-  test.assertEqual(JSON.stringify(util.getParamsFromURL("https://bugzilla.redhat.com/show_bug.cgi?id=549066")),
-    JSON.stringify({id:"549066"}),
-    "simply compare result of bugzilla show_page URL");
-  var complexURL = new urlMod.URL("http://www.google.com/search?ie=UTF-8&oe=UTF-8&sourceid=navclient"+
-    "&gfns=1&q=gg+javascript+url+parse");
-  test.assertEqual(JSON.stringify(util.getParamsFromURL(complexURL)),
-    JSON.stringify({
-      "ie":"UTF-8",
-      "oe":"UTF-8",
-      "sourceid":"navclient",
-      "gfns":"1",
-      "q":"gg+javascript+url+parse"
-    }),
-    "simply compare result of bugzilla show_page URL");
-  test.assertEqual(JSON.stringify(util.getParamsFromURL("https://bugzilla.redhat.com/")),
-    JSON.stringify({}),
-    "URL without any parameters");
-  test.assertRaises(function () {util.getParamsFromURL("");},
-    "Missing URL value!", "No URL");
+var ensureGetParamsFromURL = function(test) {
+  test
+      .assertEqual(
+          JSON
+              .stringify(util
+                  .getParamsFromURL("https://bugzilla.redhat.com/show_bug.cgi?id=549066")),
+          JSON.stringify({
+            id : "549066"
+          }), "simply compare result of bugzilla show_page URL");
+  var complexURL = new urlMod.URL(
+      "http://www.google.com/search?ie=UTF-8&oe=UTF-8&sourceid=navclient"
+          + "&gfns=1&q=gg+javascript+url+parse");
+  test.assertEqual(JSON.stringify(util
+      .getParamsFromURL(complexURL)), JSON.stringify({
+    "ie" : "UTF-8",
+    "oe" : "UTF-8",
+    "sourceid" : "navclient",
+    "gfns" : "1",
+    "q" : "gg+javascript+url+parse"
+  }), "simply compare result of bugzilla show_page URL");
+  test.assertEqual(JSON.stringify(util
+      .getParamsFromURL("https://bugzilla.redhat.com/")), JSON
+      .stringify({}), "URL without any parameters");
+  test.assertRaises(function() {
+    util.getParamsFromURL("");
+  }, "Missing URL value!", "No URL");
 };
 
 // testing util.getBugNo
-var ensureGetBugNo = function (test) {
-  var bugNo = util.getBugNo("https://bugzilla.redhat.com/show_bug.cgi?id=597141");
+var ensureGetBugNo = function(test) {
+  var bugNo = util
+      .getBugNo("https://bugzilla.redhat.com/show_bug.cgi?id=597141");
   test.assertEqual(bugNo, 597141, "getting bug number");
-  bugNo = util.getBugNo("https://bugzilla.redhat.com/show_bug.cgi?id=serialWacom");
-  test.assertEqual(bugNo, "serialWacom", "getting a bug alias; there is no guarantee of getting number!");
+  bugNo = util
+      .getBugNo("https://bugzilla.redhat.com/show_bug.cgi?id=serialWacom");
+  test
+      .assertEqual(bugNo, "serialWacom",
+          "getting a bug alias; there is no guarantee of getting number!");
 };
diff --git a/tests/test-xmlrpc.js b/tests/test-xmlrpc.js
index f3f2e56..78bf9d5 100644
--- a/tests/test-xmlrpc.js
+++ b/tests/test-xmlrpc.js
@@ -2,21 +2,21 @@
 /*jslint plusplus: false */
 "use strict";
 var xrpc = require("xmlrpc");
-var xmlOut = "<?xml version=\"1.0\"?>\n" +
-  "<methodCall>\n<methodName>bugzilla.updateAttachMimeType</methodName>\n" +
-  "<params>\n<param>\n<value><struct>\n<member>\n<name>attach_id</name>\n" +
-  "<value><string>myId</string></value>\n</member>\n<member>\n" +
-  "<name>mime_type</name>\n<value><string>text/plain</string></value>\n</member>\n" +
-  "<member>\n<name>nomail</name>\n<value><string>billg@microsoft.com</string>" +
-  "</value>\n</member>\n</struct>\n</value>\n</param>\n<param>\n" +
-  "<value><string>me@example.com</string></value>\n</param>\n" +
-  "<param>\n<value><string>secret</string></value>\n</param>\n" +
-  "<param>\n<value><double>3.14</double></value>\n</param>\n" +
-  "<param>\n<value><boolean>1</boolean></value>\n</param>\n" +
-  "</params>\n</methodCall>";
+var xmlOut = "<?xml version=\"1.0\"?>\n"
+    + "<methodCall>\n<methodName>bugzilla.updateAttachMimeType</methodName>\n"
+    + "<params>\n<param>\n<value><struct>\n<member>\n<name>attach_id</name>\n"
+    + "<value><string>myId</string></value>\n</member>\n<member>\n"
+    + "<name>mime_type</name>\n<value><string>text/plain</string></value>\n</member>\n"
+    + "<member>\n<name>nomail</name>\n<value><string>billg@microsoft.com</string>"
+    + "</value>\n</member>\n</struct>\n</value>\n</param>\n<param>\n"
+    + "<value><string>me@example.com</string></value>\n</param>\n"
+    + "<param>\n<value><string>secret</string></value>\n</param>\n"
+    + "<param>\n<value><double>3.14</double></value>\n</param>\n"
+    + "<param>\n<value><boolean>1</boolean></value>\n</param>\n"
+    + "</params>\n</methodCall>";
 
-exports.ensureLeadingZero = function (test) {
-  test.assert(typeof(xrpc.leadingZero) == "function");
+exports.ensureLeadingZero = function(test) {
+  test.assert(typeof (xrpc.leadingZero) == "function");
   test.assertEqual(xrpc.leadingZero("1"), "01");
   test.assertEqual(xrpc.leadingZero(1), "01");
   test.assertEqual(xrpc.leadingZero("11"), "11");
@@ -25,12 +25,13 @@ exports.ensureLeadingZero = function (test) {
   test.assertEqual(xrpc.leadingZero(111), "111");
   test.assertEqual(xrpc.leadingZero("-1"), "-1");
   test.assertEqual(xrpc.leadingZero(-1), "-1");
-  test.assertEqual(xrpc.leadingZero("zzz"),"zzz");
+  test.assertEqual(xrpc.leadingZero("zzz"), "zzz");
 };
 
 // testing xrpc.XMLRPCMessage
-exports.ensureGenerateXMLRPC = function (test) {
-  var msg = new xrpc.XMLRPCMessage("bugzilla.updateAttachMimeType");
+exports.ensureGenerateXMLRPC = function(test) {
+  var msg = new xrpc.XMLRPCMessage(
+      "bugzilla.updateAttachMimeType");
   msg.addParameter({
     'attach_id' : "myId",
     'mime_type' : "text/plain",
@@ -40,6 +41,6 @@ exports.ensureGenerateXMLRPC = function (test) {
   msg.addParameter("secret");
   msg.addParameter(3.14);
   msg.addParameter(true);
-  test.assertEqual(msg.xml(), xmlOut,
-    "generate XML-RPC message");
+  test
+      .assertEqual(msg.xml(), xmlOut, "generate XML-RPC message");
 };