chrome://crashes: add a link on CrOS for triggering crash uploading

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.
 
+/* Id for tracking automatic refresh of crash list.  */
+var refreshCrashListId = undefined;
+
 /**
  * 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 {boolean} dynamic_backend Whether the crash backend is dynamic.
  * @param {array} crashes The list of crashes.
  * @param {string} version The browser version.
  */
-function updateCrashList(enabled, crashes, version) {
+function updateCrashList(enabled, dynamic_backend, crashes, version) {

[xiyuan, 2014/03/25 21:30:58]

dynamic_backend -> dynamicBackend

[vapier, 2014/03/25 21:35:46]

Done.

   $('countBanner').textContent = loadTimeData.getStringF('crashCountFormat',
                                                          crashes.length);
 
   var crashSection = $('crashList');
 
   $('enabledMode').hidden = !enabled;
   $('disabledMode').hidden = enabled;
 
+  if (dynamic_backend)
+    $('crashUploadStatus').hidden = !enabled;

[xiyuan, 2014/03/25 21:30:58]

nit: we can get rid "if"
  $('crashUploadStatus').hidden = !dynamicBackend || !enabled;

[vapier, 2014/03/25 21:35:46]

sure.  i plan on adding more UI elements, but they'll be inside this one div
element, so shouldn't need to toggle any others.

+
   if (!enabled)
     return;
 
   // Clear any previous list.
   crashSection.textContent = '';
 
   for (var i = 0; i < crashes.length; i++) {
     var crash = crashes[i];
 
     var crashBlock = document.createElement('div');
@@ skipping 39 matching lines @@
     link.target = '_blank';
     link.textContent = loadTimeData.getString('bugLinkText');
     linkBlock.appendChild(link);
     crashBlock.appendChild(linkBlock);
     crashSection.appendChild(crashBlock);
   }
 
   $('noCrashes').hidden = crashes.length != 0;
 }
 
-document.addEventListener('DOMContentLoaded', requestCrashes);
+/**
+ * Request crashes get uploaded in the background.
+ */
+function requestCrashUpload() {
+  // Don't need locking with this call because the system crash reporter
+  // has locking built into itself.
+  chrome.send('requestCrashUpload');
+
+  // Trigger a refresh in 5 seconds.  Clear any previous requests.
+  clearInterval(refreshCrashListId);

[xiyuan, 2014/03/25 21:30:58]

if (refreshCrashListId)
  clearTimeout(refreshCrashListId);

[vapier, 2014/03/25 21:35:46]

any particular reason for the if ?  when i tested things, it didn't seem to care
when you pass undefined.  i'd also point out that it ignores cases where the
timeout has already passed and so the id is no longer really valid.  didn't seem
worth it to handle these cases since the code behaves fine.

+  refreshCrashListId = setTimeout(requestCrashes, 5000);
+}
+
+document.addEventListener('DOMContentLoaded', function() {
+  $('uploadCrashes').onclick = requestCrashUpload;
+  requestCrashes();
+});