Make crash reporting link creation easier to read

chrome/browser/resources/crashes.js

Index: chrome/browser/resources/crashes.js
diff --git a/chrome/browser/resources/crashes.js b/chrome/browser/resources/crashes.js
index b4cec0e1bcfec9813096d6410b2c573a9b91e2e8..e34488a6f90b4f2f69ad84c2e8f65cca5ff6242c 100644
--- a/chrome/browser/resources/crashes.js
+++ b/chrome/browser/resources/crashes.js
@@ -46,20 +46,27 @@ function updateCrashList(enabled, crashes, version) {
     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);