Bug 9109

Summary: Remove bogus assertion in RenderBlock::tabWidth
Product: WebKit Reporter: mitz
Component: Layout and RenderingAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal    
Priority: P2    
Version: 420+   
Hardware: Mac   
OS: OS X 10.4   
Attachments:
Description Flags
Remove the assert and change the uninitialized value to -1 ggaren: review+

mitz
Reported 2006-05-25 10:16:47 PDT
If the font's space width is 0 (which can happen with a very small font) then it's OK for the tab width to also be 0. See attachment 8485 [details] (bug 9062). Since 0 is a valid value, the uninitialized value should be something else.
Attachments
Remove the assert and change the uninitialized value to -1 (2.13 KB, patch)
2006-05-25 10:19 PDT, mitz 		ggaren: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
mitz
Comment 1 2006-05-25 10:19:27 PDT
Created attachment 8541 [details] Remove the assert and change the uninitialized value to -1 No layout test since DumpRenderTree forces a minimum font size.
Geoffrey Garen
Comment 2 2006-05-25 11:44:36 PDT
Comment on attachment 8541 [details] Remove the assert and change the uninitialized value to -1 r=me
Note You need to log in before you can comment on or make changes to this bug.