Implement chrome://crashes

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
new file mode 100644
index 0000000000000000000000000000000000000000..0f0062e23f93b71ee330b800bed4d213a75b8815
--- /dev/null
+++ b/chrome/browser/resources/crashes.js
@@ -0,0 +1,71 @@
+// Copyright (c) 2011 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() {
+  chrome.send('requestCrashList', [])
+}
+
+/**
+ * Callback from backend with the list of crashes. Builds the UI.
+ * @param {boolean} enabled Whether or not crash reporting is enabled.
+ * @param {array} crashes The list of crashes.
+ */
+function updateCrashList(enabled, crashes) {
+  $('countBanner').textContent = localStrings.getStringF('crashCountFormat',
+                                                         crashes.length);
+
+  var crashSection = $('crashList');
+
+  if (enabled) {
+    $('enabledMode').classList.remove('hidden');
+    $('disabledMode').classList.add('hidden');
+  } else {
+    $('enabledMode').classList.add('hidden');
+    $('disabledMode').classList.remove('hidden');
+    return;
+  }
+
+  // Clear any previous list.
+  while (crashSection.firstChild) {
+    crashSection.removeChild(crashSection.firstChild);
+  }

[Nico, 2011/02/18 18:16:01]

crashSection.innerHTML = "";

?

[stuartmorgan, 2011/02/18 19:41:21]

Done.

+
+  for (var i = 0; i < crashes.length; i++) {
+    var crash = crashes[i];
+
+    var crashBlock = document.createElement('div');
+    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?' +
+        'comment=URL%20(if%20applicable)%20where%20crash%20occurred:%20%0A%0A' +
+        'What%20steps%20will%20reproduce%20this%20crash?%0A1.%0A2.%0A3.%0A%0A' +
+        '****DO%20NOT%20CHANGE%20BELOW%20THIS%20LINE****%0Areport_id:' +
+        crash['id'];
+    link.target = '_blank';
+    link.textContent = localStrings.getString('bugLinkText');
+    linkBlock.appendChild(link);
+    crashBlock.appendChild(linkBlock);
+    crashSection.appendChild(crashBlock);

[Nico, 2011/02/18 18:16:01]

Have you considered using our html templating engine to create this? See e.g.
c/b/resources/flags.html:


      <div jsdisplay="flagsExperiments.length > 0">
      <div class="experiment" jsselect="flagsExperiments">
        <table width="100%" cellpadding="2" cellspacing="0">
        <tr class="experiment-enabled">
        <td valign="top">
          <div class="experiment-text">
            <div>
              <span class="experiment-name" dir="ltr"
                    jscontent="name">NAME</span>
              <div>
                <span dir="ltr" jsvalues=".innerHTML:description">
              </div>

etc. I don't know if there's a good way to run this asynchronously – just
wondering if you've investigated this.

[stuartmorgan, 2011/02/18 19:41:21]

I don't see a way to do it offhand. I can revisit later if I'm missing
something.

+  }
+
+  if (crashes.length == 0)
+    $('noCrashes').classList.remove('hidden');
+  else
+    $('noCrashes').classList.add('hidden');
+}
+
+document.addEventListener('DOMContentLoaded', requestCrashes);