[webkit-changes] cvs commit: WebCore/khtml/rendering bidi.cpp

[David]

hyatt       05/12/07 14:48:57

  Modified:    .        ChangeLog
               khtml/rendering bidi.cpp
  Log:
  	Fix for Radar bug #4333923, typing tab character into empty
  	document in Mail doesn't work.  This was just a silly regression
  	from my pre-wrap/pre-line implementation.  I accidentally changed
  	code to examine the wrong style().
  
          Reviewed by harrison
  
          * khtml/rendering/bidi.cpp:
          (khtml::RenderBlock::findNextLineBreak):
  
  Revision  Changes    Path
  1.487     +12 -0     WebCore/ChangeLog
  
  Index: ChangeLog
  ===================================================================
  RCS file: /cvs/root/WebCore/ChangeLog,v
  retrieving revision 1.486
  retrieving revision 1.487
  diff -u -r1.486 -r1.487
  --- ChangeLog	7 Dec 2005 21:24:55 -0000	1.486
  +++ ChangeLog	7 Dec 2005 22:48:51 -0000	1.487
  @@ -1,5 +1,17 @@
   2005-12-07  David Hyatt  <hyatt at apple.com>
   
  +	Fix for Radar bug #4333923, typing tab character into empty
  +	document in Mail doesn't work.  This was just a silly regression
  +	from my pre-wrap/pre-line implementation.  I accidentally changed
  +	code to examine the wrong style().
  +	
  +        Reviewed by harrison
  +
  +        * khtml/rendering/bidi.cpp:
  +        (khtml::RenderBlock::findNextLineBreak):
  +
  +2005-12-07  David Hyatt  <hyatt at apple.com>
  +
   	Fix the DHTML menus on monster.com.  offsetParent should only return RenderObjects that
   	have corresponding DOM elements.
   	
  
  
  
  1.161     +1 -1      WebCore/khtml/rendering/bidi.cpp
  
  Index: bidi.cpp
  ===================================================================
  RCS file: /cvs/root/WebCore/khtml/rendering/bidi.cpp,v
  retrieving revision 1.160
  retrieving revision 1.161
  diff -u -r1.160 -r1.161
  --- bidi.cpp	1 Dec 2005 10:32:23 -0000	1.160
  +++ bidi.cpp	7 Dec 2005 22:48:56 -0000	1.161
  @@ -2126,7 +2126,7 @@
                   const QChar c = str[pos];
                   currentCharacterIsSpace = c == ' ' || c == '\t' || (!o->style()->preserveNewline() && (c == '\n'));
   
  -                if (!style()->collapseWhiteSpace() || !currentCharacterIsSpace)
  +                if (!o->style()->collapseWhiteSpace() || !currentCharacterIsSpace)
                       isLineEmpty = false;
                   
                   // Check for soft hyphens.  Go ahead and ignore them.