Bug 15213

Summary:

[S60] MLIO-75LFA9: <<TRUE>> crash is generated while browsing

Product:

WebKit

Reporter:

Wei Liu <wei.liu>

Component:

Frames

Assignee:

Nobody <webkit-unassigned>

Status:

CLOSED INVALID

Severity:

Critical

Priority:

Version:

523.x (Safari 3)

Hardware:

S60 Hardware

OS:

S60 3rd edition

URL:

http://money.cnn.com/data/markets

Attachments:

Description	Flags
Add ref and deref for handling parent part.	zalan: review-
revised patch to address Zalan's comment	zalan: review+

Description Wei Liu 2007-09-14 07:24:22 PDT

1. Set "Load content" setting to "All" from browser preferences (might not need this, but E-series devices has this as default)
2. Open http://money.cnn.com/2007/07/27/magazines/business2/motorola_analysis.biz2/index.htm?source=yahoo_quote
or 
 http://money.cnn.com/data/markets
3. If page opened without crash, clear all privacy data and close browser, then start from beginning

Comment 1 Wei Liu 2007-09-14 07:49:13 PDT

Created attachment 16289 [details]
Add ref and deref for handling parent part.

Hi Zalan,

If the fix is OK, could you please check it in both 3.1, 3.2 and/or ccb? thanks!

Comment 2 zalan 2007-09-19 10:34:03 PDT

Comment on attachment 16289 [details]
Add ref and deref for handling parent part.

could you move the ref() to KWQKHTMLPart::setParent() (override the ReadOnlyPart::setParent()). ref could go out of sync if some other part of the code will call setParent on khtmlpart.

Comment 3 Wei Liu 2007-09-20 09:59:05 PDT

Created attachment 16335 [details]
revised patch to address Zalan's comment

Hi Zalan,

Please review this patch at your earliest convenience. thanks.

Comment 4 zalan 2007-09-20 10:10:23 PDT

Comment on attachment 16335 [details]
revised patch to address Zalan's comment

r=me

Comment 5 Bradley Morrison 2008-04-09 11:31:55 PDT

Bulk update of s60 bugs - tagging and closing.

Comment 6 Bradley Morrison 2008-04-09 11:39:37 PDT

Bulk closing of all s60 platform bugs. 

Sorry for the noise!

Comment 7 Joel Parks 2011-03-21 11:53:38 PDT

re-purposing InTSW keyword for use by QtWebkit team