[Autofill] Credit Card Autofill Last Used Date Experiment

base/i18n/time_formatting_unittest.cc

 // 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.
 
 #include "base/i18n/time_formatting.h"
 
 #include <memory>
 
 #include "base/i18n/rtl.h"
 #include "base/strings/utf_string_conversions.h"
@@ skipping 206 matching lines @@
             TimeFormatShortDateAndTime(time));
   EXPECT_EQ(ASCIIToUTF16("30/04/2011, 15:42:07 ") + GetShortTimeZone(time),
             TimeFormatShortDateAndTimeWithTimeZone(time));
   EXPECT_EQ(ASCIIToUTF16("April 2011"), TimeFormatMonthAndYear(time));
   EXPECT_EQ(ASCIIToUTF16("Saturday, 30 April 2011 at 15:42:07"),
             TimeFormatFriendlyDateAndTime(time));
   EXPECT_EQ(ASCIIToUTF16("Saturday, 30 April 2011"),
             TimeFormatFriendlyDate(time));
 }
 
+TEST(TimeFormattingTest, TimeFormatWithPattern) {
+  test::ScopedRestoreICUDefaultLocale restore_locale;
+
+  Time time;
+  EXPECT_TRUE(Time::FromLocalExploded(kTestDateTimeExploded, &time));
+
+  i18n::SetICUDefaultLocale("en_US");

[jungshik at Google, 2017/01/26 19:29:14]

Thank for adding tests. 

I'm sorry that I forgot to tell you about "base/test/scoped_locale.h". 

To avoid any potential misbehavior, we'd better limit the locale change to a
given scope. 

That means, different locale cases need to be in separate functions or en-US,
en-GB and ja-JP have to be enclosed by braces ( { en-US }, {en-GB test} , {ja-JP
test} ). 

[jiahuiguo, 2017/01/27 09:31:05]

Thanks for the comments, separate to different unittests. Do we still need to
disable the test on Android?  

+  EXPECT_EQ(ASCIIToUTF16("Apr 30, 2011"), TimeFormatWithPattern(time, "yMMMd"));
+  EXPECT_EQ(ASCIIToUTF16("April 30, 3:42:07 PM"),
+            TimeFormatWithPattern(time, "MMMMdjmmss"));
+
+  i18n::SetICUDefaultLocale("en_GB");
+  EXPECT_EQ(ASCIIToUTF16("30 Apr 2011"), TimeFormatWithPattern(time, "yMMMd"));
+  EXPECT_EQ(ASCIIToUTF16("30 April, 15:42:07"),
+            TimeFormatWithPattern(time, "MMMMdjmmss"));
+
+  i18n::SetICUDefaultLocale("ja_JP");
+  EXPECT_EQ(WideToUTF16(L"2011\x5e74"
+                        L"4\x6708"
+                        L"30\x65e5"),

[jungshik at Google, 2017/01/26 19:29:14]

Now that every compiler we use support UTF-16 string literals and MSVC (Chromium
requires) finally supports the source code in UTF-8 without BOM, you can just
use u"2011年4月30日".  (I know that a lot of code uses hex-escapes and
WideToUTF16). 

[jiahuiguo, 2017/01/27 09:31:05]

Actually if we use u"2011年4月30日" as the first param, it returns a const char16_t
string, while a string16 is expected here, we still need to have a convertion.
Can you provide an example or suggest how to deal with this?  In this new patch
I use EXPECT_EQ(WideToUTF16(L"2011年4月30日"), TimeFormatWithPattern(time,
"yMMMd")) directly, will this be acceptable since we need an extra conversion
anyway?

+            TimeFormatWithPattern(time, "yMMMd"));
+  EXPECT_EQ(WideToUTF16(L"4\x6708"
+                        L"30\x65e5") +
+                ASCIIToUTF16(" 15:42:07"),
+            TimeFormatWithPattern(time, "MMMMdjmmss"));
+}
+
 TEST(TimeFormattingTest, TimeDurationFormat) {
   test::ScopedRestoreICUDefaultLocale restore_locale;
   TimeDelta delta = TimeDelta::FromMinutes(15 * 60 + 42);
 
   // US English.
   i18n::SetICUDefaultLocale("en_US");
   EXPECT_EQ(ASCIIToUTF16("15 hours, 42 minutes"),
             TimeDurationFormat(delta, DURATION_WIDTH_WIDE));
   EXPECT_EQ(ASCIIToUTF16("15 hr, 42 min"),
             TimeDurationFormat(delta, DURATION_WIDTH_SHORT));
@@ skipping 94 matching lines @@
   Time end_time;
   EXPECT_TRUE(Time::FromLocalExploded(kTestIntervalEndTimeExploded, &end_time));
 
   EXPECT_EQ(
       WideToUTF16(L"Saturday, April 30 – Saturday, May 28"),
       DateIntervalFormat(begin_time, end_time, DATE_FORMAT_MONTH_WEEKDAY_DAY));
 }
 
 }  // namespace
 }  // namespace base