Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(171)

Unified Diff: components/crash/core/browser/resources/crashes.js

Issue 2581983002: Fix Incorrect date/time info under chrome://crashes. (Closed)
Patch Set: Patch in more changes suggested by phistuck@gmail.com. Created 4 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « components/crash/core/browser/crashes_ui_util.cc ('k') | components/crash_strings.grdp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/crash/core/browser/resources/crashes.js
diff --git a/components/crash/core/browser/resources/crashes.js b/components/crash/core/browser/resources/crashes.js
index 89a0bea508d545b866bf6cc69117b632d1543fa3..008ce24f94ebe76d49415bf8da2a98096b023c36 100644
--- a/components/crash/core/browser/resources/crashes.js
+++ b/components/crash/core/browser/resources/crashes.js
@@ -59,9 +59,14 @@ function updateCrashList(
crashBlock.appendChild(title);
if (uploaded) {
var date = document.createElement('p');
- date.textContent = loadTimeData.getStringF('crashTimeFormat',
- crash.time,
- crash.upload_time);
+ if (!crash.time) {
+ date.textContent = loadTimeData.getStringF('crashTimeFormat',
+ crash.upload_time);
+ } else {
+ date.textContent = loadTimeData.getStringF('crashAndUploadTimeFormat',
+ crash.time,
+ crash.upload_time);
Wez 2016/12/22 03:03:13 My suggestion was actually to have two *separate*
Jia 2016/12/22 04:16:55 Done.
+ }
crashBlock.appendChild(date);
var linkBlock = document.createElement('p');
var link = document.createElement('a');
« no previous file with comments | « components/crash/core/browser/crashes_ui_util.cc ('k') | components/crash_strings.grdp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698