[webkit-changes] cvs commit: LayoutTests/fast/text/international
bidi-AN-after-L-expected.txt bidi-AN-after-L.html
Maciej
mjs at opensource.apple.com
Sun Nov 27 20:37:34 PST 2005
mjs 05/11/27 20:37:34
Modified: . ChangeLog
khtml/rendering bidi.cpp
. ChangeLog
Added: fast/text/international bidi-AN-after-L-expected.txt
bidi-AN-after-L.html
Log:
LayoutTests:
Reviewed by Darin. Committed by Maciej.
- test case for Bidi algorithm: AN after L has incorrect resolved level
(http://bugzilla.opendarwin.org/show_bug.cgi?id=5573)
* fast/text/international/bidi-AN-after-L-expected.txt: Added.
* fast/text/international/bidi-AN-after-L.html: Added.
WebCore:
Reviewed by Darin. Committed by Maciej.
- fixed Bidi algorithm: AN after L has incorrect resolved level
(http://bugzilla.opendarwin.org/show_bug.cgi?id=5573)
* khtml/rendering/bidi.cpp:
(khtml::RenderBlock::bidiReorderLine):
Revision Changes Path
1.410 +10 -0 WebCore/ChangeLog
Index: ChangeLog
===================================================================
RCS file: /cvs/root/WebCore/ChangeLog,v
retrieving revision 1.409
retrieving revision 1.410
diff -u -r1.409 -r1.410
--- ChangeLog 27 Nov 2005 22:52:04 -0000 1.409
+++ ChangeLog 28 Nov 2005 04:37:27 -0000 1.410
@@ -1,5 +1,15 @@
2005-11-27 Mitz Pettel <opendarwin.org at mitzpettel.com>
+ Reviewed by Darin. Committed by Maciej.
+
+ - fixed Bidi algorithm: AN after L has incorrect resolved level
+ (http://bugzilla.opendarwin.org/show_bug.cgi?id=5573)
+
+ * khtml/rendering/bidi.cpp:
+ (khtml::RenderBlock::bidiReorderLine):
+
+2005-11-27 Mitz Pettel <opendarwin.org at mitzpettel.com>
+
Reviewed by hyatt. Committed by eseidel.
One test case added:
1.158 +3 -0 WebCore/khtml/rendering/bidi.cpp
Index: bidi.cpp
===================================================================
RCS file: /cvs/root/WebCore/khtml/rendering/bidi.cpp,v
retrieving revision 1.157
retrieving revision 1.158
diff -u -r1.157 -r1.158
--- bidi.cpp 27 Nov 2005 22:52:08 -0000 1.157
+++ bidi.cpp 28 Nov 2005 04:37:32 -0000 1.158
@@ -1178,6 +1178,9 @@
dirCurrent = QChar::DirAN;
switch (bidi.status.last) {
case QChar::DirL:
+ if (bidi.context->dir == QChar::DirL)
+ appendRun(bidi);
+ break;
case QChar::DirAN:
break;
case QChar::DirR:
1.105 +10 -0 LayoutTests/ChangeLog
Index: ChangeLog
===================================================================
RCS file: /cvs/root/LayoutTests/ChangeLog,v
retrieving revision 1.104
retrieving revision 1.105
diff -u -r1.104 -r1.105
--- ChangeLog 27 Nov 2005 22:53:40 -0000 1.104
+++ ChangeLog 28 Nov 2005 04:37:33 -0000 1.105
@@ -1,5 +1,15 @@
2005-11-27 Mitz Pettel <opendarwin.org at mitzpettel.com>
+ Reviewed by Darin. Committed by Maciej.
+
+ - test case for Bidi algorithm: AN after L has incorrect resolved level
+ (http://bugzilla.opendarwin.org/show_bug.cgi?id=5573)
+
+ * fast/text/international/bidi-AN-after-L-expected.txt: Added.
+ * fast/text/international/bidi-AN-after-L.html: Added.
+
+2005-11-27 Mitz Pettel <opendarwin.org at mitzpettel.com>
+
Reviewed by hyatt. Committed by eseidel.
Test cases for http://bugzilla.opendarwin.org/show_bug.cgi?id=5172
1.1 LayoutTests/fast/text/international/bidi-AN-after-L-expected.txt
Index: bidi-AN-after-L-expected.txt
===================================================================
layer at (0,0) size 800x600
RenderCanvas at (0,0) size 800x600
layer at (0,0) size 800x110
RenderBlock {HTML} at (0,0) size 800x110
RenderBody {BODY} at (8,8) size 784x86
RenderBlock (anonymous) at (0,0) size 784x18
RenderText {TEXT} at (0,0) size 279x18
text run at (0,0) width 279: "The following two lines should be the same:"
RenderBlock {P} at (0,34) size 784x18
RenderText {TEXT} at (0,0) size 72x18
text run at (0,0) width 72: "abc\x{5D2}\x{5D1}\x{5D0}\x{661}\x{662}\x{663}"
RenderBlock {P} at (0,68) size 784x18
RenderText {TEXT} at (0,0) size 72x18
text run at (0,0) width 22: "abc"
text run at (22,0) width 26: "\x{5D0}\x{5D1}\x{5D2}"
text run at (48,0) width 24: "\x{661}\x{662}\x{663}"
1.1 LayoutTests/fast/text/international/bidi-AN-after-L.html
Index: bidi-AN-after-L.html
===================================================================
<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN"
"http://www.w3.org/TR/html4/loose.dtd">
<html>
<head>
<title>AN after L</title>
<style type="text/css">
.lro { direction:ltr; unicode-bidi:bidi-override; };
</style>
</head>
<body>
The following two lines should be the same:
<p class="lro">abcגבא١٢٣</p>
<p>abc١٢٣אבג</p>
</body>
</html>
More information about the webkit-changes
mailing list