77183 2012-01-26 20:57:23 -0800 [Chromium] Layout Test svg/as-object/nested-embedded-svg-size-changes.html times out 2012-04-05 13:03:49 -0700 1 1 1 Unclassified WebKit Tools / Tests 528+ (Nightly build) All All RESOLVED FIXED P2 Normal --- 77736 1 yuzo zimmermann arv jchaffraix schenney zimmermann oldest_to_newest 543796 0 yuzo 2012-01-26 20:57:23 -0800 The following layout test times out (or passes) on Chromium. svg/as-object/nested-embedded-svg-size-changes.html 543930 1 zimmermann 2012-01-27 03:07:54 -0800 My immediate thought was that its related to my FrameView patch, but it's a JS error: CONSOLE MESSAGE: line 9: Uncaught TypeError: Cannot read property 'defaultView' of undefined Maybe its V8 specific, I've never seen this fail before - "document" is null, at some point. 548745 2 zimmermann 2012-02-03 08:29:59 -0800 Bug 77736 might fix this, once landed. 551313 3 jchaffraix 2012-02-07 13:29:43 -0800 FYI, I marked the test as slow to see if it helped and it really did not. We must be hitting the exception fairly regularly and not call notifyDone(). 551931 4 zimmermann 2012-02-08 02:27:16 -0800 (In reply to comment #3) > FYI, I marked the test as slow to see if it helped and it really did not. We must be hitting the exception fairly regularly and not call notifyDone(). *grml* It really needs to be debugged on Chrome/V8 - it only seems to happen there. 552990 5 zimmermann 2012-02-09 02:44:04 -0800 Not related to bug 77736 - it's a chromium/v8 specific problem. 552991 6 zimmermann 2012-02-09 02:45:18 -0800 Oops, wrong bug, reverting. Bug 78026 covers the same regression for Qt, which uses jsc. 553017 7 zimmermann 2012-02-09 03:26:45 -0800 Uploaded a fix to bug 78026 that should fix chromium as well. *** This bug has been marked as a duplicate of bug 78026 *** 553077 8 zimmermann 2012-02-09 05:05:52 -0800 Reopening, it seems to be an individual failure, it doesn't fail on Qt at present. 553084 9 126286 zimmermann 2012-02-09 05:26:35 -0800 Created attachment 126286 Patch 553087 10 126286 zherczeg 2012-02-09 05:36:29 -0800 Comment on attachment 126286 Patch r=me 553092 11 zimmermann 2012-02-09 05:40:10 -0800 Committed r107227: <http://trac.webkit.org/changeset/107227> 553106 12 126286 zimmermann 2012-02-09 06:26:26 -0800 Comment on attachment 126286 Patch It got fixed in trunk! Manually verified using the latest layout test results.zip from the cr-linux bot, and discussed with podivilov on IRC. 553107 13 zimmermann 2012-02-09 06:26:46 -0800 Removed the TIMEOUT from the chromium expectations in r107231. Closing bug. 596829 14 schenney 2012-04-05 13:03:49 -0700 Committed r113362: <http://trac.webkit.org/changeset/113362> 126286 2012-02-09 05:26:35 -0800 2012-02-09 06:26:26 -0800 Patch bug-77183-20120209142631.patch text/plain 1766 zimmermann U3VidmVyc2lvbiBSZXZpc2lvbjogMTA3MjI1CmRpZmYgLS1naXQgYS9MYXlvdXRUZXN0cy9DaGFu Z2VMb2cgYi9MYXlvdXRUZXN0cy9DaGFuZ2VMb2cKaW5kZXggN2VjY2UyNjY0ZDVmNzhlMGNlNjZi YzBkMTQwZWRhMGYyMGUzNDM4Yi4uZjU3NTA1ODU2NzQ5OGZhODFlNDMwYmQ0NDcyN2IwZGFiNGZl ZmI0OCAxMDA2NDQKLS0tIGEvTGF5b3V0VGVzdHMvQ2hhbmdlTG9nCisrKyBiL0xheW91dFRlc3Rz L0NoYW5nZUxvZwpAQCAtMSwzICsxLDE1IEBACisyMDEyLTAyLTA5ICBOaWtvbGFzIFppbW1lcm1h bm4gIDxuemltbWVybWFubkByaW0uY29tPgorCisgICAgICAgIFtDaHJvbWl1bV0gTGF5b3V0IFRl c3Qgc3ZnL2FzLW9iamVjdC9uZXN0ZWQtZW1iZWRkZWQtc3ZnLXNpemUtY2hhbmdlcy5odG1sIHRp bWVzIG91dAorICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9 NzcxODMKKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICBB bm90aGVyIGF0dGVtcHQgYXQgZml4aW5nIHN2Zy9hcy1vYmplY3QvbmVzdGVkLWVtYmVkZGVkLXN2 Zy1zaXplLWNoYW5nZXMuaHRtbCwKKyAgICAgICAgYnkgZ3Vlc3NpbmcgZnJvbSB0aGUgSlMgdHJh Y2UsIHdoZXJlIGl0IG1pZ2h0IGhhcHBlbmVkLgorCisgICAgICAgICogc3ZnL2FzLW9iamVjdC9u ZXN0ZWQtZW1iZWRkZWQtc3ZnLXNpemUtY2hhbmdlcy5odG1sOgorCiAyMDEyLTAyLTA5ICBBbGV4 ZWkgRmlsaXBwb3YgIDxhbGV4ZWlmQGNocm9taXVtLm9yZz4KIAogICAgICAgICBXZWJJbnNwZWN0 b3I6IFNob3cgcGVyY2VudGFnZSBieSBkZWZhdWx0IGluIGhlYXAgcHJvZmlsZXIuCmRpZmYgLS1n aXQgYS9MYXlvdXRUZXN0cy9zdmcvYXMtb2JqZWN0L25lc3RlZC1lbWJlZGRlZC1zdmctc2l6ZS1j aGFuZ2VzLmh0bWwgYi9MYXlvdXRUZXN0cy9zdmcvYXMtb2JqZWN0L25lc3RlZC1lbWJlZGRlZC1z dmctc2l6ZS1jaGFuZ2VzLmh0bWwKaW5kZXggNzNiYWJjY2QxNDNhOGFkYTE2ZjM1MmVhY2Y4NjVh NmY2MmUxODVjMi4uNjc4MWNjNzhlOTVhZWVjYjE0ODAzMTRjMzE1MjdiMzlhZjY3NmQzOCAxMDA2 NDQKLS0tIGEvTGF5b3V0VGVzdHMvc3ZnL2FzLW9iamVjdC9uZXN0ZWQtZW1iZWRkZWQtc3ZnLXNp emUtY2hhbmdlcy5odG1sCisrKyBiL0xheW91dFRlc3RzL3N2Zy9hcy1vYmplY3QvbmVzdGVkLWVt YmVkZGVkLXN2Zy1zaXplLWNoYW5nZXMuaHRtbApAQCAtMzEsOCArMzEsMTIgQEAKICAgICBmdW5j dGlvbiByZXF1ZXN0U2l6ZUNoYW5nZSgpIHsKICAgICAgICAgb2JqZWN0MSA9IGRvY3VtZW50Lmdl dEVsZW1lbnRzQnlUYWdOYW1lKCJvYmplY3QiKVswXTsKICAgICAgICAgdmFyIGh0bWxEb2MgPSBv YmplY3QxLmNvbnRlbnREb2N1bWVudDsKLSAgICAgICAgdmFyIGh0bWxXaW5kb3cgPSBodG1sRG9j LmRlZmF1bHRWaWV3OworICAgICAgICBpZiAoIWh0bWxEb2MpIHsKKyAgICAgICAgICAgIHNldFRp bWVvdXQocmVxdWVzdFNpemVDaGFuZ2UsIDUwKTsKKyAgICAgICAgICAgIHJldHVybjsKKyAgICAg ICAgfQogCisgICAgICAgIHZhciBodG1sV2luZG93ID0gaHRtbERvYy5kZWZhdWx0VmlldzsKICAg ICAgICAgLy8gV2FpdCB1bnRpbCB0aGUgZW1iZWRkZWQgU1ZHIChvZiB0aGUgZW1iZWRkZWQgSFRN TCB0aGF0IHdlJ3JlIGxvYWRpbmcpLCBpcyBsb2FkZWQuCiAgICAgICAgIGlmIChodG1sV2luZG93 LmNhbkNoYW5nZVNpemUpIHsKICAgICAgICAgICAgIGh0bWxXaW5kb3cuY2hhbmdlU2l6ZSgpOwo=