[webkit-changes] cvs commit: WebCore/layout-tests/fast/text/international bidi-CS-after-AN-expected.txt bidi-CS-after-AN.html

Darin darin at opensource.apple.com
Fri Jun 24 21:47:06 PDT 2005


darin       05/06/24 21:47:06

  Modified:    .        ChangeLog
               khtml/rendering bidi.cpp
  Added:       layout-tests/fast/text/international
                        bidi-CS-after-AN-expected.txt bidi-CS-after-AN.html
  Log:
          Changes by Mitz Pettel.
          Reviewed by me.
  
          Test cases added:
          * layout-tests/fast/text/international/bidi-CS-after-AN-expected.txt: Added.
          * layout-tests/fast/text/international/bidi-CS-after-AN.html: Added.
  
          - fixed <http://bugzilla.opendarwin.org/show_bug.cgi?id=3693>
            Bug_72578 in KHTML: bidi algorithm misbehaves with CS after AN
  
          * khtml/rendering/bidi.cpp: (khtml::RenderBlock::bidiReorderLine): Rolled in the
          patch from the KDE tree.
  
  Revision  Changes    Path
  1.4315    +15 -0     WebCore/ChangeLog
  
  Index: ChangeLog
  ===================================================================
  RCS file: /cvs/root/WebCore/ChangeLog,v
  retrieving revision 1.4314
  retrieving revision 1.4315
  diff -u -r1.4314 -r1.4315
  --- ChangeLog	24 Jun 2005 21:05:08 -0000	1.4314
  +++ ChangeLog	25 Jun 2005 04:47:01 -0000	1.4315
  @@ -1,3 +1,18 @@
  +2005-06-24  Darin Adler  <darin at apple.com>
  +
  +        Changes by Mitz Pettel.
  +        Reviewed by me.
  +
  +        Test cases added:
  +        * layout-tests/fast/text/international/bidi-CS-after-AN-expected.txt: Added.
  +        * layout-tests/fast/text/international/bidi-CS-after-AN.html: Added.
  +
  +        - fixed <http://bugzilla.opendarwin.org/show_bug.cgi?id=3693>
  +          Bug_72578 in KHTML: bidi algorithm misbehaves with CS after AN
  +
  +        * khtml/rendering/bidi.cpp: (khtml::RenderBlock::bidiReorderLine): Rolled in the
  +        patch from the KDE tree.
  +
   2005-06-24  Justin Garcia  <justin.garcia at apple.com>
   
           - <http://bugzilla.opendarwin.org/show_bug.cgi?id=3673> 
  
  
  
  1.129     +1 -1      WebCore/khtml/rendering/bidi.cpp
  
  Index: bidi.cpp
  ===================================================================
  RCS file: /cvs/root/WebCore/khtml/rendering/bidi.cpp,v
  retrieving revision 1.128
  retrieving revision 1.129
  diff -u -r1.128 -r1.129
  --- bidi.cpp	22 Jun 2005 17:39:53 -0000	1.128
  +++ bidi.cpp	25 Jun 2005 04:47:05 -0000	1.129
  @@ -1104,7 +1104,7 @@
                       break;
                   case QChar::DirCS:
                       if(bidi.status.eor == QChar::DirAN) {
  -                        bidi.eor = bidi.current; bidi.status.eor = QChar::DirR; break;
  +                        bidi.eor = bidi.current; break;
                       }
                   case QChar::DirES:
                   case QChar::DirET:
  
  
  
  1.1                  WebCore/layout-tests/fast/text/international/bidi-CS-after-AN-expected.txt
  
  Index: bidi-CS-after-AN-expected.txt
  ===================================================================
  layer at (0,0) size 800x600
    RenderCanvas at (0,0) size 800x600
  layer at (0,0) size 800x84
    RenderBlock {HTML} at (0,0) size 800x84
      RenderBody {BODY} at (8,16) size 784x52
        RenderBlock {P} at (0,0) size 784x18
          RenderText {TEXT} at (0,0) size 471x18
            text run at (0,0) width 471: "The following should look like: Arabic characters 123.4 Arabic characters."
        RenderBlock {P} at (0,34) size 784x18
          RenderText {TEXT} at (0,0) size 74x18
            text run at (0,0) width 18: " \x{644}\x{645}\x{646}"
            text run at (18,0) width 36: "123.4"
            text run at (54,0) width 20: "\x{627}\x{628}\x{62C} "
  
  
  
  1.1                  WebCore/layout-tests/fast/text/international/bidi-CS-after-AN.html
  
  Index: bidi-CS-after-AN.html
  ===================================================================
  <!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN" 
  "http://www.w3.org/TR/html4/loose.dtd">
  <html>
  <head>
  <title>CS after AN</title>
  </head>
  <body>
  <p>
  The following should look like: Arabic characters 123.4 Arabic characters.
  </p>
  <p>
  &#x0627;&#x0628;&#x062C; 123.4 &#x0644;&#x0645;&#x0646;
  </p>
  </body >
  
  
  



More information about the webkit-changes mailing list