summaryrefslogtreecommitdiffstats
path: root/chrome/common/time_format_browsertest.cc
blob: 3aa85825cc911deaf645ea815a928bbd1fb5d1d5 (plain)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
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, DecimalPointNotDot) {
  // 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)