Bug 15213

Summary: [S60] MLIO-75LFA9: <<TRUE>> crash is generated while browsing
Product: WebKit Reporter: Wei Liu <wei.liu>
Component: FramesAssignee: Nobody <webkit-unassigned>
Status: CLOSED INVALID    
Severity: Critical    
Priority: P2    
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+

Wei Liu
Reported 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
Attachments
Add ref and deref for handling parent part. (1.36 KB, patch)
2007-09-14 07:49 PDT, Wei Liu 		zalan: review-
DetailsFormatted DiffDiff
revised patch to address Zalan's comment (2.34 KB, patch)
2007-09-20 09:59 PDT, Wei Liu 		zalan: review+
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Wei Liu
Comment 1 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!
zalan
Comment 2 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.
Wei Liu
Comment 3 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.
zalan
Comment 4 2007-09-20 10:10:23 PDT
Comment on attachment 16335 [details] revised patch to address Zalan's comment r=me
Bradley Morrison
Comment 5 2008-04-09 11:31:55 PDT
Bulk update of s60 bugs - tagging and closing.
Bradley Morrison
Comment 6 2008-04-09 11:39:37 PDT
Bulk closing of all s60 platform bugs. Sorry for the noise!
Joel Parks
Comment 7 2011-03-21 11:53:38 PDT
re-purposing InTSW keyword for use by QtWebkit team
Note You need to log in before you can comment on or make changes to this bug.