base: Make TimeDurationFormat* report failures.

chrome/browser/ui/task_manager/task_manager_table_model.cc

 // Copyright 2015 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.
 
 #include "chrome/browser/ui/task_manager/task_manager_table_model.h"
 
 #include <stddef.h>
 
 #include "base/command_line.h"
 #include "base/i18n/number_formatting.h"
@@ skipping 123 matching lines @@
     if (start_time.is_null())
       return n_a_string_;
 
     return base::TimeFormatShortDateAndTime(start_time);
   }
 
   base::string16 GetCpuTimeText(base::TimeDelta cpu_time) {
     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) {
     if (memory_usage == -1)
       return n_a_string_;
 
 #if defined(OS_MACOSX)
     // System expectation is to show "100 kB", "200 MB", etc.
     // TODO(thakis): [This TODO has been taken as is from the old task manager]:
     // Switch to metric units (as opposed to powers of two).
@@ skipping 773 matching lines @@
 
 bool TaskManagerTableModel::IsTaskFirstInGroup(int row_index) const {
   if (row_index == 0)
     return true;
 
   return observed_task_manager()->GetProcessId(tasks_[row_index - 1]) !=
       observed_task_manager()->GetProcessId(tasks_[row_index]);
 }
 
 }  // namespace task_manager