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

Side by Side Diff: third_party/WebKit/Source/platform/fonts/AcceptLanguagesResolverTest.cpp

Issue 1693563003: Take Accept-Language into account in CJK font fallback for Android/Win (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@aceept-lang
Patch Set: Fix non-Windows builds Created 4 years, 10 months 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 unified diff | Download patch
OLDNEW
(Empty)
1 // Copyright 2016 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file.
4
5 #include "platform/fonts/AcceptLanguagesResolver.h"
6
7 #include "testing/gtest/include/gtest/gtest.h"
8
9 namespace blink {
10
11 TEST(AcceptLanguagesResolverTest, AcceptLanguagesChanged)
12 {
13 struct {
14 const char* acceptLanguages;
15 UScriptCode script;
16 const char* locale;
17 } tests[] = {
18 // Non-Han script cases.
19 { "", USCRIPT_COMMON, nullptr },
20 { "en-US", USCRIPT_COMMON, nullptr },
21 { ",en-US", USCRIPT_COMMON, nullptr },
22
23 // Single value cases.
24 { "ja-JP", USCRIPT_KATAKANA_OR_HIRAGANA, "ja-jp" },
25 { "ko-KR", USCRIPT_HANGUL, "ko-kr" },
26 { "zh-CN", USCRIPT_SIMPLIFIED_HAN, "zh-Hans" },
27 { "zh-HK", USCRIPT_TRADITIONAL_HAN, "zh-Hant" },
28 { "zh-TW", USCRIPT_TRADITIONAL_HAN, "zh-Hant" },
29
30 // Unusual combinations.
31 { "en-JP", USCRIPT_KATAKANA_OR_HIRAGANA, "ja-jp" },
32
33 // Han scripts not in the first item.
34 { "en-US,ja-JP", USCRIPT_KATAKANA_OR_HIRAGANA, "ja-jp" },
35 { "en-US,en-JP", USCRIPT_KATAKANA_OR_HIRAGANA, "ja-jp" },
36
37 // Multiple Han scripts. The first one wins.
38 { "ja-JP,zh-CN", USCRIPT_KATAKANA_OR_HIRAGANA, "ja-jp" },
39 { "zh-TW,ja-JP", USCRIPT_TRADITIONAL_HAN, "zh-Hant" },
40 };
41
42 for (auto& test : tests) {
43 AcceptLanguagesResolver::updateFromAcceptLanguages(test.acceptLanguages) ;
44
45 EXPECT_EQ(test.script, AcceptLanguagesResolver::preferredHanScript())
46 << test.acceptLanguages;
47 EXPECT_STREQ(test.locale,
48 AcceptLanguagesResolver::preferredHanSkFontMgrLocale())
49 << test.acceptLanguages;
50 }
51 }
52
53 } // namespace blink
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698