Commit b0931420 authored by Dominik Röttsches's avatar Dominik Röttsches Committed by Commit Bot

Remove initializer list compiler workarounds in segmentation tests

We don't need to compiler workarounds for initializing text runs
anymore.

Also, harmonize escaped notation: Replace \x notation with using full
length \U universal character names.

Cq-Include-Trybots: master.tryserver.chromium.linux:linux_layout_tests_layout_ng
Change-Id: I020cb509bf3a89eebbf9629da45a26061e4ac04c
Reviewed-on: https://chromium-review.googlesource.com/1091461
Commit-Queue: Dominik Röttsches <drott@chromium.org>
Commit-Queue: Emil A Eklund <eae@chromium.org>
Reviewed-by: default avatarEmil A Eklund <eae@chromium.org>
Cr-Commit-Position: refs/heads/master@{#565384}
parent 76257d1a
...@@ -57,6 +57,14 @@ class RunSegmenterTest : public testing::Test { ...@@ -57,6 +57,14 @@ class RunSegmenterTest : public testing::Test {
VerifyRuns(&run_segmenter, expect); VerifyRuns(&run_segmenter, expect);
} }
void CheckRunsMixed(const Vector<SegmenterTestRun>& runs) {
CheckRuns(runs, FontOrientation::kVerticalMixed);
}
void CheckRunsHorizontal(const Vector<SegmenterTestRun>& runs) {
CheckRuns(runs, FontOrientation::kHorizontal);
}
void VerifyRuns(RunSegmenter* run_segmenter, void VerifyRuns(RunSegmenter* run_segmenter,
const Vector<SegmenterExpectedRun>& expect) { const Vector<SegmenterExpectedRun>& expect) {
RunSegmenter::RunSegmenterRange segmenter_range; RunSegmenter::RunSegmenterRange segmenter_range;
...@@ -78,20 +86,6 @@ class RunSegmenterTest : public testing::Test { ...@@ -78,20 +86,6 @@ class RunSegmenterTest : public testing::Test {
} }
}; };
// Some of our compilers cannot initialize a vector from an array yet.
#define DECLARE_SEGMENTER_RUNSVECTOR(...) \
static const SegmenterTestRun kRunsArray[] = __VA_ARGS__; \
Vector<SegmenterTestRun> runs; \
runs.Append(kRunsArray, sizeof(kRunsArray) / sizeof(*kRunsArray));
#define CHECK_RUNS_MIXED(...) \
DECLARE_SEGMENTER_RUNSVECTOR(__VA_ARGS__); \
CheckRuns(runs, FontOrientation::kVerticalMixed);
#define CHECK_RUNS_HORIZONTAL(...) \
DECLARE_SEGMENTER_RUNSVECTOR(__VA_ARGS__); \
CheckRuns(runs, FontOrientation::kHorizontal);
TEST_F(RunSegmenterTest, Empty) { TEST_F(RunSegmenterTest, Empty) {
String empty(g_empty_string16_bit); String empty(g_empty_string16_bit);
RunSegmenter::RunSegmenterRange segmenter_range = { RunSegmenter::RunSegmenterRange segmenter_range = {
...@@ -109,19 +103,19 @@ TEST_F(RunSegmenterTest, Empty) { ...@@ -109,19 +103,19 @@ TEST_F(RunSegmenterTest, Empty) {
} }
TEST_F(RunSegmenterTest, LatinPunctuationSideways) { TEST_F(RunSegmenterTest, LatinPunctuationSideways) {
CHECK_RUNS_MIXED({{"Abc.;?Xyz", USCRIPT_LATIN, CheckRunsMixed({{"Abc.;?Xyz", USCRIPT_LATIN,
OrientationIterator::kOrientationRotateSideways, OrientationIterator::kOrientationRotateSideways,
FontFallbackPriority::kText}}); FontFallbackPriority::kText}});
} }
TEST_F(RunSegmenterTest, OneSpace) { TEST_F(RunSegmenterTest, OneSpace) {
CHECK_RUNS_MIXED( CheckRunsMixed(
{{" ", USCRIPT_COMMON, OrientationIterator::kOrientationRotateSideways, {{" ", USCRIPT_COMMON, OrientationIterator::kOrientationRotateSideways,
FontFallbackPriority::kText}}); FontFallbackPriority::kText}});
} }
TEST_F(RunSegmenterTest, ArabicHangul) { TEST_F(RunSegmenterTest, ArabicHangul) {
CHECK_RUNS_MIXED( CheckRunsMixed(
{{"نص", USCRIPT_ARABIC, OrientationIterator::kOrientationRotateSideways, {{"نص", USCRIPT_ARABIC, OrientationIterator::kOrientationRotateSideways,
FontFallbackPriority::kText}, FontFallbackPriority::kText},
{"키스의", USCRIPT_HANGUL, OrientationIterator::kOrientationKeep, {"키스의", USCRIPT_HANGUL, OrientationIterator::kOrientationKeep,
...@@ -129,7 +123,7 @@ TEST_F(RunSegmenterTest, ArabicHangul) { ...@@ -129,7 +123,7 @@ TEST_F(RunSegmenterTest, ArabicHangul) {
} }
TEST_F(RunSegmenterTest, JapaneseHindiEmojiMix) { TEST_F(RunSegmenterTest, JapaneseHindiEmojiMix) {
CHECK_RUNS_MIXED( CheckRunsMixed(
{{"百家姓", USCRIPT_HAN, OrientationIterator::kOrientationKeep, {{"百家姓", USCRIPT_HAN, OrientationIterator::kOrientationKeep,
FontFallbackPriority::kText}, FontFallbackPriority::kText},
{"ऋषियों", USCRIPT_DEVANAGARI, {"ऋषियों", USCRIPT_DEVANAGARI,
...@@ -144,13 +138,13 @@ TEST_F(RunSegmenterTest, JapaneseHindiEmojiMix) { ...@@ -144,13 +138,13 @@ TEST_F(RunSegmenterTest, JapaneseHindiEmojiMix) {
} }
TEST_F(RunSegmenterTest, CombiningCirlce) { TEST_F(RunSegmenterTest, CombiningCirlce) {
CHECK_RUNS_HORIZONTAL( CheckRunsHorizontal(
{{"◌́◌̀◌̈◌̂◌̄◌̊", USCRIPT_COMMON, OrientationIterator::kOrientationKeep, {{"◌́◌̀◌̈◌̂◌̄◌̊", USCRIPT_COMMON, OrientationIterator::kOrientationKeep,
FontFallbackPriority::kText}}); FontFallbackPriority::kText}});
} }
TEST_F(RunSegmenterTest, HangulSpace) { TEST_F(RunSegmenterTest, HangulSpace) {
CHECK_RUNS_MIXED( CheckRunsMixed(
{{"키스의", USCRIPT_HANGUL, OrientationIterator::kOrientationKeep, {{"키스의", USCRIPT_HANGUL, OrientationIterator::kOrientationKeep,
FontFallbackPriority::kText}, FontFallbackPriority::kText},
{" ", USCRIPT_HANGUL, OrientationIterator::kOrientationRotateSideways, {" ", USCRIPT_HANGUL, OrientationIterator::kOrientationRotateSideways,
...@@ -160,19 +154,18 @@ TEST_F(RunSegmenterTest, HangulSpace) { ...@@ -160,19 +154,18 @@ TEST_F(RunSegmenterTest, HangulSpace) {
} }
TEST_F(RunSegmenterTest, TechnicalCommonUpright) { TEST_F(RunSegmenterTest, TechnicalCommonUpright) {
CHECK_RUNS_MIXED( CheckRunsMixed({{"⌀⌁⌂", USCRIPT_COMMON, OrientationIterator::kOrientationKeep,
{{"⌀⌁⌂", USCRIPT_COMMON, OrientationIterator::kOrientationKeep, FontFallbackPriority::kText}});
FontFallbackPriority::kText}});
} }
TEST_F(RunSegmenterTest, PunctuationCommonSideways) { TEST_F(RunSegmenterTest, PunctuationCommonSideways) {
CHECK_RUNS_MIXED( CheckRunsMixed(
{{".…¡", USCRIPT_COMMON, OrientationIterator::kOrientationRotateSideways, {{".…¡", USCRIPT_COMMON, OrientationIterator::kOrientationRotateSideways,
FontFallbackPriority::kText}}); FontFallbackPriority::kText}});
} }
TEST_F(RunSegmenterTest, JapanesePunctuationMixedInside) { TEST_F(RunSegmenterTest, JapanesePunctuationMixedInside) {
CHECK_RUNS_MIXED( CheckRunsMixed(
{{"いろはに", USCRIPT_HIRAGANA, OrientationIterator::kOrientationKeep, {{"いろはに", USCRIPT_HIRAGANA, OrientationIterator::kOrientationKeep,
FontFallbackPriority::kText}, FontFallbackPriority::kText},
{".…¡", USCRIPT_HIRAGANA, {".…¡", USCRIPT_HIRAGANA,
...@@ -183,19 +176,19 @@ TEST_F(RunSegmenterTest, JapanesePunctuationMixedInside) { ...@@ -183,19 +176,19 @@ TEST_F(RunSegmenterTest, JapanesePunctuationMixedInside) {
} }
TEST_F(RunSegmenterTest, JapanesePunctuationMixedInsideHorizontal) { TEST_F(RunSegmenterTest, JapanesePunctuationMixedInsideHorizontal) {
CHECK_RUNS_HORIZONTAL( CheckRunsHorizontal(
{{"いろはに.…¡ほへと", USCRIPT_HIRAGANA, {{"いろはに.…¡ほへと", USCRIPT_HIRAGANA,
OrientationIterator::kOrientationKeep, FontFallbackPriority::kText}}); OrientationIterator::kOrientationKeep, FontFallbackPriority::kText}});
} }
TEST_F(RunSegmenterTest, PunctuationDevanagariCombining) { TEST_F(RunSegmenterTest, PunctuationDevanagariCombining) {
CHECK_RUNS_HORIZONTAL( CheckRunsHorizontal(
{{"क+े", USCRIPT_DEVANAGARI, OrientationIterator::kOrientationKeep, {{"क+े", USCRIPT_DEVANAGARI, OrientationIterator::kOrientationKeep,
FontFallbackPriority::kText}}); FontFallbackPriority::kText}});
} }
TEST_F(RunSegmenterTest, EmojiZWJSequences) { TEST_F(RunSegmenterTest, EmojiZWJSequences) {
CHECK_RUNS_HORIZONTAL( CheckRunsHorizontal(
{{"👩‍👩‍👧‍👦👩‍❤️‍💋‍👨", USCRIPT_LATIN, {{"👩‍👩‍👧‍👦👩‍❤️‍💋‍👨", USCRIPT_LATIN,
OrientationIterator::kOrientationKeep, OrientationIterator::kOrientationKeep,
FontFallbackPriority::kEmojiEmoji}, FontFallbackPriority::kEmojiEmoji},
...@@ -208,7 +201,7 @@ TEST_F(RunSegmenterTest, EmojiZWJSequences) { ...@@ -208,7 +201,7 @@ TEST_F(RunSegmenterTest, EmojiZWJSequences) {
} }
TEST_F(RunSegmenterTest, JapaneseLetterlikeEnd) { TEST_F(RunSegmenterTest, JapaneseLetterlikeEnd) {
CHECK_RUNS_MIXED( CheckRunsMixed(
{{"いろは", USCRIPT_HIRAGANA, OrientationIterator::kOrientationKeep, {{"いろは", USCRIPT_HIRAGANA, OrientationIterator::kOrientationKeep,
FontFallbackPriority::kText}, FontFallbackPriority::kText},
{"ℐℒℐℒℐℒℐℒℐℒℐℒℐℒ", USCRIPT_HIRAGANA, {"ℐℒℐℒℐℒℐℒℐℒℐℒℐℒ", USCRIPT_HIRAGANA,
...@@ -217,7 +210,7 @@ TEST_F(RunSegmenterTest, JapaneseLetterlikeEnd) { ...@@ -217,7 +210,7 @@ TEST_F(RunSegmenterTest, JapaneseLetterlikeEnd) {
} }
TEST_F(RunSegmenterTest, JapaneseCase) { TEST_F(RunSegmenterTest, JapaneseCase) {
CHECK_RUNS_MIXED( CheckRunsMixed(
{{"いろは", USCRIPT_HIRAGANA, OrientationIterator::kOrientationKeep, {{"いろは", USCRIPT_HIRAGANA, OrientationIterator::kOrientationKeep,
FontFallbackPriority::kText}, FontFallbackPriority::kText},
{"aaAA", USCRIPT_LATIN, OrientationIterator::kOrientationRotateSideways, {"aaAA", USCRIPT_LATIN, OrientationIterator::kOrientationRotateSideways,
...@@ -227,13 +220,13 @@ TEST_F(RunSegmenterTest, JapaneseCase) { ...@@ -227,13 +220,13 @@ TEST_F(RunSegmenterTest, JapaneseCase) {
} }
TEST_F(RunSegmenterTest, DingbatsMiscSymbolsModifier) { TEST_F(RunSegmenterTest, DingbatsMiscSymbolsModifier) {
CHECK_RUNS_HORIZONTAL({{"⛹🏻✍🏻✊🏼", USCRIPT_COMMON, CheckRunsHorizontal({{"⛹🏻✍🏻✊🏼", USCRIPT_COMMON,
OrientationIterator::kOrientationKeep, OrientationIterator::kOrientationKeep,
FontFallbackPriority::kEmojiEmoji}}); FontFallbackPriority::kEmojiEmoji}});
} }
TEST_F(RunSegmenterTest, ArmenianCyrillicCase) { TEST_F(RunSegmenterTest, ArmenianCyrillicCase) {
CHECK_RUNS_HORIZONTAL( CheckRunsHorizontal(
{{"աբգ", USCRIPT_ARMENIAN, OrientationIterator::kOrientationKeep, {{"աբգ", USCRIPT_ARMENIAN, OrientationIterator::kOrientationKeep,
FontFallbackPriority::kText}, FontFallbackPriority::kText},
{"αβγ", USCRIPT_GREEK, OrientationIterator::kOrientationKeep, {"αβγ", USCRIPT_GREEK, OrientationIterator::kOrientationKeep,
...@@ -243,7 +236,7 @@ TEST_F(RunSegmenterTest, ArmenianCyrillicCase) { ...@@ -243,7 +236,7 @@ TEST_F(RunSegmenterTest, ArmenianCyrillicCase) {
} }
TEST_F(RunSegmenterTest, EmojiSubdivisionFlags) { TEST_F(RunSegmenterTest, EmojiSubdivisionFlags) {
CHECK_RUNS_HORIZONTAL( CheckRunsHorizontal(
{{"🏴󠁧󠁢󠁷󠁬󠁳󠁿🏴󠁧󠁢󠁳󠁣󠁴󠁿🏴󠁧󠁢" {{"🏴󠁧󠁢󠁷󠁬󠁳󠁿🏴󠁧󠁢󠁳󠁣󠁴󠁿🏴󠁧󠁢"
"󠁥󠁮󠁧󠁿", "󠁥󠁮󠁧󠁿",
USCRIPT_COMMON, OrientationIterator::kOrientationKeep, USCRIPT_COMMON, OrientationIterator::kOrientationKeep,
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment