Bug 53416

Summary: [Qt] Symbian runs OOM when rendering large texts
Product: WebKit Reporter: Janne Koskinen <koshuin>
Component: New BugsAssignee: Nobody <webkit-unassigned>
Status: RESOLVED WONTFIX    
Severity: Normal CC: benjamin, ganesh.2.prabhu
Priority: P3 Keywords: Qt
Version: 528+ (Nightly build)   
Hardware: S60 Hardware   
OS: S60 3rd edition   
Attachments:
Description Flags
Test Case
none
Crash Log none

Janne Koskinen
Reported 2011-01-31 01:44:58 PST
Created attachment 80626 [details] Test Case Using QtWebkit2.1 with QtTestBrowser on Symbian^3 devices runs out of memory if a paragraph contains more than 1MB of text. I create this bug to set this issue apart from 'downloading crashes'-bugs Bug 46746 and Bug 51165 . Memory usage has increased quite a bit from Qt 4.6.3 as bugreports related to this issue are regressions. Question is should we consider this as a serious issue and whether we should in the future optimise for smaller memory consuption instead of speed. It is almost impossible to bisect against QtWebkit using Symbian so finding if there is a set of commits causing this is difficult :S
Attachments
Test Case (1.13 MB, text/html)
2011-01-31 01:44 PST, Janne Koskinen 		no flags
Details
Crash Log (128.47 KB, text/html)
2011-01-31 01:46 PST, Janne Koskinen 		no flags
Details
View All Add attachment proposed patch, testcase, etc.
Janne Koskinen
Comment 1 2011-01-31 01:46:35 PST
Created attachment 80627 [details] Crash Log Crash log output, scroll way down to see the stack.
Janne Koskinen
Comment 2 2011-01-31 01:47:43 PST
*** Bug 51165 has been marked as a duplicate of this bug. ***
Benjamin Poulain
Comment 3 2011-01-31 03:55:45 PST
If I understand the backtrace correctly, this crashes OOM when allocating the inline box for the render tree. I think killing the app in that situation is a reasonable thing, if you can't render the page, close the app. What do you expect from this bug report?
Benjamin Poulain
Comment 4 2011-01-31 07:23:47 PST
Won't fix since there are no way to handle those cases nicely.
Note You need to log in before you can comment on or make changes to this bug.