Show the time remaining of download items in the download manager....

chrome/browser/dom_ui/downloads_dom_handler.cc

 // Copyright (c) 2006-2009 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/dom_ui/downloads_dom_handler.h"
 
 #include "app/l10n_util.h"
 #include "base/basictypes.h"
 #include "base/gfx/png_encoder.h"
 #include "base/string_piece.h"
@@ skipping 314 matching lines @@
     amount.assign(received_size);
   }
   amount_units = GetByteDisplayUnits(download->CurrentSpeed());
   std::wstring speed_text = FormatSpeed(download->CurrentSpeed(),
                                         amount_units, true);
   std::wstring speed_text_localized;
   if (l10n_util::AdjustStringForLocaleDirection(speed_text,
                                                 &speed_text_localized))
     speed_text.assign(speed_text_localized);
 
-  return l10n_util::GetStringF(IDS_DOWNLOAD_TAB_PROGRESS_SPEED,
+
+  base::TimeDelta remaining;
+  std::wstring time_remaining;
+  if (download->is_paused())
+    time_remaining = l10n_util::GetString(IDS_DOWNLOAD_PROGRESS_PAUSED);
+  else if (download->TimeRemaining(&remaining))
+    time_remaining = TimeFormat::TimeRemaining(remaining);
+
+  if (time_remaining.empty())
+    return l10n_util::GetStringF(IDS_DOWNLOAD_TAB_PROGRESS_STATUS_TIME_UNKNOWN,

[Peter Kasting, 2009/05/26 23:18:30]

Nit: Now, on _this_ conditional body, since it's >1 line, you do need { } :)

(Random passing note: giving each arg its own line is fairly uncommon within our
code, we usually do as many args as will fit on a line)

+                                 speed_text,
+                                 amount);
+  return l10n_util::GetStringF(IDS_DOWNLOAD_TAB_PROGRESS_STATUS,
                                speed_text,
-                               amount);
+                               amount,
+                               time_remaining);
 }