103508 2012-11-28 05:24:40 -0800 [EFL][WK2] input-with-validation-without-shadow.html makes following tests crash after r135935 2012-11-28 21:03:23 -0800 1 1 1 Unclassified WebKit WebKit EFL 528+ (Nightly build) Unspecified Unspecified RESOLVED FIXED P2 Normal --- 1 jussi.kukkonen yael lucas.de.marchi shinyak tkent webkit.review.bot oldest_to_newest 777785 0 jussi.kukkonen 2012-11-28 05:24:40 -0800 The patch in bug 102833 seems to have started to "crash" fast/dom/shadow/input-with-validation.html (really PROCESS UNRESPONSIVE). It won't do it alone but e.g. running these tests will make the last one fail with "webprocess crashed" fast/dom/shadow/input-with-validation-without-shadow.html failed fast/dom/shadow/input-with-validation.html failed unexpectedly 777813 1 yael 2012-11-28 05:48:56 -0800 We also have these bugs to track the crash: https://bugs.webkit.org/show_bug.cgi?id=98026 https://bugs.webkit.org/show_bug.cgi?id=99886 777874 2 yael 2012-11-28 06:44:20 -0800 My first observation is that removing fast/dom/shadow/input-with-validation.html from the run list causes insertion-point-list-menu-crash.html to timeout, but removing input-with-validation-without-shadow.html from the list results in all the tests passing. 777893 3 jussi.kukkonen 2012-11-28 07:22:33 -0800 (In reply to comment #1) > We also have these bugs to track the crash: > https://bugs.webkit.org/show_bug.cgi?id=98026 > https://bugs.webkit.org/show_bug.cgi?id=99886 These have different symptoms. I don't see why they'd be the same bug. (In reply to comment #2) > My first observation is that removing fast/dom/shadow/input-with-validation.html from the run list causes insertion-point-list-menu-crash.html to timeout, but removing input-with-validation-without-shadow.html from the list results in all the tests passing. Well, I'll skip input-with-validation-without-shadow.html for now then. Can't keep the bot red all day. 778406 4 yael 2012-11-28 15:24:41 -0800 When we run the test input-with-validation-without-shadow.html, we load immediately after that the ref test input-with-validation-without-shadow-expected.html . The test sends a click event to the form control to submit it. At this point we start page transition and lock the tree. The page transition never finishes. When we try to dump the pixels for the test, the tree is locked. It becomes unlocked only after the timeout expires and we load about:blank. 778570 5 176620 yael 2012-11-28 18:08:46 -0800 Created attachment 176620 Patch Avoid submitting the ref test's form as discussed on iRC. 778641 6 shinyak 2012-11-28 20:58:49 -0800 Hi, I'm the person who wrote the test. I'm sorry that my wrong test has bothered you. Right, submit() should not be called in input-with-validation-without-shadow-expected.html . Thank you for fixing it! 778642 7 176620 tkent 2012-11-28 20:59:37 -0800 Comment on attachment 176620 Patch ok 778644 8 176620 webkit.review.bot 2012-11-28 21:03:18 -0800 Comment on attachment 176620 Patch Clearing flags on attachment: 176620 Committed r136088: <http://trac.webkit.org/changeset/136088> 778645 9 webkit.review.bot 2012-11-28 21:03:23 -0800 All reviewed patches have been landed. Closing bug. 176620 2012-11-28 18:08:46 -0800 2012-11-28 21:03:18 -0800 Patch 103508.patch text/plain 2443 yael SW5kZXg6IExheW91dFRlc3RzL3BsYXRmb3JtL2VmbC13azIvVGVzdEV4cGVjdGF0aW9ucwo9PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09Ci0tLSBMYXlvdXRUZXN0cy9wbGF0Zm9ybS9lZmwtd2syL1Rlc3RFeHBlY3RhdGlvbnMJ KHJldmlzaW9uIDEzNjA3OSkKKysrIExheW91dFRlc3RzL3BsYXRmb3JtL2VmbC13azIvVGVzdEV4 cGVjdGF0aW9ucwkod29ya2luZyBjb3B5KQpAQCAtMzUxLDcgKzM1MSw2IEBACiB3ZWJraXQub3Jn L2IvMTAzNDY4IGNzczMvY2FsYy90YWJsZS1lbXB0eS1jZWxscy5odG1sIFsgSW1hZ2VPbmx5RmFp bHVyZSBQYXNzIF0KIHdlYmtpdC5vcmcvYi8xMDM0NjggY3NzMy92aWV3cG9ydC1wZXJjZW50YWdl LWxlbmd0aHMvY3NzMy12aWV3cG9ydC1wZXJjZW50YWdlLWxlbmd0aHMtdmgtYWJzb2x1dGUuaHRt bCBbIEltYWdlT25seUZhaWx1cmUgUGFzcyBdCiB3ZWJraXQub3JnL2IvMTAzNDY4IGZhc3QvY3Nz L3N0aWNreS9zdGlja3ktYXMtcG9zaXRpb25pbmctY29udGFpbmVyLmh0bWwgWyBJbWFnZU9ubHlG YWlsdXJlIFBhc3MgXQotd2Via2l0Lm9yZy9iLzEwMzQ2OCBmYXN0L2RvbS9zaGFkb3cvaW5wdXQt d2l0aC12YWxpZGF0aW9uLXdpdGhvdXQtc2hhZG93Lmh0bWwgWyBJbWFnZU9ubHlGYWlsdXJlIFBh c3MgXQogd2Via2l0Lm9yZy9iLzEwMzQ2OCBmYXN0L2Zvcm1zL2ZpbGUvZmlsZS1yZXNldC1pbi1j aGFuZ2UuaHRtbCBbIEltYWdlT25seUZhaWx1cmUgUGFzcyBdCiB3ZWJraXQub3JnL2IvMTAzNDY4 IGZhc3QvZm9ybXMvZmlsZS9oaWRkZW4taW5wdXQtZmlsZS5odG1sIFsgSW1hZ2VPbmx5RmFpbHVy ZSBQYXNzIF0KIHdlYmtpdC5vcmcvYi8xMDM0NjggZmFzdC9pbWFnZXMvaW1hZ2VtYXAtZm9jdXMt cmluZy16b29tLXN0eWxlLmh0bWwgWyBJbWFnZU9ubHlGYWlsdXJlIFBhc3MgXQpJbmRleDogTGF5 b3V0VGVzdHMvZmFzdC9kb20vc2hhZG93L2lucHV0LXdpdGgtdmFsaWRhdGlvbi13aXRob3V0LXNo YWRvdy1leHBlY3RlZC5odG1sCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIExheW91dFRlc3RzL2Zhc3QvZG9tL3No YWRvdy9pbnB1dC13aXRoLXZhbGlkYXRpb24td2l0aG91dC1zaGFkb3ctZXhwZWN0ZWQuaHRtbAko cmV2aXNpb24gMTM1NjMzKQorKysgTGF5b3V0VGVzdHMvZmFzdC9kb20vc2hhZG93L2lucHV0LXdp dGgtdmFsaWRhdGlvbi13aXRob3V0LXNoYWRvdy1leHBlY3RlZC5odG1sCSh3b3JraW5nIGNvcHkp CkBAIC0xMCw3ICsxMCw2IEBACiA8c2NyaXB0PgogdmFyIGlucHV0ID0gZG9jdW1lbnQuZ2V0RWxl bWVudEJ5SWQoJ3Bhc3N3b3JkJyk7CiAKLWRvY3VtZW50LmdldEVsZW1lbnRCeUlkKCdzdWJtaXQn KS5jbGljaygpOwogaWYgKHdpbmRvdy50ZXN0UnVubmVyKSB7CiAgICAgdGVzdFJ1bm5lci53YWl0 VW50aWxEb25lKCk7CiAgICAgc2V0VGltZW91dChmdW5jdGlvbigpIHsKSW5kZXg6IExheW91dFRl c3RzL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBMYXlvdXRUZXN0cy9DaGFuZ2VMb2cJKHJldmlz aW9uIDEzNjA3OSkKKysrIExheW91dFRlc3RzL0NoYW5nZUxvZwkod29ya2luZyBjb3B5KQpAQCAt MSwzICsxLDE3IEBACisyMDEyLTExLTI4ICBZYWVsIEFoYXJvbiAgPHlhZWwuYWhhcm9uQGludGVs LmNvbT4KKworICAgICAgICBbRUZMXVtXSzJdIGlucHV0LXdpdGgtdmFsaWRhdGlvbi13aXRob3V0 LXNoYWRvdy5odG1sIG1ha2VzIGZvbGxvd2luZyB0ZXN0cyBjcmFzaCBhZnRlciByMTM1OTM1Cisg ICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0xMDM1MDgKKwor ICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICBEbyBub3Qgc3Vi bWl0IHRoZSBmb3JtIGluIHRoZSByZWYgdGVzdCwgYXMgaXQgaXMgYWxzbyBub3Qgc3VibWl0dGVk IGJ5IHRoZQorICAgICAgICBvcmlnaW5hbCB0ZXN0LiBUaGlzIGFsc28gZml4ZXMgdGhlIHRpbWVv dXQgaW4gRUZMJ3MgdGVzdHMuCisgICAgICAgIFJlbW92ZSB0aGUgdGVzdCBmcm9tIFRlc3RFZXhw ZWN0YXRpb25zIGFzIHRoZSB0ZXN0IGlzIHBhc3Npbmcgbm93LgorCisgICAgICAgICogZmFzdC9k b20vc2hhZG93L2lucHV0LXdpdGgtdmFsaWRhdGlvbi13aXRob3V0LXNoYWRvdy1leHBlY3RlZC5o dG1sOgorICAgICAgICAqIHBsYXRmb3JtL2VmbC13azIvVGVzdEV4cGVjdGF0aW9uczoKKwogMjAx Mi0xMS0yOCAgS2VuaWNoaSBJc2hpYmFzaGkgIDxiYXNoaUBjaHJvbWl1bS5vcmc+CiAKICAgICAg ICAgU3R5bGVSZXNvbHZlciBzaG91bGQgbm90IHNldCBOYU4gdG8gZm9udCBzaXplCg==