Bug 48730

Summary:

[Qt] QtTestBrowser runs out of memory with most of javascript benchmarks.

Product:

WebKit

Reporter:

Viatcheslav Ostapenko <ostap73>

Component:

Tools / Tests

Assignee:

Nobody <webkit-unassigned>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

ademar, commit-queue, kling, laszlo.gombos, nancy.piedra, suresh.voruganti

Priority:

Version:

528+ (Nightly build)

Hardware:

S60 Hardware

OS:

S60 3rd edition

Attachments:

Description	Flags
Increase symbian heap size to 256Mb	kling: review-
Update changelog by Andreas suggestion.	none

Viatcheslav Ostapenko

Reported 2010-10-31 09:56:11 PDT

Heap limit is 96Mb on symbian. Just not enough RAM for JS benchmarks to run (for example, Dromaeo or Celtic Kane).

Attachments
Increase symbian heap size to 256Mb (1.14 KB, patch) 2010-10-31 10:05 PDT, Viatcheslav Ostapenko	kling: review-	Details Formatted Diff Diff
Update changelog by Andreas suggestion. (1.23 KB, patch) 2010-11-01 11:20 PDT, Viatcheslav Ostapenko	no flags	Details Formatted Diff Diff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.

Viatcheslav Ostapenko

Comment 1 2010-10-31 10:05:12 PDT

Created attachment 72464 [details] Increase symbian heap size to 256Mb

Andreas Kling

Comment 2 2010-10-31 10:15:50 PDT

Related to bug 47918?

Nancy Piedra

Comment 3 2010-10-31 10:28:05 PDT

This change helps but does not completely solve the issue with dromeao dom tests on symbian. Still, we need this change.

Andreas Kling

Comment 4 2010-10-31 12:22:51 PDT

Comment on attachment 72464 [details] Increase symbian heap size to 256Mb The change looks fine, but the ChangeLog is too terse. Suggestion: [Qt][Symbian] QtTestBrowser runs out of memory with most of javascript benchmarks. https://bugs.webkit.org/show_bug.cgi?id=48730 Increase the maximum heap size on Symbian to 256 MB.

Viatcheslav Ostapenko

Comment 5 2010-11-01 11:20:37 PDT

Created attachment 72525 [details] Update changelog by Andreas suggestion.

Suresh Voruganti

Comment 6 2010-11-01 13:15:46 PDT

Fix is required for Qtwebkit 2.1 as this increased the stability for Browser.

WebKit Commit Bot

Comment 7 2010-11-01 19:19:51 PDT

The commit-queue encountered the following flaky tests while processing attachment 72525 [details]: fast/workers/storage/use-same-database-in-page-and-workers.html Please file bugs against the tests. These tests were authored by dumi@chromium.org. The commit-queue is continuing to process your patch.

WebKit Commit Bot

Comment 8 2010-11-01 19:20:44 PDT

Comment on attachment 72525 [details] Update changelog by Andreas suggestion. Clearing flags on attachment: 72525 Committed r71096: <http://trac.webkit.org/changeset/71096>

WebKit Commit Bot

Comment 9 2010-11-01 19:20:50 PDT

All reviewed patches have been landed. Closing bug.

Ademar Reis

Comment 10 2010-11-03 10:21:32 PDT

Revision r71096 cherry-picked into qtwebkit-2.1 with commit dcd6fcd <http://gitorious.org/webkit/qtwebkit/commit/dcd6fcd>

Note You need to log in before you can comment on or make changes to this bug.