Bug 135599 - [test case] overflow sometimes not painted/repainted
Summary: [test case] overflow sometimes not painted/repainted
Status: RESOLVED FIXED
Alias: None
Product: konqueror
Classification: Applications
Component: khtml renderer (show other bugs)
Version: unspecified
Platform: Compiled Sources Linux
: NOR normal
Target Milestone: ---
Assignee: Konqueror Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2006-10-13 18:06 UTC by Allan Sandfeld
Modified: 2006-10-13 19:24 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:

Attachments
Test case (203 bytes, text/html)
2006-10-13 18:07 UTC, Allan Sandfeld 		Details
View All Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
Description Allan Sandfeld 2006-10-13 18:06:08 UTC 
Version:            (using KDE Devel)
Installed from:    Compiled sources

I discovered a problem with the online commic sinfest.net and have investigated the issue.

It seems overflow is sometimes not painted or repainted in the following specific circumstance:
  <div style="position: absolute; width: 800; height: 141;">
  
  <div style="width: 800; height: 200;">
  
  <div style="border: 5px solid black; height: 500px; width: 600px;"> </div>

It seems to need to be constrained by two boxes and the outer box needs to be positioned for the bug to trigger.
Comment 1 Allan Sandfeld 2006-10-13 18:07:05 UTC 
Created attachment 18113 [details]
Test case
Comment 2 Allan Sandfeld 2006-10-13 19:24:41 UTC 
SVN commit 595245 by carewolf:

Remove complete nonsense I accidently imported from WebCore.
BUG: 135599


 M  +0 -4      render_box.cpp  


--- branches/KDE/3.5/kdelibs/khtml/rendering/render_box.cpp #595244:595245
@@ -1709,10 +1709,6 @@
 
     height += bordersPlusPadding;
 
-    // If our natural/content height exceeds the new height once we've set it, then we
-    // need to make sure to update overflow to track the spillout.
-    if (m_height > height && isRenderBlock())
-        static_cast<RenderBlock*>(this)->setOverflowHeight(m_height);
     // Set final height value.
     m_height = height;
 }