[webkit-changes] cvs commit: WebKit/WebCoreSupport.subproj WebTextRenderer.m

[Darin]

darin       05/12/07 23:41:21

  Modified:    .        ChangeLog
               WebCoreSupport.subproj WebTextRenderer.m
  Log:
          Reviewed by Maciej, landed by Darin.
  
          - fix http://bugzilla.opendarwin.org/show_bug.cgi?id=5878
            REGRESSION (WebTextRenderer.m r1.201): pointToOffset always takes the CG code path
  
          * WebCoreSupport.subproj/WebTextRenderer.m:
          (-[WebTextRenderer pointToOffset:style:position:reversed:includePartialGlyphs:]):
          Added the missing "return".
  
  Revision  Changes    Path
  1.3387    +11 -0     WebKit/ChangeLog
  
  Index: ChangeLog
  ===================================================================
  RCS file: /cvs/root/WebKit/ChangeLog,v
  retrieving revision 1.3386
  retrieving revision 1.3387
  diff -u -r1.3386 -r1.3387
  --- ChangeLog	8 Dec 2005 00:19:13 -0000	1.3386
  +++ ChangeLog	8 Dec 2005 07:41:08 -0000	1.3387
  @@ -1,3 +1,14 @@
  +2005-12-07  Mitz Pettel  <opendarwin.org at mitzpettel.com>
  +
  +        Reviewed by Maciej, landed by Darin.
  +
  +        - fix http://bugzilla.opendarwin.org/show_bug.cgi?id=5878
  +          REGRESSION (WebTextRenderer.m r1.201): pointToOffset always takes the CG code path
  +
  +        * WebCoreSupport.subproj/WebTextRenderer.m:
  +        (-[WebTextRenderer pointToOffset:style:position:reversed:includePartialGlyphs:]):
  +        Added the missing "return".
  +
   2005-12-07  John Sullivan  <sullivan at apple.com>
   
           Reviewed by Tim Omernick.
  
  
  
  1.202     +1 -1      WebKit/WebCoreSupport.subproj/WebTextRenderer.m
  
  Index: WebTextRenderer.m
  ===================================================================
  RCS file: /cvs/root/WebKit/WebCoreSupport.subproj/WebTextRenderer.m,v
  retrieving revision 1.201
  retrieving revision 1.202
  diff -u -r1.201 -r1.202
  --- WebTextRenderer.m	24 Oct 2005 23:38:31 -0000	1.201
  +++ WebTextRenderer.m	8 Dec 2005 07:41:21 -0000	1.202
  @@ -582,7 +582,7 @@
   - (int)pointToOffset:(const WebCoreTextRun *)run style:(const WebCoreTextStyle *)style position:(int)x reversed:(BOOL)reversed includePartialGlyphs:(BOOL)includePartialGlyphs
   {
       if (shouldUseATSU(run))
  -        ATSU_pointToOffset(self, run, style, x, reversed, includePartialGlyphs);
  +        return ATSU_pointToOffset(self, run, style, x, reversed, includePartialGlyphs);
       return CG_pointToOffset(self, run, style, x, reversed, includePartialGlyphs);
   }