Bug 44691
| Summary: | [Qt] QtTestBrowser crashed after loading 60 pages | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Alexandra Santos <alexandra.1.santos> |
| Component: | New Bugs | Assignee: | Viatcheslav Ostapenko <ostap73> |
| Status: | RESOLVED WORKSFORME | ||
| Severity: | Blocker | CC: | abecsi, hausmann, jay.tucker, kling, nancy.piedra, robert, suresh.voruganti |
| Priority: | P1 | Keywords: | Qt, QtTriaged |
| Version: | 528+ (Nightly build) | ||
| Hardware: | S60 Hardware | ||
| OS: | S60 3rd edition | ||
| Bug Depends on: | 49012 | ||
| Bug Blocks: | |||
Alexandra Santos
While running stability test on QtTestBrowser using QtWebkit 2.1, the browser crashed after loading 60 live websites. This, when running on Symbian 4 platform. Please refer to the attached crash log for analysis.
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Suresh Voruganti
Proposing this to top issue for QtWebkit 2.1, so adding this bug 39121
Simon Hausmann
The binary crash log can be found in the internal Qt issue QT-3827
Alexandra Santos
After retesting this issue using a build from the Oslo codecamp folder, Only 38 live sites could be loaded before a crash was observed. This, when using the Symbian 4,QtWebkit 2.1 + QtTestBrowser.
Jay Tucker
Assigning to myself.
Nancy Piedra
On N8 we can now load 80 live websites and 585 (all) canned web sites. This is leading me to believe the problem could be in XHR as Simon suggested. I imagine all XHRs would fail on canned sites since those URLs would not have been canned.
The following two errors seem like they may be related:
https://bugs.webkit.org/show_bug.cgi?id=46746
https://bugs.webkit.org/show_bug.cgi?id=37191
Nancy Piedra
After applying the Qt fix for QVGImagePool we can now load 189 live websites.
We are still looking at XHR issues. There are patches being proposed to 2.1
Viatcheslav Ostapenko
One of the crashes I've got several times looks very similar to this (on Qt 4.7.0 release): https://bugs.webkit.org/show_bug.cgi?id=44691
I'll rerun test with patch applied to verify that it helps.
Viatcheslav Ostapenko
(In reply to comment #7)
> One of the crashes I've got several times looks very similar to this (on Qt 4.7.0 release): https://bugs.webkit.org/show_bug.cgi?id=44691
>
> I'll rerun test with patch applied to verify that it helps.
Sorry, I mean this: http://bugreports.qt.nokia.com/browse/QTBUG-12285
Robert Hogan
(In reply to comment #2)
> The binary crash log can be found in the internal Qt issue QT-3827
How can I test this? Can we get a copy of the backtrace here?
Viatcheslav Ostapenko
After running stability tests on N8 with released Qt 4.7.0 :
There is 3 problems in Qt (already solved, but not included into release), that affect QtTestBrowser stability:
1. QVgImagePool:reclaimSpace() problem
http://qt.gitorious.org/+qt-developers/qt/releases/commit/771cfe6f172820a1a370255cb74e066913408a6f
2. Problem in with abort of http post:
http://bugreports.qt.nokia.com/browse/QTBUG-12285
3. Bug in gif conversion:
http://qt.gitorious.org/qt/staging/commit/4d974ff0a748b22e668a4cb7ef38101122c85b3b
After patching QT 4.7.0 release QtTestBrowser is able to load about 180-190 websites. Running memory monitor in parallel showed, that QtTestBrowser eats all memory (96Mb by default).
Increase of process available memory to 256Mb allowed to load about 230-240 websites before crash. After loading about 220 websites phone became slow and unresponsive (got into paging? ).
I've caught also several crashes in WebCore::IconDatabase::syncThreadMainLoop. There are several bug reports open with similar crash stack. Trying to figure out, if this is fixed and what patch applies.
Nancy Piedra
When running QtTestBrowser with Qt 4.7 stable branch we have better memory consumption (and there for don't see the out-of-memory problems). Also, Qt 4.7 stable we don't seem to see the network crashes. However, networking stops working after around 280-290 sites.
So, current status is that we can load about 280-290 sites when using a more recent version of Qt.
Nancy Piedra
The following two patches will improve memory consumption on Symbian. These patches are required to achieve the 300 site stability metric.
https://bugs.webkit.org/show_bug.cgi?id=48730
https://bugs.webkit.org/show_bug.cgi?id=48767
Suresh Voruganti
(In reply to comment #12)
> The following two patches will improve memory consumption on Symbian. These patches are required to achieve the 300 site stability metric.
> https://bugs.webkit.org/show_bug.cgi?id=48730
> https://bugs.webkit.org/show_bug.cgi?id=48767
I have marked the above 2 errors as blocking 39121, so that these fixes are cherry picked to Qtwebkit 2.1
Nancy Piedra
In order to load all websites we will need this fix also:
https://bugs.webkit.org/show_bug.cgi?id=49012
Suresh Voruganti
(In reply to comment #14)
> In order to load all websites we will need this fix also:
> https://bugs.webkit.org/show_bug.cgi?id=49012
Added the dependency for 39121 master bug for Qtwebkit 2.1, so the fix should be cherry picked to Qtwebkit 2.1
Suresh Voruganti
This issue is no longer reproducible as originally described. After testing on N8 device, QtTestHarness successfully completed the stability test loading 437 live websites. This test was run overnight using WLAN.
NOTE: A second run during the day (when WLAN is normally really busy) showed that QtTestHarness could load 398 live websites, ending the test with a "Page not Found Message" due to underlying networking issues . No crash observed.
So closing the error as WorksForMe.