Added a test checking for TimeFormat:: producing NaN, rolled icu version

chrome/common/time_format_browsertest.cc

Index: chrome/common/time_format_browsertest.cc
diff --git a/chrome/common/time_format_browsertest.cc b/chrome/common/time_format_browsertest.cc
new file mode 100644
index 0000000000000000000000000000000000000000..a4a5c45e135e1946fb6fe6418ee1c3e4a1a471a2
--- /dev/null
+++ b/chrome/common/time_format_browsertest.cc
@@ -0,0 +1,42 @@
+// Copyright (c) 2011 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.
+
+// This whole test runs as a separate browser_test because it depends on a
+// static initialization inside third_party/icu (gDecimal in digitlst.cpp).
+//
+// That initialization depends on the current locale, and on certain locales
+// will lead to wrong behavior. To make sure that the locale is set before
+// icu is used, and that the "wrong" static value doesn't affect other tests,
+// this test is executed on its own process.
+
+#include "base/string16.h"
+#include "base/test/test_util.h"
+#include "base/utf_string_conversions.h"
+#include "chrome/common/time_format.h"
+#include "chrome/test/in_process_browser_test.h"
+
+#if defined(OS_POSIX)
+using base::TimeDelta;
+
+class TimeFormatBrowserTest: public InProcessBrowserTest {
+ public:
+  TimeFormatBrowserTest(): scoped_locale_("fr_FR.utf-8") {}
+
+ private:
+  base::ScopedSetLocale scoped_locale_;
+};
+
+IN_PROC_BROWSER_TEST_F(TimeFormatBrowserTest, FAILS_DecimalPointNotDot) {

[jungshik at Google, 2011/06/07 21:48:01]

If you include ICU roll (DEPS change) in this CL, you can drop 'FAILS_' above. 

+  // Some locales use a comma ',' instead of a dot '.' as the separator for
+  // decimal digits. The icu library wasn't handling this, leading to "1"
+  // being internally converted to "+1,0e00" and ultimately leading to "NaN".
+  // This showed up on the browser on estimated download time, for example.
+  // http://crbug.com/60476
+
+  string16 one_min = TimeFormat::TimeRemainingShort(TimeDelta::FromMinutes(1));
+  EXPECT_EQ(ASCIIToUTF16("1 min"), one_min) << "fr_FR.utf-8 locale generates "
+                                            << one_min;
+}
+
+#endif // defined(OS_POSIX)