[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&#x05d2;&#x05d1;&#x05d0;&#x0661;&#x0662;&#x0663;</p>
  <p>abc&#x0661;&#x0662;&#x0663;&#x05d0;&#x05d1;&#x05d2;</p>
  </body>
  </html>
  
  



More information about the webkit-changes mailing list