2017-01-29 00:17:24 +00:00
|
|
|
From 424584b4984bde7c831f42e9fb47f1ad583a1c46 Mon Sep 17 00:00:00 2001
|
|
|
|
From: jshin <jshin@chromium.org>
|
|
|
|
Date: Fri, 21 Oct 2016 01:15:50 -0700
|
|
|
|
Subject: [PATCH] Update aspirational_scripts per Unicode 9
|
|
|
|
|
|
|
|
This is a preparation to update ICU to 58.1.
|
|
|
|
Without this change, ICU update CL cannot be tested on
|
|
|
|
trybots because a compile will fail due to '#error' checking
|
|
|
|
if ICU version is less than 58.
|
|
|
|
|
|
|
|
BUG=637001
|
|
|
|
TEST=None
|
|
|
|
TBR=pkasting@chromium.org
|
|
|
|
|
|
|
|
Review-Url: https://chromiumcodereview.appspot.com/2436113003
|
|
|
|
Cr-Commit-Position: refs/heads/master@{#426749}
|
|
|
|
---
|
|
|
|
components/url_formatter/url_formatter.cc | 10 +++++-----
|
|
|
|
1 file changed, 5 insertions(+), 5 deletions(-)
|
|
|
|
|
|
|
|
diff --git a/components/url_formatter/url_formatter.cc b/components/url_formatter/url_formatter.cc
|
|
|
|
index 4455db3..2b82c0c 100644
|
|
|
|
--- qtwebengine-opensource-src-5.7.0.orig/src/3rdparty/chromium/components/url_formatter/url_formatter.cc
|
|
|
|
+++ qtwebengine-opensource-src-5.7.0/src/3rdparty/chromium/components/url_formatter/url_formatter.cc
|
|
|
|
@@ -428,9 +428,9 @@ void IDNSpoofChecker::SetAllowedUnicodeSet(UErrorCode* status) {
|
|
|
|
// section at
|
|
|
|
// http://www.unicode.org/Public/security/latest/xidmodifications.txt) are
|
|
|
|
// are added to the allowed set. The list has to be updated when a new
|
|
|
|
- // version of Unicode is released. The current version is 8.0.0 and ICU 58
|
|
|
|
- // will have Unicode 9.0 data.
|
|
|
|
-#if U_ICU_VERSION_MAJOR_NUM < 58
|
|
|
|
+ // version of Unicode is released. The current version is 9.0.0 and ICU 60
|
|
|
|
+ // will have Unicode 10.0 data.
|
|
|
|
+#if U_ICU_VERSION_MAJOR_NUM < 60
|
|
|
|
const icu::UnicodeSet aspirational_scripts(
|
|
|
|
icu::UnicodeString(
|
|
|
|
// Unified Canadian Syllabics
|
|
|
|
@@ -444,13 +444,13 @@ void IDNSpoofChecker::SetAllowedUnicodeSet(UErrorCode* status) {
|
|
|
|
// Yi
|
|
|
|
"\\uA000-\\uA48C"
|
|
|
|
// Miao
|
|
|
|
- "\\U00016F00-\\U00016F44\\U00016F50-\\U00016F7F"
|
|
|
|
+ "\\U00016F00-\\U00016F44\\U00016F50-\\U00016F7E"
|
|
|
|
"\\U00016F8F-\\U00016F9F]",
|
|
|
|
-1, US_INV),
|
|
|
|
*status);
|
|
|
|
allowed_set.addAll(aspirational_scripts);
|
|
|
|
#else
|
|
|
|
-#error "Update aspirational_scripts per Unicode 9.0"
|
|
|
|
+#error "Update aspirational_scripts per Unicode 10.0"
|
|
|
|
#endif
|
|
|
|
|
|
|
|
// U+0338 is included in the recommended set, while U+05F4 and U+2027 are in
|
|
|
|
diff -Naur qtwebengine-opensource-src-5.7.0.orig/src/3rdparty/chromium/third_party/WebKit/Source/platform/text/TextBreakIterator.cpp
|
|
|
|
qtwebengine-opensource-src-5.7.0/src/3rdparty/chromium/third_party/WebKit/Source/platform/text/TextBreakIterator.cpp
|
2016-12-04 19:26:56 +00:00
|
|
|
--- qtwebengine-opensource-src-5.7.0.orig/src/3rdparty/chromium/third_party/WebKit/Source/platform/text/TextBreakIterator.cpp 2016-11-15 17:07:38.680665385 -0800
|
|
|
|
+++ qtwebengine-opensource-src-5.7.0/src/3rdparty/chromium/third_party/WebKit/Source/platform/text/TextBreakIterator.cpp 2016-11-15 17:12:17.564001354 -0800
|
|
|
|
@@ -26,6 +26,9 @@
|
|
|
|
#include "wtf/StdLibExtras.h"
|
|
|
|
#include "wtf/text/CharacterNames.h"
|
|
|
|
|
|
|
|
+#include <unicode/uchar.h>
|
|
|
|
+#include <unicode/uvernum.h>
|
|
|
|
+
|
|
|
|
namespace blink {
|
|
|
|
|
|
|
|
unsigned numGraphemeClusters(const String& string)
|
|
|
|
@@ -122,13 +125,18 @@
|
|
|
|
{ B(0, 0, 0, 0, 0, 0, 0, 0), B(0, 0, 0, 0, 0, 0, 0, 0), 0, B(0, 0, 0, 0, 0, 0, 0, 0), 0, 0, 0, B(0, 0, 0, 0, 0, 0, 0, 0), 0, 0, 0, B(0, 0, 0, 0, 0, 0, 0, 0) }, // DEL
|
|
|
|
};
|
|
|
|
|
|
|
|
+#if U_ICU_VERSION_MAJOR_NUM >= 58
|
|
|
|
+#define BA_LB_COUNT (U_LB_COUNT - 3)
|
|
|
|
+#else
|
|
|
|
+#define BA_LB_COUNT U_LB_COUNT
|
|
|
|
+#endif
|
|
|
|
// Line breaking table for CSS word-break: break-all. This table differs from
|
|
|
|
// asciiLineBreakTable in:
|
|
|
|
// - Indices are Line Breaking Classes defined in UAX#14 Unicode Line Breaking
|
|
|
|
// Algorithm: http://unicode.org/reports/tr14/#DescriptionOfProperties
|
|
|
|
// - 1 indicates additional break opportunities. 0 indicates to fallback to
|
|
|
|
// normal line break, not "prohibit break."
|
|
|
|
-static const unsigned char breakAllLineBreakClassTable[][U_LB_COUNT / 8 + 1] = {
|
|
|
|
+static const unsigned char breakAllLineBreakClassTable[][BA_LB_COUNT / 8 + 1] = {
|
|
|
|
// XX AI AL B2 BA BB BK CB CL CM CR EX GL HY ID IN IS LF NS NU OP PO PR QU SA SG SP SY ZW NL WJ H2 H3 JL JT JV CP CJ HL RI
|
|
|
|
{ B(0, 0, 0, 0, 0, 0, 0, 0), B(0, 0, 0, 0, 0, 0, 0, 0), B(0, 0, 0, 0, 0, 0, 0, 0), B(0, 0, 0, 0, 0, 0, 0, 0), B(0, 0, 0, 0, 0, 0, 0, 0) }, // XX
|
|
|
|
{ B(0, 1, 1, 0, 1, 0, 0, 0), B(0, 0, 0, 0, 0, 1, 0, 0), B(0, 0, 0, 1, 1, 0, 1, 0), B(1, 0, 0, 0, 0, 0, 0, 0), B(0, 0, 0, 0, 0, 0, 1, 0) }, // AI
|
|
|
|
@@ -178,7 +186,7 @@
|
|
|
|
#undef AL
|
|
|
|
|
|
|
|
static_assert(WTF_ARRAY_LENGTH(asciiLineBreakTable) == asciiLineBreakTableLastChar - asciiLineBreakTableFirstChar + 1, "asciiLineBreakTable should be consistent");
|
|
|
|
-static_assert(WTF_ARRAY_LENGTH(breakAllLineBreakClassTable) == U_LB_COUNT, "breakAllLineBreakClassTable should be consistent");
|
|
|
|
+static_assert(WTF_ARRAY_LENGTH(breakAllLineBreakClassTable) == BA_LB_COUNT, "breakAllLineBreakClassTable should be consistent");
|
|
|
|
|
|
|
|
static inline bool shouldBreakAfter(UChar lastCh, UChar ch, UChar nextCh)
|
|
|
|
{
|
|
|
|
@@ -209,7 +217,7 @@
|
|
|
|
|
|
|
|
static inline bool shouldBreakAfterBreakAll(ULineBreak lastLineBreak, ULineBreak lineBreak)
|
|
|
|
{
|
|
|
|
- if (lineBreak >= 0 && lineBreak < U_LB_COUNT && lastLineBreak >= 0 && lastLineBreak < U_LB_COUNT) {
|
|
|
|
+ if (lineBreak >= 0 && lineBreak < BA_LB_COUNT && lastLineBreak >= 0 && lastLineBreak < BA_LB_COUNT) {
|
|
|
|
const unsigned char* tableRow = breakAllLineBreakClassTable[lastLineBreak];
|
|
|
|
return tableRow[lineBreak / 8] & (1 << (lineBreak % 8));
|
|
|
|
}
|