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

Unified Diff: Source/web/tests/LocaleMacTest.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/LocaleMacTest.cpp
diff --git a/Source/web/tests/LocaleMacTest.cpp b/Source/web/tests/LocaleMacTest.cpp
index 0d1426cc205c94d9e49b04cd3dc0b9bad979c023..eda1df219d8ac23fa840183b38044e82cdec3d5f 100644
--- a/Source/web/tests/LocaleMacTest.cpp
+++ b/Source/web/tests/LocaleMacTest.cpp
@@ -99,7 +99,6 @@ protected:
return locale->formatDateTime(timeComponents(hour, minute, second, millisecond), (useShortFormat ? Locale::FormatTypeShort : Locale::FormatTypeMedium));
}
-#if ENABLE(CALENDAR_PICKER)
unsigned firstDayOfWeek(const String& localeString)
{
OwnPtr<LocaleMac> locale = LocaleMac::create(localeString);
@@ -123,7 +122,6 @@ protected:
OwnPtr<LocaleMac> locale = LocaleMac::create(localeString);
return locale->isRTL();
}
-#endif
#if ENABLE(INPUT_MULTIPLE_FIELDS_UI)
String monthFormat(const String& localeString)
@@ -221,7 +219,6 @@ TEST_F(LocaleMacTest, formatTime)
EXPECT_STREQ("\xDB\xB7:\xDB\xB0\xDB\xB7:\xDB\xB0\xDB\xB7\xD9\xAB\xDB\xB0\xDB\xB0\xDB\xB7", formatTime("fa", 07, 07, 07, 007, false).utf8().data());
}
-#if ENABLE(CALENDAR_PICKER)
TEST_F(LocaleMacTest, firstDayOfWeek)
{
EXPECT_EQ(Sunday, firstDayOfWeek("en_US"));
@@ -266,7 +263,6 @@ TEST_F(LocaleMacTest, isRTL)
EXPECT_FALSE(isRTL("ja-jp"));
EXPECT_FALSE(isRTL("**invalid**"));
}
-#endif
#if ENABLE(INPUT_MULTIPLE_FIELDS_UI)
TEST_F(LocaleMacTest, monthFormat)

Powered by Google App Engine
This is Rietveld 408576698