base: Make TimeDurationFormat* report failures.

chrome/browser/ui/task_manager/task_manager_table_model.cc

Index: chrome/browser/ui/task_manager/task_manager_table_model.cc
diff --git a/chrome/browser/ui/task_manager/task_manager_table_model.cc b/chrome/browser/ui/task_manager/task_manager_table_model.cc
index 8123a35f5e43724522213cb284c583e6349e15a8..5c3c772b0666e24c225c892ecc8d25bbbeb49fa7 100644
--- a/chrome/browser/ui/task_manager/task_manager_table_model.cc
+++ b/chrome/browser/ui/task_manager/task_manager_table_model.cc
@@ -141,8 +141,13 @@ class TaskManagerValuesStringifier {
     if (cpu_time.is_zero())
       return n_a_string_;
 
-    return base::TimeDurationFormatWithSeconds(cpu_time,
-                                               base::DURATION_WIDTH_NARROW);
+    base::string16 str;

[Greg Levin, 2017/03/06 21:40:07]

nit: Maybe something more descriptive, like |duration| or |time_string|?

[Daniel Erat, 2017/03/06 23:19:34]

Done.

+    if (!base::TimeDurationFormatWithSeconds(
+            cpu_time, base::DURATION_WIDTH_NARROW, &str)) {
+      LOG(ERROR) << "Failed to format duration " << cpu_time.ToInternalValue();
+      return n_a_string_;
+    }
+    return str;
   }
 
   base::string16 GetMemoryUsageText(int64_t memory_usage, bool has_duplicates) {