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

[Adele]

adele       05/10/27 13:13:44

  Modified:    .        ChangeLog
               khtml    khtml_part.cpp
  Log:
          Reviewed by Vicki.
  
          Fixed bug that prevents overflow areas from scrolling to reveal anchor.
          Part of <rdar://problem/3612121>.
  
          * khtml/khtml_part.cpp: (KHTMLPart::gotoAnchor):
  
  Revision  Changes    Path
  1.303     +9 -0      WebCore/ChangeLog
  
  Index: ChangeLog
  ===================================================================
  RCS file: /cvs/root/WebCore/ChangeLog,v
  retrieving revision 1.302
  retrieving revision 1.303
  diff -u -r1.302 -r1.303
  --- ChangeLog	27 Oct 2005 17:39:32 -0000	1.302
  +++ ChangeLog	27 Oct 2005 20:13:35 -0000	1.303
  @@ -2,6 +2,15 @@
   
           Reviewed by Vicki.
   
  +        Fixed bug that prevents overflow areas from scrolling to reveal anchor.
  +        Part of <rdar://problem/3612121>.
  +
  +        * khtml/khtml_part.cpp: (KHTMLPart::gotoAnchor):  
  +
  +2005-10-27  Adele Peterson  <adele at apple.com>
  +
  +        Reviewed by Vicki.
  +
           - fixed <rdar://problem/4318167> REGRESSION: content doesn't scroll far enough to the left after clicking links at aplacecalledcommon.co.uk
   
           * khtml/rendering/render_layer.cpp: 
  
  
  
  1.354     +3 -3      WebCore/khtml/khtml_part.cpp
  
  Index: khtml_part.cpp
  ===================================================================
  RCS file: /cvs/root/WebCore/khtml/khtml_part.cpp,v
  retrieving revision 1.353
  retrieving revision 1.354
  diff -u -r1.353 -r1.354
  --- khtml_part.cpp	27 Oct 2005 17:39:35 -0000	1.353
  +++ khtml_part.cpp	27 Oct 2005 20:13:42 -0000	1.354
  @@ -2307,11 +2307,11 @@
     }
   #if APPLE_CHANGES
     // scrollRectToVisible will expose correctly from within nested layers and frames.
  -  if (d->m_doc && d->m_doc->renderer()) {
  -    khtml::RenderLayer *layer =  d->m_doc->renderer()->enclosingLayer();
  +  if (n && n->renderer()) {
  +    khtml::RenderLayer *layer = n->renderer()->enclosingLayer();
       if (layer)
           // We used to align to the top left corner, but now, if the anchor is already horizontally visible, we won't scroll horizontally.
  -        d->m_doc->renderer()->enclosingLayer()->scrollRectToVisible(QRect(x, y, 0, 0), alignTop, alignLeft);
  +        n->renderer()->enclosingLayer()->scrollRectToVisible(QRect(x, y, 0, 0), alignTop, alignLeft);
     }
   #else
     d->m_view->setContentsPos(x, y);