Unreviewed, rolling out r202057.
authorryanhaddad@apple.com <ryanhaddad@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 14 Jun 2016 20:15:47 +0000 (20:15 +0000)
committerryanhaddad@apple.com <ryanhaddad@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 14 Jun 2016 20:15:47 +0000 (20:15 +0000)
https://bugs.webkit.org/show_bug.cgi?id=158749

This change broke the Windows build. (Requested by ryanhaddad
on #webkit).

Reverted changeset:

"Honor bidi unicode codepoints"
https://bugs.webkit.org/show_bug.cgi?id=149170
http://trac.webkit.org/changeset/202057

Patch by Commit Queue <commit-queue@webkit.org> on 2016-06-14

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@202059 268f45cc-cd09-0410-ab3c-d52691b4dbfc

LayoutTests/ChangeLog
LayoutTests/fast/text/isolate-ignore-expected.html [deleted file]
LayoutTests/fast/text/isolate-ignore.html [deleted file]
Source/WTF/ChangeLog
Source/WTF/wtf/unicode/CharacterNames.h
Source/WebCore/ChangeLog
Source/WebCore/platform/graphics/Font.cpp
Source/WebCore/platform/text/BidiResolver.h

index 0380f77..d2a7f47 100644 (file)
@@ -1,3 +1,17 @@
+2016-06-14  Commit Queue  <commit-queue@webkit.org>
+
+        Unreviewed, rolling out r202057.
+        https://bugs.webkit.org/show_bug.cgi?id=158749
+
+        This change broke the Windows build. (Requested by ryanhaddad
+        on #webkit).
+
+        Reverted changeset:
+
+        "Honor bidi unicode codepoints"
+        https://bugs.webkit.org/show_bug.cgi?id=149170
+        http://trac.webkit.org/changeset/202057
+
 2016-06-14  Myles C. Maxfield  <mmaxfield@apple.com>
 
         Honor bidi unicode codepoints
diff --git a/LayoutTests/fast/text/isolate-ignore-expected.html b/LayoutTests/fast/text/isolate-ignore-expected.html
deleted file mode 100644 (file)
index 6a02113..0000000
+++ /dev/null
@@ -1,18 +0,0 @@
-<!DOCTYPE html>
-<html>
-<body>
-This test makes sure that isolate codepoints are ignored.
-<div dir="ltr">&#x05e2;&#x05d1;&#x05e8;&#x05d9;&#x05ea; English &#x05e2;&#x05d1;&#x05e8;&#x05d9;&#x05ea;</div>
-<div dir="ltr">English &#x05e2;&#x05d1;&#x05e8;&#x05d9;&#x05ea; English</div>
-<div dir="rtl">&#x05e2;&#x05d1;&#x05e8;&#x05d9;&#x05ea; English &#x05e2;&#x05d1;&#x05e8;&#x05d9;&#x05ea;</div>
-<div dir="rtl">English &#x05e2;&#x05d1;&#x05e8;&#x05d9;&#x05ea; English</div>
-<div dir="ltr">&#x05e2;&#x05d1;&#x05e8;&#x05d9;&#x05ea; English &#x05e2;&#x05d1;&#x05e8;&#x05d9;&#x05ea;</div>
-<div dir="ltr">English &#x05e2;&#x05d1;&#x05e8;&#x05d9;&#x05ea; English</div>
-<div dir="rtl">&#x05e2;&#x05d1;&#x05e8;&#x05d9;&#x05ea; English &#x05e2;&#x05d1;&#x05e8;&#x05d9;&#x05ea;</div>
-<div dir="rtl">English &#x05e2;&#x05d1;&#x05e8;&#x05d9;&#x05ea; English</div>
-<div dir="ltr">&#x05e2;&#x05d1;&#x05e8;&#x05d9;&#x05ea; English &#x05e2;&#x05d1;&#x05e8;&#x05d9;&#x05ea;</div>
-<div dir="ltr">English &#x05e2;&#x05d1;&#x05e8;&#x05d9;&#x05ea; English</div>
-<div dir="rtl">&#x05e2;&#x05d1;&#x05e8;&#x05d9;&#x05ea; English &#x05e2;&#x05d1;&#x05e8;&#x05d9;&#x05ea;</div>
-<div dir="rtl">English &#x05e2;&#x05d1;&#x05e8;&#x05d9;&#x05ea; English</div>
-</body>
-</html>
diff --git a/LayoutTests/fast/text/isolate-ignore.html b/LayoutTests/fast/text/isolate-ignore.html
deleted file mode 100644 (file)
index 1061e11..0000000
+++ /dev/null
@@ -1,18 +0,0 @@
-<!DOCTYPE html>
-<html>
-<body>
-This test makes sure that isolate codepoints are ignored.
-<div dir="ltr">&#x05e2;&#x05d1;&#x05e8;&#x05d9;&#x05ea; &#x2068;English&#x2069; &#x05e2;&#x05d1;&#x05e8;&#x05d9;&#x05ea;</div>
-<div dir="ltr">English &#x2068;&#x05e2;&#x05d1;&#x05e8;&#x05d9;&#x05ea;&#x2069; English</div>
-<div dir="rtl">&#x05e2;&#x05d1;&#x05e8;&#x05d9;&#x05ea; &#x2068;English&#x2069; &#x05e2;&#x05d1;&#x05e8;&#x05d9;&#x05ea;</div>
-<div dir="rtl">English &#x2068;&#x05e2;&#x05d1;&#x05e8;&#x05d9;&#x05ea;&#x2069; English</div>
-<div dir="ltr">&#x05e2;&#x05d1;&#x05e8;&#x05d9;&#x05ea; &#x2066;English&#x2069; &#x05e2;&#x05d1;&#x05e8;&#x05d9;&#x05ea;</div>
-<div dir="ltr">English &#x2066;&#x05e2;&#x05d1;&#x05e8;&#x05d9;&#x05ea;&#x2069; English</div>
-<div dir="rtl">&#x05e2;&#x05d1;&#x05e8;&#x05d9;&#x05ea; &#x2066;English&#x2069; &#x05e2;&#x05d1;&#x05e8;&#x05d9;&#x05ea;</div>
-<div dir="rtl">English &#x2066;&#x05e2;&#x05d1;&#x05e8;&#x05d9;&#x05ea;&#x2069; English</div>
-<div dir="ltr">&#x05e2;&#x05d1;&#x05e8;&#x05d9;&#x05ea; &#x2067;English&#x2069; &#x05e2;&#x05d1;&#x05e8;&#x05d9;&#x05ea;</div>
-<div dir="ltr">English &#x2067;&#x05e2;&#x05d1;&#x05e8;&#x05d9;&#x05ea;&#x2069; English</div>
-<div dir="rtl">&#x05e2;&#x05d1;&#x05e8;&#x05d9;&#x05ea; &#x2067;English&#x2069; &#x05e2;&#x05d1;&#x05e8;&#x05d9;&#x05ea;</div>
-<div dir="rtl">English &#x2067;&#x05e2;&#x05d1;&#x05e8;&#x05d9;&#x05ea;&#x2069; English</div>
-</body>
-</html>
index 0ac50f0..99be243 100644 (file)
@@ -1,3 +1,17 @@
+2016-06-14  Commit Queue  <commit-queue@webkit.org>
+
+        Unreviewed, rolling out r202057.
+        https://bugs.webkit.org/show_bug.cgi?id=158749
+
+        This change broke the Windows build. (Requested by ryanhaddad
+        on #webkit).
+
+        Reverted changeset:
+
+        "Honor bidi unicode codepoints"
+        https://bugs.webkit.org/show_bug.cgi?id=149170
+        http://trac.webkit.org/changeset/202057
+
 2016-06-14  Myles C. Maxfield  <mmaxfield@apple.com>
 
         Honor bidi unicode codepoints
index 1dc2794..894c5af 100644 (file)
@@ -48,7 +48,6 @@ const UChar bullseye = 0x25CE;
 const UChar carriageReturn = 0x000D;
 const UChar ethiopicPrefaceColon = 0x1366;
 const UChar ethiopicWordspace = 0x1361;
-const UChar firstStrongIsolate = 0x2068;
 const UChar fisheye = 0x25C9;
 const UChar quotationMark = 0x0022;
 const UChar hebrewPunctuationGeresh = 0x05F3;
@@ -63,7 +62,6 @@ const UChar ideographicSpace = 0x3000;
 const UChar leftDoubleQuotationMark = 0x201C;
 const UChar leftSingleQuotationMark = 0x2018;
 const UChar leftToRightEmbed = 0x202A;
-const UChar leftToRightIsolate = 0x2066;
 const UChar leftToRightMark = 0x200E;
 const UChar leftToRightOverride = 0x202D;
 const UChar minusSign = 0x2212;
@@ -72,12 +70,10 @@ const UChar newlineCharacter = 0x000A;
 const UChar noBreakSpace = 0x00A0;
 const UChar objectReplacementCharacter = 0xFFFC;
 const UChar popDirectionalFormatting = 0x202C;
-const UChar popDirectionalIsolate = 0x2069;
 const UChar replacementCharacter = 0xFFFD;
 const UChar rightDoubleQuotationMark = 0x201D;
 const UChar rightSingleQuotationMark = 0x2019;
 const UChar rightToLeftEmbed = 0x202B;
-const UChar rightToLeftIsolate = 0x2067;
 const UChar rightToLeftMark = 0x200F;
 const UChar rightToLeftOverride = 0x202E;
 const UChar sesameDot = 0xFE45;
@@ -111,7 +107,6 @@ using WTF::Unicode::bullseye;
 using WTF::Unicode::carriageReturn;
 using WTF::Unicode::ethiopicPrefaceColon;
 using WTF::Unicode::ethiopicWordspace;
-using WTF::Unicode::firstStrongIsolate;
 using WTF::Unicode::fisheye;
 using WTF::Unicode::hebrewPunctuationGeresh;
 using WTF::Unicode::hebrewPunctuationGershayim;
@@ -125,7 +120,6 @@ using WTF::Unicode::ideographicSpace;
 using WTF::Unicode::leftDoubleQuotationMark;
 using WTF::Unicode::leftSingleQuotationMark;
 using WTF::Unicode::leftToRightEmbed;
-using WTF::Unicode::leftToRightIsolate;
 using WTF::Unicode::leftToRightMark;
 using WTF::Unicode::leftToRightOverride;
 using WTF::Unicode::minusSign;
@@ -134,12 +128,10 @@ using WTF::Unicode::newlineCharacter;
 using WTF::Unicode::noBreakSpace;
 using WTF::Unicode::objectReplacementCharacter;
 using WTF::Unicode::popDirectionalFormatting;
-using WTF::Unicode::popDirectionalIsolate;
 using WTF::Unicode::replacementCharacter;
 using WTF::Unicode::rightDoubleQuotationMark;
 using WTF::Unicode::rightSingleQuotationMark;
 using WTF::Unicode::rightToLeftEmbed;
-using WTF::Unicode::rightToLeftIsolate;
 using WTF::Unicode::rightToLeftMark;
 using WTF::Unicode::rightToLeftOverride;
 using WTF::Unicode::sesameDot;
index 2be5a3e..577114a 100644 (file)
@@ -1,3 +1,17 @@
+2016-06-14  Commit Queue  <commit-queue@webkit.org>
+
+        Unreviewed, rolling out r202057.
+        https://bugs.webkit.org/show_bug.cgi?id=158749
+
+        This change broke the Windows build. (Requested by ryanhaddad
+        on #webkit).
+
+        Reverted changeset:
+
+        "Honor bidi unicode codepoints"
+        https://bugs.webkit.org/show_bug.cgi?id=149170
+        http://trac.webkit.org/changeset/202057
+
 2016-06-14  Myles C. Maxfield  <mmaxfield@apple.com>
 
         Honor bidi unicode codepoints
index f9914e5..f7f4381 100644 (file)
@@ -185,13 +185,9 @@ static RefPtr<GlyphPage> createAndFillGlyphPage(unsigned pageNumber, const Font&
             buffer[rightToLeftEmbed - start] = zeroWidthSpace;
             buffer[leftToRightOverride - start] = zeroWidthSpace;
             buffer[rightToLeftOverride - start] = zeroWidthSpace;
-            buffer[leftToRightIsolate - start] = zeroWidthSpace;
-            buffer[rightToLeftIsolate - start] = zeroWidthSpace;
             buffer[zeroWidthNonJoiner - start] = zeroWidthSpace;
             buffer[zeroWidthJoiner - start] = zeroWidthSpace;
             buffer[popDirectionalFormatting - start] = zeroWidthSpace;
-            buffer[popDirectionalIsolate - start] = zeroWidthSpace;
-            buffer[firstStrongIsolate - start] = zeroWidthSpace;
         } else if (start == (objectReplacementCharacter & ~(GlyphPage::size - 1))) {
             // Object replacement character must not render at all.
             buffer[objectReplacementCharacter - start] = zeroWidthSpace;
index d854af1..6bf858b 100644 (file)
@@ -632,7 +632,7 @@ void BidiResolverBase<Iterator, Run, Subclass>::createBidiRunsForLine(const Iter
 
         // We ignore all character directionality while in unicode-bidi: isolate spans.
         // We'll handle ordering the isolated characters in a second pass.
-        if (inIsolate() || dirCurrent == U_FIRST_STRONG_ISOLATE || dirCurrent == U_LEFT_TO_RIGHT_ISOLATE || dirCurrent == U_RIGHT_TO_LEFT_ISOLATE || dirCurrent == U_POP_DIRECTIONAL_ISOLATE)
+        if (inIsolate())
             dirCurrent = U_OTHER_NEUTRAL;
 
         ASSERT(m_status.eor != U_OTHER_NEUTRAL || m_eor.atEnd());