117703 2013-06-17 09:23:46 -0700 [WK2][GTK] 100% CPU usage rendering a page with animations 2013-06-28 07:50:24 -0700 1 1 1 Unclassified WebKit WebKitGTK 528+ (Nightly build) Unspecified Unspecified RESOLVED FIXED http://hugogiraudel.com/2013/04/04/css-grid-layout/ P2 Major --- 1 svillar svillar alex andersca ap cgarcia gustavo mrobinson ossy rego sergio svillar thorton oldest_to_newest 901063 0 svillar 2013-06-17 09:23:46 -0700 Try to load the above page using the WebKit2 Gtk port (Minibrowser is OK). After the load finishes scrolling becomes impossible, it's even not possible to click on links. From what I've seen trying to get WebProcess backtraces while this happens, I think the culprit is some animation at the very end of the page. 901065 1 svillar 2013-06-17 09:29:17 -0700 Not sure if very useful, but it looks like the code is constantly doing this: (gdb) bt #0 0x00007fbde8989b9a in WTF::HashTableConstIterator<WTF::RefPtr<WebCore::Widget>, WTF::RefPtr<WebCore::Widget>, WTF::IdentityExtractor, WTF::PtrHash<WTF::RefPtr<WebCore::Widget> >, WTF::HashTraits<WTF::RefPtr<WebCore::Widget> >, WTF::HashTraits<WTF::RefPtr<WebCore::Widget> > >::get () from /opt/checkout/WebKit/WebKitBuild/Debug/.libs/libwebkit2gtk-3.0.so.25 #1 0x00007fbde8988d32 in WTF::HashTableConstIteratorAdapter<WTF::HashTable<WTF::RefPtr<WebCore::Widget>, WTF::RefPtr<WebCore::Widget>, WTF::IdentityExtractor, WTF::PtrHash<WTF::RefPtr<WebCore::Widget> >, WTF::HashTraits<WTF::RefPtr<WebCore::Widget> >, WTF::HashTraits<WTF::RefPtr<WebCore::Widget> > >, WTF::RefPtr<WebCore::Widget> >::get () from /opt/checkout/WebKit/WebKitBuild/Debug/.libs/libwebkit2gtk-3.0.so.25 #2 0x00007fbde8987d8c in WTF::HashTableConstIteratorAdapter<WTF::HashTable<WTF::RefPtr<WebCore::Widget>, WTF::RefPtr<WebCore::Widget>, WTF::IdentityExtractor, WTF::PtrHash<WTF::RefPtr<WebCore::Widget> >, WTF::HashTraits<WTF::RefPtr<WebCore::Widget> >, WTF::HashTraits<WTF::RefPtr<WebCore::Widget> > >, WTF::RefPtr<WebCore::Widget> >::operator* () from /opt/checkout/WebKit/WebKitBuild/Debug/.libs/libwebkit2gtk-3.0.so.25 #3 0x00007fbde91320c8 in collectFrameViewChildren () from /opt/checkout/WebKit/WebKitBuild/Debug/.libs/libwebkit2gtk-3.0.so.25 #4 0x00007fbde913bcca in WebCore::FrameView::updateLayoutAndStyleIfNeededRecursive () from /opt/checkout/WebKit/WebKitBuild/Debug/.libs/libwebkit2gtk-3.0.so.25 #5 0x00007fbde88490ce in WebKit::WebPage::layoutIfNeeded () from /opt/checkout/WebKit/WebKitBuild/Debug/.libs/libwebkit2gtk-3.0.so.25 #6 0x00007fbde88758d4 in WebKit::LayerTreeHostGtk::flushAndRenderLayers () from /opt/checkout/WebKit/WebKitBuild/Debug/.libs/libwebkit2gtk-3.0.so.25 #7 0x00007fbde887554c in WebKit::LayerTreeHostGtk::layerFlushTimerFired () from /opt/checkout/WebKit/WebKitBuild/Debug/.libs/libwebkit2gtk-3.0.so.25 #8 0x00007fbde88754ec in WebKit::LayerTreeHostGtk::layerFlushTimerFiredCallback () from /opt/checkout/WebKit/WebKitBuild/Debug/.libs/libwebkit2gtk-3.0.so.25 #9 0x00007fbde503bb5e in g_timeout_dispatch (source=0x1bf20c0, callback=0x7fbde88754d4 <WebKit::LayerTreeHostGtk::layerFlushTimerFiredCallback(WebKit::LayerTreeHostGtk*)>, user_data=0x17f5fe0) at gmain.c:4417 #10 0x00007fbde5039e01 in g_main_dispatch (context=0x112f960) at gmain.c:3058 #11 0x00007fbde503ab66 in g_main_context_dispatch (context=0x112f960) at gmain.c:3634 #12 0x00007fbde503ad56 in g_main_context_iterate (context=0x112f960, block=1, dispatch=1, self=0x117b500) at gmain.c:3705 #13 0x00007fbde503b186 in g_main_loop_run (loop=0x1167ea0) at gmain.c:3899 #14 0x00007fbde9ed149a in WebCore::RunLoop::run () from /opt/checkout/WebKit/WebKitBuild/Debug/.libs/libwebkit2gtk-3.0.so.25 #15 0x00007fbde8782a8c in WebProcessMainGtk () from /opt/checkout/WebKit/WebKitBuild/Debug/.libs/libwebkit2gtk-3.0.so.25 #16 0x00000000004009fc in main () 901077 2 mrobinson 2013-06-17 10:00:35 -0700 Hrm. Do you see the same issue with the Qt or EFL port? From the stack, what's happening is that it's doing a relayout. The amount that this happens should be limited to the page framerate. 901087 3 svillar 2013-06-17 10:33:30 -0700 (In reply to comment #2) > Hrm. Do you see the same issue with the Qt or EFL port? From the stack, what's happening is that it's doing a relayout. The amount that this happens should be limited to the page framerate. My Qt build does not show that behavior, although it's a bit older, I'll try with a more recent build. Anyway, maybe the problem is precisely that it's constantly relayouting when it shouldn't? 901142 4 svillar 2013-06-17 12:25:25 -0700 (In reply to comment #3) > (In reply to comment #2) > > Hrm. Do you see the same issue with the Qt or EFL port? From the stack, what's happening is that it's doing a relayout. The amount that this happens should be limited to the page framerate. > > My Qt build does not show that behavior, although it's a bit older, I'll try with a more recent build. Qt port behaves correctly, as the summary suggests this is a gtk only issue 901318 5 rego 2013-06-18 02:01:07 -0700 (In reply to comment #0) > Try to load the above page using the WebKit2 Gtk port (Minibrowser is OK). After the load finishes scrolling becomes impossible, it's even not possible to click on links. From what I've seen trying to get WebProcess backtraces while this happens, I think the culprit is some animation at the very end of the page. I can confirm this behavior it's happening the same to me, even WebView becomes black after a while. 904235 6 205606 svillar 2013-06-27 07:39:28 -0700 Created attachment 205606 Patch 904620 7 205606 svillar 2013-06-28 07:50:09 -0700 Comment on attachment 205606 Patch Clearing flags on attachment: 205606 Committed r152165: <http://trac.webkit.org/changeset/152165> 904621 8 svillar 2013-06-28 07:50:24 -0700 All reviewed patches have been landed. Closing bug. 205606 2013-06-27 07:39:28 -0700 2013-06-28 07:50:09 -0700 Patch bug-117703-20130627163729.patch text/plain 1897 svillar U3VidmVyc2lvbiBSZXZpc2lvbjogMTUxODI1CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViS2l0Mi9D aGFuZ2VMb2cgYi9Tb3VyY2UvV2ViS2l0Mi9DaGFuZ2VMb2cKaW5kZXggOTM5NjU2ZGE1MTU4YjRi MzQ5ZjlmYjc5ZWE3YzVhNzRlYzgzZTI1Yi4uNjE2MWFjMTA4ZDU3MDJiNTM3YmUxZGYyYTc0Y2Fl MTNhNWUwM2I3OCAxMDA2NDQKLS0tIGEvU291cmNlL1dlYktpdDIvQ2hhbmdlTG9nCisrKyBiL1Nv dXJjZS9XZWJLaXQyL0NoYW5nZUxvZwpAQCAtMSwzICsxLDIwIEBACisyMDEzLTA2LTI3ICBTZXJn aW8gVmlsbGFyIFNlbmluICA8c3ZpbGxhckBpZ2FsaWEuY29tPgorCisgICAgICAgIFtXSzJdW0dU S10gMTAwJSBDUFUgdXNhZ2UgcmVuZGVyaW5nIGEgcGFnZSB3aXRoIGFuaW1hdGlvbnMKKyAgICAg ICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTExNzcwMworCisgICAg ICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgIFVuY29uZGl0aW9uYWxs eSB1cGRhdGUgdGhlIHZhbHVlIG9mIG1fbGFzdEZsdXNoVGltZSBpbgorICAgICAgICA6OmZsdXNo QW5kUmVuZGVyTGF5ZXJzLiBCeSBkb2luZyB0aGF0IHdlIGF2b2lkIGEgbmV2ZXJlbmRpbmcKKyAg ICAgICAgc2VxdWVuY2Ugb2YgbGF5ZXIgZmx1c2hlcyB0aGF0IGhhcHBlbmVkIHdoZW4gbV9sYXN0 Rmx1c2hUaW1lPT0wCisgICAgICAgIGFuZCBmbHVzaFBlbmRpbmdMYXllckNoYW5nZXMoKSB3YXMg ZmFsc2UuIEluIHRoYXQgc2l0dWF0aW9uIHRoZQorICAgICAgICB2YWx1ZSBtX2xhc3RGbHVzaFRp bWUgd2FzIG5ldmVyIHVwZGF0ZWQgc28gYSBuZXcgZmx1c2ggd2FzIGFsd2F5cworICAgICAgICBy ZXNjaGVkdWxlZC4KKworICAgICAgICAqIFdlYlByb2Nlc3MvV2ViUGFnZS9ndGsvTGF5ZXJUcmVl SG9zdEd0ay5jcHA6CisgICAgICAgIChXZWJLaXQ6OkxheWVyVHJlZUhvc3RHdGs6OmZsdXNoQW5k UmVuZGVyTGF5ZXJzKToKKwogMjAxMy0wNi0yMCAgU2VyZ2lvIENvcnJlaWEgIDxzZXJnaW8uY29y cmVpYUBvcGVuYm9zc2Eub3JnPgogCiAgICAgICAgIFtXSzJdIExvb3BpbmcgZm9yIEVJTlRSIG9u IGNsb3NlKCkgaXMgaW5jb3JyZWN0IGZvciBMaW51eCwgYXQgbGVhc3QKZGlmZiAtLWdpdCBhL1Nv dXJjZS9XZWJLaXQyL1dlYlByb2Nlc3MvV2ViUGFnZS9ndGsvTGF5ZXJUcmVlSG9zdEd0ay5jcHAg Yi9Tb3VyY2UvV2ViS2l0Mi9XZWJQcm9jZXNzL1dlYlBhZ2UvZ3RrL0xheWVyVHJlZUhvc3RHdGsu Y3BwCmluZGV4IDA0Njc5YmQ4ZDNmZDhmZDE3YWQ0ZTlhNTM0MzI2YTAzODJjNzI4Y2UuLmQzMDZl YTc4NTAyM2EyOWY4YjAwMjJiMWVhNWJmODgwNTc4NDUwYzIgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9X ZWJLaXQyL1dlYlByb2Nlc3MvV2ViUGFnZS9ndGsvTGF5ZXJUcmVlSG9zdEd0ay5jcHAKKysrIGIv U291cmNlL1dlYktpdDIvV2ViUHJvY2Vzcy9XZWJQYWdlL2d0ay9MYXllclRyZWVIb3N0R3RrLmNw cApAQCAtMzYwLDEwICszNjAsMTAgQEAgdm9pZCBMYXllclRyZWVIb3N0R3RrOjpmbHVzaEFuZFJl bmRlckxheWVycygpCiAgICAgaWYgKCFjb250ZXh0IHx8ICFjb250ZXh0LT5tYWtlQ29udGV4dEN1 cnJlbnQoKSkKICAgICAgICAgcmV0dXJuOwogCisgICAgbV9sYXN0Rmx1c2hUaW1lID0gY3VycmVu dFRpbWUoKTsKICAgICBpZiAoIWZsdXNoUGVuZGluZ0xheWVyQ2hhbmdlcygpKQogICAgICAgICBy ZXR1cm47CiAKLSAgICBtX2xhc3RGbHVzaFRpbWUgPSBjdXJyZW50VGltZSgpOwogICAgIC8vIE91 ciBtb2RlbCBpcyB2ZXJ5IHNpbXBsZS4gV2UgYWx3YXlzIGNvbXBvc2l0ZSBhbmQgcmVuZGVyIHRo ZSB0cmVlIGltbWVkaWF0ZWx5IGFmdGVyIHVwZGF0aW5nIGl0LgogICAgIGNvbXBvc2l0ZUxheWVy c1RvQ29udGV4dCgpOwogCg==