[webkit-changes] cvs commit: WebCore/kwq WebCoreBridge.mm

Justin justing at opensource.apple.com
Mon Oct 3 18:59:56 PDT 2005


justing     05/10/03 18:59:55

  Modified:    .        ChangeLog
               kwq      WebCoreBridge.mm
  Log:
          Reviewed by darin
  
          <rdar://problem/4060947> Safari crashes trying to save PDF file as web archive (NULL KHTMLPart)
  
          * kwq/WebCoreBridge.mm:
          (+[WebCoreBridge bridgeForDOMDocument:]):
          Added a check for a null part.
  
  Revision  Changes    Path
  1.192     +10 -0     WebCore/ChangeLog
  
  Index: ChangeLog
  ===================================================================
  RCS file: /cvs/root/WebCore/ChangeLog,v
  retrieving revision 1.191
  retrieving revision 1.192
  diff -u -r1.191 -r1.192
  --- ChangeLog	3 Oct 2005 23:11:11 -0000	1.191
  +++ ChangeLog	4 Oct 2005 01:59:54 -0000	1.192
  @@ -1,5 +1,15 @@
   2005-10-03  Justin Garcia  <justin.garcia at apple.com>
   
  +        Reviewed by darin
  +        
  +        <rdar://problem/4060947> Safari crashes trying to save PDF file as web archive (NULL KHTMLPart)
  +
  +        * kwq/WebCoreBridge.mm:
  +        (+[WebCoreBridge bridgeForDOMDocument:]):
  +        Added a check for a null part.
  +
  +2005-10-03  Justin Garcia  <justin.garcia at apple.com>
  +
           Reviewed by geoff, darin
   
           * khtml/khtml_part.cpp:
  
  
  
  1.420     +2 -1      WebCore/kwq/WebCoreBridge.mm
  
  Index: WebCoreBridge.mm
  ===================================================================
  RCS file: /cvs/root/WebCore/kwq/WebCoreBridge.mm,v
  retrieving revision 1.419
  retrieving revision 1.420
  diff -u -r1.419 -r1.420
  --- WebCoreBridge.mm	3 Oct 2005 21:13:11 -0000	1.419
  +++ WebCoreBridge.mm	4 Oct 2005 01:59:55 -0000	1.420
  @@ -243,7 +243,8 @@
   
   + (WebCoreBridge *)bridgeForDOMDocument:(DOMDocument *)document
   {
  -    return ((KWQKHTMLPart *)[document _documentImpl]->part())->bridge();
  +    KHTMLPart *part = [document _documentImpl]->part();
  +    return part ? KWQ(part)->bridge() : nil;
   }
   
   - init
  
  
  



More information about the webkit-changes mailing list