Make crash reporting link creation easier to read

chrome/browser/resources/crashes.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.
 
 localStrings = new LocalStrings();
 
 /**
  * Requests the list of crashes from the backend.
  */
 function requestCrashes() {
@@ skipping 28 matching lines @@
     var title = document.createElement('h3');
     title.textContent = localStrings.getStringF('crashHeaderFormat',
                                                 crash['id']);
     crashBlock.appendChild(title);
     var date = document.createElement('p');
     date.textContent = localStrings.getStringF('crashTimeFormat',
                                                crash['time']);
     crashBlock.appendChild(date);
     var linkBlock = document.createElement('p');
     var link = document.createElement('a');
-    link.href = 'http://code.google.com/p/chromium/issues/entry?' +
-        'template=Crash%20Report&comment=' +
-        'Chrome%20Version:%20' + version + '%0A' +
-        'Operating%20System:%20e.g.,%20"Windows%207",%20' +
-        '"Mac%20OS%20X%2010.6"%0A%0A' +
-        'URL%20(if%20applicable)%20where%20crash%20occurred:%20%0A%0A' +
-        'Can%20you%20reproduce%20this%20crash?%0A%0A' +
-        'What%20steps%20will%20reproduce%20this%20crash%20' +
-        '(or%20if%20it\'s%20not%20reproducible,%20what%20were%20you%20doing' +
-        '%20just%20before%20the%20crash)?%0A1.%0A2.%0A3.%0A%0A' +
-        '*Please%20note%20that%20issues%20filed%20with%20no%20information%20' +
-        'filled%20in%20above%20will%20be%20marked%20as%20WontFix*%0A%0A' +
-        '****DO%20NOT%20CHANGE%20BELOW%20THIS%20LINE****%0Areport_id:' +
-        crash['id'];
+    var params = {
+      template: 'Crash Report',
+      comment: 'Chrome Version: ' + version + '\n' +

[Dan Beam, 2012/04/06 20:40:08]

probably easier to read if you:

  ['line',
   'line',
   'line'].join('\n')

[Tyler Breisacher (Chromium), 2012/04/30 21:38:23]

Done.

+          // TODO(tbreisacher): fill in the OS automatically?
+          'Operating System: e.g., "Windows 7", "Mac OSX 10.6"\n\n' +
+          'URL (if applicable) where crash occurred:\n\n' +
+          'Can you reproduce this crash?\n\n' +
+          'What steps will reproduce this crash? (or if it\'s not ' +
+          'reproducible, what were you doing just before the crash)?\n' +
+          '1.\n2.\n3.\n\n' +
+          '*Please note that issues filed with no information filled in ' +
+          'above will be marked as WontFix*\n\n' +
+          '****DO NOT CHANGE BELOW THIS LINE****\n' +
+          'report_id:' + crash.id,
+    };
+    var href = 'http://code.google.com/p/chromium/issues/entry?';
+    for (var param in params) {
+      href += encodeURIComponent(param) + '=' +
+          encodeURIComponent(params[param]) + '&';
+    }
+    link.href = href;
     link.target = '_blank';
     link.textContent = localStrings.getString('bugLinkText');
     linkBlock.appendChild(link);
     crashBlock.appendChild(linkBlock);
     crashSection.appendChild(crashBlock);
   }
 
   $('noCrashes').hidden = crashes.length != 0;
 }
 
 document.addEventListener('DOMContentLoaded', requestCrashes);