Show
Ignore:
Timestamp:
01/21/07 12:29:41 (2 years ago)
Author:
weinig
Message:

LayoutTests:

Reviewed by Darin.

  • fast/overflow/position-relative-expected.checksum: Added.
  • fast/overflow/position-relative-expected.png: Added.
  • fast/overflow/position-relative-expected.txt: Added.
  • fast/overflow/position-relative.html: Added.

WebCore:

Reviewed by Darin.

Test: fast/overflow/position-relative.html

  • rendering/RenderBox.cpp: (WebCore::RenderBox::lowestPosition): Add the relative position offset only if 'includeSelf' is true. (WebCore::RenderBox::rightmostPosition): Ditto. (WebCore::RenderBox::leftmostPosition): Ditto.
  • rendering/RenderFlow.cpp: (WebCore::RenderFlow::lowestPosition): Ditto. (WebCore::RenderFlow::rightmostPosition): Ditto. (WebCore::RenderFlow::leftmostPosition): Ditto.
Files:
1 modified

Legend:

Unmodified
Added
Removed

  • trunk/WebCore/ChangeLog

    r19011 r19012  
     12007-01-21  Mitz Pettel  <mitz@webkit.org> 
     2 
     3        Reviewed by Darin. 
     4 
     5        - fix http://bugs.webkit.org/show_bug.cgi?id=10692 
     6          REGRESSION (r14847): Unnecessary scrollbars with position:relative; and overflow:auto; 
     7 
     8        Test: fast/overflow/position-relative.html 
     9 
     10        * rendering/RenderBox.cpp: 
     11        (WebCore::RenderBox::lowestPosition): Add the relative position offset only 
     12        if 'includeSelf' is true. 
     13        (WebCore::RenderBox::rightmostPosition): Ditto. 
     14        (WebCore::RenderBox::leftmostPosition): Ditto. 
     15        * rendering/RenderFlow.cpp: 
     16        (WebCore::RenderFlow::lowestPosition): Ditto. 
     17        (WebCore::RenderFlow::rightmostPosition): Ditto. 
     18        (WebCore::RenderFlow::leftmostPosition): Ditto. 
     19 
    1202007-01-21  Mitz Pettel  <mitz@webkit.org> 
    221