Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1464)

Unified Diff: Source/web/tests/LocaleWinTest.cpp

Issue 52553008: Eliminate the CALENDAR_PICKER compile time flag as only android uses this flag and the web facing p… (Closed) Base URL: https://chromium.googlesource.com/chromium/blink@master
Patch Set: Fix winbot by eliminating last checks for CALENDAR_PICKER Created 7 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: Source/web/tests/LocaleWinTest.cpp
diff --git a/Source/web/tests/LocaleWinTest.cpp b/Source/web/tests/LocaleWinTest.cpp
index c71e2df0b332f9fd2d676cc9dee7203b9b84a6e1..2e1b7eb6f07fae641c67457dbbfd344cec145556 100644
--- a/Source/web/tests/LocaleWinTest.cpp
+++ b/Source/web/tests/LocaleWinTest.cpp
@@ -90,7 +90,6 @@ protected:
return locale->formatDateTime(dateComponents(year, month, day));
}
-#if ENABLE(CALENDAR_PICKER)
unsigned firstDayOfWeek(LCID lcid)
{
OwnPtr<LocaleWin> locale = LocaleWin::create(lcid, true /* defaultsForLocale */);
@@ -114,7 +113,6 @@ protected:
OwnPtr<LocaleWin> locale = LocaleWin::create(lcid, true /* defaultsForLocale */);
return locale->isRTL();
}
-#endif
#if ENABLE(INPUT_MULTIPLE_FIELDS_UI)
String monthFormat(LCID lcid)
@@ -162,7 +160,6 @@ TEST_F(LocaleWinTest, formatDate)
EXPECT_STREQ("2005/04/27", formatDate(JapaneseJP, 2005, April, 27).utf8().data());
}
-#if ENABLE(CALENDAR_PICKER)
TEST_F(LocaleWinTest, firstDayOfWeek)
{
EXPECT_EQ(Sunday, firstDayOfWeek(EnglishUS));
@@ -206,8 +203,6 @@ TEST_F(LocaleWinTest, isRTL)
EXPECT_FALSE(isRTL(EnglishUS));
}
-#endif
-
#if ENABLE(INPUT_MULTIPLE_FIELDS_UI)
TEST_F(LocaleWinTest, dateFormat)
{
« no previous file with comments | « Source/web/tests/LocaleMacTest.cpp ('k') | Source/web/web.gyp » ('j') | Source/web/web.gyp » ('J')

Powered by Google App Engine
This is Rietveld 408576698