JavaScript style fixups in feedback.js

chrome/browser/resources/feedback.js

Index: chrome/browser/resources/feedback.js
diff --git a/chrome/browser/resources/feedback.js b/chrome/browser/resources/feedback.js
index c5293401db7f81b71fdcde4621d6b3181356c9d8..817806f576c798aa44ac69c23abb61b68998440b 100644
--- a/chrome/browser/resources/feedback.js
+++ b/chrome/browser/resources/feedback.js
@@ -20,6 +20,8 @@ var localStrings = new LocalStrings();
 
 /**
  * Selects an image thumbnail in the specified div.
+ * @param {string} divId The id of the div to search in.
+ * @param {string} thumbnailId The id of the thumbnail to search for.
  */
 function selectImage(divId, thumbnailId) {
   var thumbnailDivs = $(divId).children;
@@ -43,6 +45,8 @@ function selectImage(divId, thumbnailId) {
 
 /**
  * Adds an image thumbnail to the specified div.
+ * @param {string} divId The id of the div to add a screenshot to.
+ * @param {string} screenshot The URL of the screenshot being added.
  */
 function addScreenshot(divId, screenshot) {
   var thumbnailDiv = document.createElement('div');
@@ -86,9 +90,10 @@ function disableScreenshots() {
 }
 
 /**
- * Send's the report; after the report is sent, we need to be redirected to
+ * Sends the report; after the report is sent, we need to be redirected to
  * the landing page, but we shouldn't be able to navigate back, hence
  * we open the landing page in a new tab and sendReport closes this tab.
+ * @return {boolean} True if the report was sent.
  */
 function sendReport() {
   if ($('description-text').value.length == 0) {
@@ -123,6 +128,10 @@ function sendReport() {
   return true;
 }
 
+/**
+ * Click listener for the cancel button.
+ * @return {boolean} true

[Dan Beam, 2012/04/27 03:28:46]

remove this and change to take |e| param and call e.preventDefault() IMO

+ */
 function cancel() {
   chrome.send('cancel');
   return true;
@@ -141,8 +150,6 @@ function currentSelected() {
   if (selectedThumbnailDivId != 'current-screenshots')
     selectImage('current-screenshots',
                 savedThumbnailIds['current-screenshots']);
-
-  return true;
 }
 
 /**
@@ -160,8 +167,6 @@ function savedSelected() {
     if (selectedThumbnailDivId != 'saved-screenshots')
       selectImage('saved-screenshots', savedThumbnailIds['saved-screenshots']);
   }
-
-  return true;
 }