132884 2014-05-13 14:51:16 -0700 [New multicolumn] Spin in RenderMultiColumnSet::repaintFlowThreadContent() 2014-05-16 01:29:12 -0700 1 1 1 Unclassified WebKit Layout and Rendering 528+ (Nightly build) Unspecified Unspecified RESOLVED FIXED http://en.wikipedia.org/wiki/Ch%C3%A2teau_de_Chambord InRadar P2 Normal --- 1 jonlee hyatt abucur bdakin commit-queue esprehn+autocc glenn kondapallykalyan simon.fraser thorton webkit-bug-importer oldest_to_newest 1009085 0 jonlee 2014-05-13 14:51:16 -0700 <rdar://problem/16828118> We're seeing hangs: 967 WebCore::RenderFlowThread::repaintRectangleInRegions(WebCore::LayoutRect const&) const (in WebCore) + 204 [0x192ea066c] 466 WebCore::RenderMultiColumnSet::repaintFlowThreadContent(WebCore::LayoutRect const&) (in WebCore) + 788 [0x192ede774] on app.ft.com, http://en.wikipedia.org/wiki/Château_de_Chambord, http://en.m.wikipedia.org/wiki/IAC/InterActiveCorp', on various iOS devices 1009086 1 jonlee 2014-05-13 14:51:32 -0700 Simon believes it may be related to bug 132725. 1009355 2 simon.fraser 2014-05-14 15:58:42 -0700 More context: 967 WebCore::RenderBlockFlow::layoutSpecialExcludedChild(bool) (in WebCore) + 192 [0x192e739c4] 967 WebCore::RenderMultiColumnFlowThread::layout() (in WebCore) + 76 [0x192eda680] 967 WebCore::RenderFlowThread::layout() (in WebCore) + 280 [0x192e9fae8] 967 WebCore::RenderBlock::layout() (in WebCore) + 68 [0x1923b1634] 967 WebCore::RenderBlockFlow::layoutBlock(bool, WebCore::LayoutUnit) (in WebCore) + 756 [0x192e68be0] 967 WebCore::RenderBlockFlow::layoutBlockChildren(bool, WebCore::LayoutUnit&) (in WebCore) + 520 [0x192e69920] 967 WebCore::RenderBlockFlow::layoutBlockChild(WebCore::RenderBox&, WebCore::RenderBlockFlow::MarginInfo&, WebCore::LayoutUnit&, WebCore::LayoutUnit&) (in WebCore) + 812 [0x192e6a830] 967 WebCore::RenderBlock::layout() (in WebCore) + 68 [0x1923b1634] 967 WebCore::RenderBlockFlow::layoutBlock(bool, WebCore::LayoutUnit) (in WebCore) + 756 [0x192e68be0] 967 WebCore::RenderBlockFlow::layoutBlockChildren(bool, WebCore::LayoutUnit&) (in WebCore) + 520 [0x192e69920] 967 WebCore::RenderBlockFlow::layoutBlockChild(WebCore::RenderBox&, WebCore::RenderBlockFlow::MarginInfo&, WebCore::LayoutUnit&, WebCore::LayoutUnit&) (in WebCore) + 1820 [0x192e6ac20] 967 WebCore::RenderObject::repaint() const (in WebCore) + 132 [0x192ee9264] 967 WebCore::RenderObject::repaintUsingContainer(WebCore::RenderLayerModelObject const*, WebCore::LayoutRect const&, bool) const (in WebCore) + 72 [0x192ee8fe4] 967 WebCore::RenderFlowThread::repaintRectangleInRegions(WebCore::LayoutRect const&) const (in WebCore) + 204 [0x192ea066c] 466 WebCore::RenderMultiColumnSet::repaintFlowThreadContent(WebCore::LayoutRect const&) (in WebCore) + 788 [0x192ede774] ! 175 WebCore::RenderMultiColumnSet::columnRectAt(unsigned int) const (in WebCore) + 68 [0x192edd75c] ! : 124 WebCore::RenderBoxModelObject::paddingBefore() const (in WebCore) + 52 [0x1923b1188] ! : | 99 WebCore::RenderBoxModelObject::computedCSSPadding(WebCore::Length const&) const (in WebCore) + 92 [0x192e83f34] ! : | + 91 WebCore::RenderBoxModelObject::computedCSSPadding(WebCore::Length const&) const (in WebCore) + 92 [0x192e83f34] ! : | + ! 91 WebCore::minimumValueForLength(WebCore::Length const&, WebCore::LayoutUnit, WebCore::RenderView*, bool) (in WebCore) + 396,36,... [0x1923a1070,0x1923a0f08,...] ! : | + 7 WebCore::minimumValueForLength(WebCore::Length const&, WebCore::LayoutUnit, WebCore::RenderView*, bool) (in WebCore) + 408 [0x1923a107c] 1009403 3 231480 simon.fraser 2014-05-14 19:30:51 -0700 Created attachment 231480 Patch 1009405 4 simon.fraser 2014-05-14 19:34:22 -0700 http://trac.webkit.org/changeset/168882 1009550 5 abucur 2014-05-15 08:52:35 -0700 Just a hunch: this may be related with https://codereview.chromium.org/263203003/ 1009728 6 hyatt 2014-05-16 01:29:12 -0700 Yeah seems related. 231480 2014-05-14 19:30:51 -0700 2014-05-14 19:32:24 -0700 Patch bug-132884-20140514193028.patch text/plain 2018 simon.fraser U3VidmVyc2lvbiBSZXZpc2lvbjogMTY4ODY0CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9D aGFuZ2VMb2cgYi9Tb3VyY2UvV2ViQ29yZS9DaGFuZ2VMb2cKaW5kZXggZmRiZjJmMjIzZjY2MDFj NDNiMDU4MmYxMjM1YzQ1N2ZiOTk0MzI5NC4uODdjYjk1YzU1YmJiYjg1YmM5MDcwYWQ3NjAwMmE0 ZDg5MzBmOGZkMCAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCisrKyBiL1Nv dXJjZS9XZWJDb3JlL0NoYW5nZUxvZwpAQCAtMSwzICsxLDIxIEBACisyMDE0LTA1LTE0ICBTaW1v biBGcmFzZXIgIDxzaW1vbi5mcmFzZXJAYXBwbGUuY29tPgorCisgICAgICAgIFtOZXcgbXVsdGlj b2x1bW5dIFNwaW4gaW4gUmVuZGVyTXVsdGlDb2x1bW5TZXQ6OnJlcGFpbnRGbG93VGhyZWFkQ29u dGVudCgpCisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0x MzI4ODQKKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICBP biBpT1MsIHdpa2lwZWRpYSBwYWdlcyBjYW4gaGFuZyB1bmRlciBSZW5kZXJNdWx0aUNvbHVtblNl dDo6cmVwYWludEZsb3dUaHJlYWRDb250ZW50KCkuCisgICAgICAgIEl0IGFwcGVhcnMgdGhhdCBj b21wdXRlZENvbHVtbkhlaWdodCBpcyBzZXQgdG8gMCBpbiBwcmVwYXJlRm9yTGF5b3V0LCBidXQg bGF5b3V0IG5ldmVyIGhhcHBlbnMKKyAgICAgICAgb24gdGhlIFJlbmRlck11bHRpQ29sdW1uU2V0 IGluIHNvbWUgY2FzZXMsIGxlYXZpbmcgdGhlIGNvbHVtbiBoZWlnaHQgc2V0IHRvIHplcm8uCisg ICAgICAgIFRoaXMgY2F1c2VkIGNvbHVtbkluZGV4QXRPZmZzZXQoKSB0byByZXR1cm4gYmFkIHZh bHVlcywgd2hpY2ggcmVzdWx0ZWQgaW4gdmVyeSBsb25nIGxvb3BzCisgICAgICAgIGluIHJlcGFp bnRGbG93VGhyZWFkQ29udGVudCgpLgorICAgICAgICAKKyAgICAgICAgVGhpcyBmaXggaXMgYSBz dG9wLWdhcC4KKworICAgICAgICAqIHJlbmRlcmluZy9SZW5kZXJNdWx0aUNvbHVtblNldC5jcHA6 CisgICAgICAgIChXZWJDb3JlOjpSZW5kZXJNdWx0aUNvbHVtblNldDo6Y29sdW1uSW5kZXhBdE9m ZnNldCk6CisKIDIwMTQtMDUtMTQgIEJyYWR5IEVpZHNvbiAgPGJlaWRzb25AYXBwbGUuY29tPgog CiAgICAgICAgIEltcGxlbWVudCBOU1NoYXJpbmdTZXJ2aWNlRGVsZWdhdGUgbWV0aG9kICJ0cmFu c2l0aW9uSW1hZ2VGb3JTaGFyZUl0ZW0iCmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9yZW5k ZXJpbmcvUmVuZGVyTXVsdGlDb2x1bW5TZXQuY3BwIGIvU291cmNlL1dlYkNvcmUvcmVuZGVyaW5n L1JlbmRlck11bHRpQ29sdW1uU2V0LmNwcAppbmRleCA4MDJjNjQwNjNhNmY5NjE0ODBjNTE0ODQ2 YTM4Njg5NTBlYWQ3YTExLi40MWZlOTc1NTA0MDViMjU0NDhmYWJlZGZkMDQyNzU2NjRkYjE1NDQ0 IDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViQ29yZS9yZW5kZXJpbmcvUmVuZGVyTXVsdGlDb2x1bW5T ZXQuY3BwCisrKyBiL1NvdXJjZS9XZWJDb3JlL3JlbmRlcmluZy9SZW5kZXJNdWx0aUNvbHVtblNl dC5jcHAKQEAgLTQ4OCw2ICs0ODgsMTAgQEAgdW5zaWduZWQgUmVuZGVyTXVsdGlDb2x1bW5TZXQ6 OmNvbHVtbkluZGV4QXRPZmZzZXQoTGF5b3V0VW5pdCBvZmZzZXQsIENvbHVtbkluZGUKICAgICAg ICAgICAgIHJldHVybiBjb2x1bW5Db3VudCgpIC0gMTsKICAgICB9CiAKKyAgICAvLyBTb21ldGlt ZXMgY29tcHV0ZWRDb2x1bW5IZWlnaHQoKSBpcyAwIGhlcmU6IHNlZSBodHRwczovL2J1Z3Mud2Vi a2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MTMyODg0CisgICAgaWYgKCFjb21wdXRlZENvbHVtbkhl aWdodCgpKQorICAgICAgICByZXR1cm4gMDsKKwogICAgIC8vIEp1c3QgZGl2aWRlIGJ5IHRoZSBj b2x1bW4gaGVpZ2h0IHRvIGRldGVybWluZSB0aGUgY29ycmVjdCBjb2x1bW4uCiAgICAgcmV0dXJu IHN0YXRpY19jYXN0PGZsb2F0PihvZmZzZXQgLSBmbG93VGhyZWFkTG9naWNhbFRvcCkgLyBjb21w dXRlZENvbHVtbkhlaWdodCgpOwogfQo=