Allow feedback form to be shown in an App Launcher feedback mode.

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 =
     'https://www.google.com/support/chrome/go/feedback_confirmation';
 /** @const */ var MAX_ATTACH_FILE_SIZE = 3 * 1024 * 1024;
 
 var selectedThumbnailDivId = '';
@@ skipping 77 matching lines @@
   innerDiv.appendChild(thumbnail);
 
   thumbnailDiv.appendChild(innerDiv);
   $(divId).appendChild(thumbnailDiv);
 
   if (!selectedThumbnailId)
     selectImage(divId, thumbnailDiv.id);
 }
 
 /**
- * Disables screenshots completely.
+ * Shows the app launcher version of the feedback form.
+ */
+function showLauncherFeedback() {
+  document.body.classList.remove('normal-layout');

[Dan Beam, 2013/03/19 04:43:53]

^ I don't think you need this

[benwells, 2013/03/20 07:49:32]

Done.

+  document.body.classList.add('launcher-layout');

[Dan Beam, 2013/03/19 04:43:53]

it's arguable that you should be using `document.documentElement` instead

[benwells, 2013/03/20 07:49:32]

I was using body as I didn't know how to add a style to the document in the
html. Now I don't need to :)

+}
+
+/**
+ * Enables screenshots.
  */
 function enableScreenshots() {
   if (forceDisableScreenshots)
     return;
   $('screenshot-row').hidden = false;
 }
 
 /**
  * Reads the selected file when the user selects a file.
  * @param {event} evtFileSelected The on changed event for the file input box.
@@ skipping 282 matching lines @@
 function setupDialogDefaults(defaults) {
   // Current url.
   if ($('page-url-text').value == '')
     $('page-url-text').value = defaults.currentUrl;
   if (defaults.currentUrl == '')
     $('page-url-checkbox').checked = false;
   // User e-mail.
   $('user-email-text').value = defaults.userEmail;
   $('user-email-checkbox').checked = defaults.emailCheckboxDefault;
 
-  // Are screenshots disabled?
+  if (defaults.launcherFeedback)
+    showLauncherFeedback();

[Dan Beam, 2013/03/19 04:43:53]

this could be reduced to

  document.documentElement.toggleClass(
      'launcher-layout', defaults.launcherFeedback);

[benwells, 2013/03/20 07:49:32]

That didn't seem to work. Do we have our own toggleClass thing (googling shows
it as a jQuery thing).

[Dan Beam, 2013/03/21 01:09:11]

sorry, meant document.documentElement.classList.toggle()

+
   if (!defaults.disableScreenshots)
     enableScreenshots();
 
   if (defaults.useSaved) {
     $('screenshot-link-tosaved').hidden = false;
   }
 }
 
 window.addEventListener('DOMContentLoaded', load);