Bug 15213 - [S60] MLIO-75LFA9: <<TRUE>> crash is generated while browsing
Summary: [S60] MLIO-75LFA9: <<TRUE>> crash is generated while browsing
Status: CLOSED INVALID
Alias: None
Product: WebKit
Classification: Unclassified
Component: Frames (show other bugs)
Version: 523.x (Safari 3)
Hardware: S60 Hardware S60 3rd edition
: P2 Critical
Assignee: Nobody
URL: http://money.cnn.com/data/markets
Keywords:
Depends on:
Blocks:
 
Reported: 2007-09-14 07:24 PDT by Wei Liu
Modified: 2011-03-21 11:53 PDT (History)
0 users

See Also:


Attachments
Add ref and deref for handling parent part. (1.36 KB, patch)
2007-09-14 07:49 PDT, Wei Liu
zalan: review-
Details | Formatted Diff | Diff
revised patch to address Zalan's comment (2.34 KB, patch)
2007-09-20 09:59 PDT, Wei Liu
zalan: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
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