Bug 6860

Summary:

REGRESSION: crash when setting a list with list-style: none to display: none

Product:

WebKit

Reporter:

Maciej Stachowiak <mjs>

Component:

Layout and Rendering

Assignee:

Maciej Stachowiak <mjs>

Status:

RESOLVED WORKSFORME

Severity:

Major

Priority:

Version:

420+

Hardware:

Mac

OS:

OS X 10.4

Attachments:

Description	Flags
A test case demonstrating the crash	none
the fix (including test case)	timo: review+

Description Maciej Stachowiak 2006-01-26 19:29:08 PST

When you change a list that has list-style: none to also be display: none, there is a crash due to a missing nil check.

Comment 1 Maciej Stachowiak 2006-01-26 19:31:31 PST

Created attachment 6004 [details]
A test case demonstrating the crash

Comment 2 Maciej Stachowiak 2006-01-26 19:31:54 PST

This is a regression, it does not happen in Safari 2.0.2.

Comment 3 Maciej Stachowiak 2006-01-26 19:40:48 PST

Created attachment 6005 [details]
the fix (including test case)

Comment 4 Maciej Stachowiak 2006-01-26 19:45:02 PST

A reproducible crasher regression should be P1.

Comment 5 Tim Omernick 2006-01-26 20:44:10 PST

Comment on attachment 6005 [details]
the fix (including test case)

r=me

Comment 6 Maciej Stachowiak 2006-01-26 21:15:56 PST

Looks like Dave Harrison fixed this (in the exact same way, line for line) but I'll add the layout test anyway.

Comment 7 Eric Seidel (no email) 2006-01-31 21:20:48 PST

Removing Regression keyword from bugs already fixed.