whip the feedback page into shape

chrome/browser/resources/feedback.js

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 // Constants.
 /** @const */ var FEEDBACK_LANDING_PAGE =
     'http://www.google.com/support/chrome/go/feedback_confirmation';
 
 var selectedThumbnailDivId = '';
 var selectedThumbnailId = '';
 var selectedImageUrl;
 
 var savedThumbnailIds = [];
 savedThumbnailIds['current-screenshots'] = '';
 savedThumbnailIds['saved-screenshots'] = '';
 
 var categoryTag = '';
 
-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;
   selectedThumbnailDivId = divId;
   if (thumbnailDivs.length == 0) {
     $(divId).hidden = true;
     return;
   }
+
   for (var i = 0; i < thumbnailDivs.length; i++) {
+    thumbnailDivs[i].className = 'image-thumbnail-container';
+
     // If the the current div matches the thumbnail id provided,
     // or there is no thumbnail id given, and we're at the first thumbnail.
-    if ((thumbnailDivs[i].id == thumbnailId) || (!thumbnailId && !i)) {
-      thumbnailDivs[i].className = 'image-thumbnail-container-selected';
+    if (thumbnailDivs[i].id == thumbnailId || (!thumbnailId && !i)) {
+      thumbnailDivs[i].classList.add('image-thumbnail-container-selected');
       selectedThumbnailId = thumbnailId;
       savedThumbnailIds[divId] = thumbnailId;
-    } else {
-      thumbnailDivs[i].className = 'image-thumbnail-container';
     }
   }
 }
 
 /**
  * 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');
   thumbnailDiv.className = 'image-thumbnail-container';
 
   thumbnailDiv.id = divId + '-thumbnailDiv-' + $(divId).children.length;
   thumbnailDiv.onclick = function() {
     selectImage(divId, thumbnailDiv.id);
   };
 
   var innerDiv = document.createElement('div');
-  if (divId == 'current-screenshots')
-    innerDiv.className = 'image-thumbnail-current';
-  else
-    innerDiv.className = 'image-thumbnail';
+  innerDiv.className = 'image-thumbnail';
 
   var thumbnail = document.createElement('img');
   thumbnail.id = thumbnailDiv.id + '-image';
   // We add the ?+timestamp to make sure the image URLs are unique
   // and Chrome does not load the image from cache.
   thumbnail.src = screenshot + '?' + Date.now();
   innerDiv.appendChild(thumbnail);
 
   thumbnailDiv.appendChild(innerDiv);
   $(divId).appendChild(thumbnailDiv);
@@ skipping 15 matching lines @@
 }
 
 /**
  * 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) {
-    alert(localStrings.getString('no-description'));
+    alert(loadTimeData.getString('no-description'));
     return false;
   }
 
   var imagePath = '';
   if ($('screenshot-checkbox').checked && selectedThumbnailId)
     imagePath = $(selectedThumbnailId + '-image').src;
   var pageUrl = $('page-url-text').value;
   if (!$('page-url-checkbox').checked)
     pageUrl = '';
 
@@ skipping 39 matching lines @@
   if (selectedThumbnailDivId != 'current-screenshots')
     selectImage('current-screenshots',
                 savedThumbnailIds['current-screenshots']);
 }
 
 /**
  * Select the saved screenshots div, restoring the image that was
  * selected when we had this div open previously.
  */
 function savedSelected() {
-  $('current-screenshots').hidden = true;
-
   if ($('saved-screenshots').childElementCount == 0) {
     // setupSavedScreenshots will take care of changing visibility
     chrome.send('refreshSavedScreenshots');
   } else {
+    $('current-screenshots').hidden = true;
     $('saved-screenshots').hidden = false;
     if (selectedThumbnailDivId != 'saved-screenshots')
       selectImage('saved-screenshots', savedThumbnailIds['saved-screenshots']);
   }
 }
 
-
 /**
  * Change the type of screenshot we're showing to the user from
  * the current screenshot to saved screenshots
  */
 function changeToSaved() {
   $('screenshot-label-current').hidden = true;
   $('screenshot-label-saved').hidden = false;
 
   // Change the link to say "go to original"
   $('screenshot-link-tosaved').hidden = true;
@@ skipping 77 matching lines @@
   chrome.send('refreshCurrentScreenshot');
 }
 
 function setupCurrentScreenshot(screenshot) {
   addScreenshot('current-screenshots', screenshot);
 }
 
 function setupSavedScreenshots(screenshots) {
   if (screenshots.length == 0) {
     $('saved-screenshots').textContent =
-        localStrings.getString('no-saved-screenshots');
+        loadTimeData.getString('no-saved-screenshots');
 
     // Make sure we make the display the message.
+    $('current-screenshots').hidden = true;
     $('saved-screenshots').hidden = false;
 
     // In case the user tries to send now; fail safe, do not send a screenshot
     // at all versus sending the current screenshot.
     selectedThumbnailDivId = '';
     selectedThumbnailId = '';
   } else {
     for (i = 0; i < screenshots.length; ++i)
       addScreenshot('saved-screenshots', screenshots[i]);
 
@@ skipping 19 matching lines @@
         $('user-email-table').hidden = true;
 
         // this also means we are in privacy mode, so no saved screenshots.
         $('screenshot-link-tosaved').hidden = true;
       }
     }
   }
 }
 
 window.addEventListener('DOMContentLoaded', load);