Summary: | Timer restart loop during page loading | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Ismail Donmez <ismail> | ||||||
Component: | Page Loading | Assignee: | QtWebKit Unassigned <webkit-qt-unassigned> | ||||||
Status: | CLOSED FIXED | ||||||||
Severity: | Normal | CC: | benjamin, darin, hamaji, hausmann, hayato, jchaffraix, kenneth, koivisto, seo | ||||||
Priority: | P1 | ||||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | Other | ||||||||
OS: | Other | ||||||||
URL: | http://www.cnnturk.com | ||||||||
Bug Depends on: | |||||||||
Bug Blocks: | 35784 | ||||||||
Attachments: |
|
Description
Ismail Donmez
2010-03-27 07:48:08 PDT
Created attachment 51829 [details]
Backtrace
Backtrace file, easier to read this way
Any other URLs having the same issue? http://haberturk.com exhibits the same behaviour. This looks like a regression from http://trac.webkit.org/changeset/54526 and I believe affects all platforms, not just Qt. That change introduces a timer repeat loop that will peak CPU during page loading. (In reply to comment #4) > This looks like a regression from http://trac.webkit.org/changeset/54526 and I > believe affects all platforms, not just Qt. > > That change introduces a timer repeat loop that will peak CPU during page > loading. Reverting that change fixes cnnturk.com & haberturk.com for me. Thanks for hunting this down! Created attachment 51892 [details]
Patch
Patch fixes the problem for me, also http://www.parentsdanslesparages.com/ page works fine. Comment on attachment 51892 [details]
Patch
Tests look good too.
Sending WebCore/ChangeLog Sending WebCore/loader/loader.cpp Transmitting file data .. Committed revision 56723. Revision r56723 cherry-picked into qtwebkit-2.0 with commit fe2de805222c2bf698244473e5d160305403f357 (In reply to Ismail Donmez from comment #7) > Patch fixes the problem for me, also http://www.guvenlikonline.com/ > page works fine. (In reply to murat from comment #11) > (In reply to Ismail Donmez from comment #7) > > Patch fixes the problem for me, also http://www.guvenlikonline.com/ And http://www.akinprefabrik.com > > page works fine. |