117070 2013-05-31 06:12:54 -0700 [Qt][WK2] Newly added editing/selection/caret-after-keypress.html crashing 2014-02-03 03:25:57 -0800 1 1 1 Unclassified WebKit Tools / Tests 528+ (Nightly build) Unspecified Unspecified RESOLVED INVALID Qt P2 Normal --- 110742 42194 79668 1 zarvai eustas allan.jensen eustas jturcotte kadam rniwa oldest_to_newest 895867 0 zarvai 2013-05-31 06:12:54 -0700 crash log for WebProcess (pid <unknown>): STDOUT: <empty> STDERR: Received an invalid message "WebContextLegacyMessage.PostSynchronousMessage" from the web process. http://build.webkit.sed.hu/results/x86-32%20Linux%20Qt%20Release%20WebKit2/r151013%20%2835908%29/results.html backtrace https://gist.github.com/azbesthu/5684836 895871 1 eustas 2013-05-31 06:38:09 -0700 posting messages in WK2 test causes crash. I'll add TextExpectation. 895874 2 kadam 2013-05-31 06:41:31 -0700 It have been landed in http://trac.webkit.org/changeset/151014. :) 895877 3 203440 eustas 2013-05-31 06:51:08 -0700 Created attachment 203440 Patch 895880 4 203442 eustas 2013-05-31 06:53:55 -0700 Created attachment 203442 Patch 895881 5 eustas 2013-05-31 06:56:24 -0700 Well, I've moved expectation, because there is a meta-bug that is responsible for the whole a group of tests that fail on WK2. 895883 6 zarvai 2013-05-31 07:03:45 -0700 (In reply to comment #5) > Well, I've moved expectation, because there is a meta-bug that is responsible > for the whole a group of tests that fail on WK2. Yes, that sounds reasonable. Probably the change can be landed without review like most of the unreviewed gardening patches. 895896 7 eustas 2013-05-31 08:22:05 -0700 Committed r151015: <http://trac.webkit.org/changeset/151015> 895913 8 rniwa 2013-05-31 09:29:08 -0700 But why is this causing a crash? 896308 9 eustas 2013-06-02 23:39:26 -0700 In new test we use "eventSender.keyDown". As it is said in https://bugs.webkit.org/show_bug.cgi?id=42194 : "WebKitTestRunner needs an implementation of eventSender. Many tests fail because of this (hundreds)." What I see in stack trace: in WebPageProxy::didReceiveEvent in case of synchronous (test) keyboard event handling we face that type of first message in keyboard event queue is not the same as type of event we just appended to this queue. Assertion that type of last message in queue is the same as first message in queue means that we expect that queue is never longer that 1 item (at least for synchronous case). Stack trace has depth of 206 frames; there are 6 nested key events. 5 events are at "WebPageProxy.cpp:1395", invoked from handleKeyPressEvent. 6-th is at "WebPageProxy.cpp:1396", invoked from handleKeyReleaseEvent. So, when test reaches exit condition and begins unrolling, the problem occurs. I'm not sure, is it OK or NOK to nest eventSender.keyDown invocations. I'll prepare asynchronous test to see if will make Qt5-WK bot happy. (In reply to comment #8) > But why is this causing a crash? 896731 10 203650 eustas 2013-06-03 22:01:00 -0700 Created attachment 203650 Patch 975736 11 jturcotte 2014-02-03 03:25:57 -0800 === Bulk closing of Qt bugs === If you believe that this bug report is still relevant for a non-Qt port of webkit.org, please re-open it and remove [Qt] from the summary. If you believe that this is still an important QtWebKit bug, please fill a new report at https://bugreports.qt-project.org and add a link to this issue. See http://qt-project.org/wiki/ReportingBugsInQt for additional guidelines. 203440 2013-05-31 06:51:08 -0700 2013-05-31 06:53:48 -0700 Patch bug-117070-20130531174943.patch text/plain 1551 eustas U3VidmVyc2lvbiBSZXZpc2lvbjogMTUxMDE0CmRpZmYgLS1naXQgYS9MYXlvdXRUZXN0cy9DaGFu Z2VMb2cgYi9MYXlvdXRUZXN0cy9DaGFuZ2VMb2cKaW5kZXggMWI2NjFhNGYxYTZmMzFjZThkZDVk MWMyMjM2OGE5MWQyNTVmZGM1OC4uNTZlZDQxNTI1NmNkZTVhZGFmYjhjZTRiMzZiODk4Y2M2ZDQ0 NzVmZSAxMDA2NDQKLS0tIGEvTGF5b3V0VGVzdHMvQ2hhbmdlTG9nCisrKyBiL0xheW91dFRlc3Rz L0NoYW5nZUxvZwpAQCAtMSwzICsxLDE3IEBACisyMDEzLTA1LTMxICBFdWdlbmUgS2x5dWNobmlr b3YgIDxldXN0YXNAY2hyb21pdW0ub3JnPgorCisgICAgICAgIFtRdF1bV0syXSBOZXdseSBhZGRl ZCBlZGl0aW5nL3NlbGVjdGlvbi9jYXJldC1hZnRlci1rZXlwcmVzcy5odG1sIGNyYXNoaW5nCisg ICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0xMTcwNzAKKwor ICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICBXZWJLaXRUZXN0 UnVubmVyIG5lZWRzIGEgbW9yZS1jb21wbGV0ZSBpbXBsZW1lbnRhdGlvbiBvZiBldmVudFNlbmRl ci4KKyAgICAgICAgVW50aWwgdGhlbiBjYXJldC1hZnRlci1rZXlwcmVzcyB0ZXN0IGlzIGRpc2Fi bGVkCisKKyAgICAgICAgU2VlIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9p ZD00MjE5NAorCisgICAgICAgICogcGxhdGZvcm0vcXQtNS4wLXdrMi9UZXN0RXhwZWN0YXRpb25z OgorCiAyMDEzLTA1LTMxICBab2x0YW4gQXJ2YWkgIDx6YXJ2YWlAaW5mLnUtc3plZ2VkLmh1Pgog CiAgICAgICAgIFtRdF0gVW5yZXZpZXdlZCBnYXJkZW5pbmcuCmRpZmYgLS1naXQgYS9MYXlvdXRU ZXN0cy9wbGF0Zm9ybS9xdC01LjAtd2syL1Rlc3RFeHBlY3RhdGlvbnMgYi9MYXlvdXRUZXN0cy9w bGF0Zm9ybS9xdC01LjAtd2syL1Rlc3RFeHBlY3RhdGlvbnMKaW5kZXggYTFiNTM5ODEyZTAyNjZm MDI5ZGQ3NzU2NzMyMzdiZTE4NzY2ZmY1YS4uMmM2MmFiODRiYWRlM2Q0ZDVlNGRlYWIwNjQwZGM3 MGMyMjI4ZWIzMCAxMDA2NDQKLS0tIGEvTGF5b3V0VGVzdHMvcGxhdGZvcm0vcXQtNS4wLXdrMi9U ZXN0RXhwZWN0YXRpb25zCisrKyBiL0xheW91dFRlc3RzL3BsYXRmb3JtL3F0LTUuMC13azIvVGVz dEV4cGVjdGF0aW9ucwpAQCAtMzksNiArMzksNyBAQCBlZGl0aW5nL3NlbGVjdGlvbi9jb250ZXh0 LW1lbnUtb24tdGV4dC5odG1sCiBlZGl0aW5nL3NlbGVjdGlvbi9kb3VibGVjbGljay13aGl0ZXNw YWNlLWNyYXNoLmh0bWwKIGVkaXRpbmcvc2VsZWN0aW9uL2RvdWJsZWNsaWNrLXdoaXRlc3BhY2Ut aW1nLWNyYXNoLmh0bWwKIGVkaXRpbmcvc2VsZWN0aW9uL2RvdWJsZWNsaWNrLXdoaXRlc3BhY2Uu aHRtbAorZWRpdGluZy9zZWxlY3Rpb24vY2FyZXQtYWZ0ZXIta2V5cHJlc3MuaHRtbAogZWRpdGlu Zy9zZWxlY3Rpb24vZW1wdHktY2VsbC1yaWdodC1jbGljay5odG1sCiBlZGl0aW5nL3NlbGVjdGlv bi9tb3ZlLWJlZ2luLWVuZC5odG1sCiBlZGl0aW5nL3NlbGVjdGlvbi9zZWxlY3Qtb3V0LW9mLWVk aXRhYmxlLmh0bWwK 203442 2013-05-31 06:53:55 -0700 2013-06-03 22:00:51 -0700 Patch bug-117070-20130531175229.patch text/plain 1967 eustas U3VidmVyc2lvbiBSZXZpc2lvbjogMTUxMDE0CmRpZmYgLS1naXQgYS9MYXlvdXRUZXN0cy9DaGFu Z2VMb2cgYi9MYXlvdXRUZXN0cy9DaGFuZ2VMb2cKaW5kZXggMWI2NjFhNGYxYTZmMzFjZThkZDVk MWMyMjM2OGE5MWQyNTVmZGM1OC4uNTZlZDQxNTI1NmNkZTVhZGFmYjhjZTRiMzZiODk4Y2M2ZDQ0 NzVmZSAxMDA2NDQKLS0tIGEvTGF5b3V0VGVzdHMvQ2hhbmdlTG9nCisrKyBiL0xheW91dFRlc3Rz L0NoYW5nZUxvZwpAQCAtMSwzICsxLDE3IEBACisyMDEzLTA1LTMxICBFdWdlbmUgS2x5dWNobmlr b3YgIDxldXN0YXNAY2hyb21pdW0ub3JnPgorCisgICAgICAgIFtRdF1bV0syXSBOZXdseSBhZGRl ZCBlZGl0aW5nL3NlbGVjdGlvbi9jYXJldC1hZnRlci1rZXlwcmVzcy5odG1sIGNyYXNoaW5nCisg ICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0xMTcwNzAKKwor ICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICBXZWJLaXRUZXN0 UnVubmVyIG5lZWRzIGEgbW9yZS1jb21wbGV0ZSBpbXBsZW1lbnRhdGlvbiBvZiBldmVudFNlbmRl ci4KKyAgICAgICAgVW50aWwgdGhlbiBjYXJldC1hZnRlci1rZXlwcmVzcyB0ZXN0IGlzIGRpc2Fi bGVkCisKKyAgICAgICAgU2VlIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9p ZD00MjE5NAorCisgICAgICAgICogcGxhdGZvcm0vcXQtNS4wLXdrMi9UZXN0RXhwZWN0YXRpb25z OgorCiAyMDEzLTA1LTMxICBab2x0YW4gQXJ2YWkgIDx6YXJ2YWlAaW5mLnUtc3plZ2VkLmh1Pgog CiAgICAgICAgIFtRdF0gVW5yZXZpZXdlZCBnYXJkZW5pbmcuCmRpZmYgLS1naXQgYS9MYXlvdXRU ZXN0cy9wbGF0Zm9ybS9xdC01LjAtd2syL1Rlc3RFeHBlY3RhdGlvbnMgYi9MYXlvdXRUZXN0cy9w bGF0Zm9ybS9xdC01LjAtd2syL1Rlc3RFeHBlY3RhdGlvbnMKaW5kZXggYTFiNTM5ODEyZTAyNjZm MDI5ZGQ3NzU2NzMyMzdiZTE4NzY2ZmY1YS4uY2YwOThmZmIyNjMxYzU4MTAwNTk5MmM4N2EzY2M0 ZThiYzAyMWI2MSAxMDA2NDQKLS0tIGEvTGF5b3V0VGVzdHMvcGxhdGZvcm0vcXQtNS4wLXdrMi9U ZXN0RXhwZWN0YXRpb25zCisrKyBiL0xheW91dFRlc3RzL3BsYXRmb3JtL3F0LTUuMC13azIvVGVz dEV4cGVjdGF0aW9ucwpAQCAtMzksNiArMzksNyBAQCBlZGl0aW5nL3NlbGVjdGlvbi9jb250ZXh0 LW1lbnUtb24tdGV4dC5odG1sCiBlZGl0aW5nL3NlbGVjdGlvbi9kb3VibGVjbGljay13aGl0ZXNw YWNlLWNyYXNoLmh0bWwKIGVkaXRpbmcvc2VsZWN0aW9uL2RvdWJsZWNsaWNrLXdoaXRlc3BhY2Ut aW1nLWNyYXNoLmh0bWwKIGVkaXRpbmcvc2VsZWN0aW9uL2RvdWJsZWNsaWNrLXdoaXRlc3BhY2Uu aHRtbAorZWRpdGluZy9zZWxlY3Rpb24vY2FyZXQtYWZ0ZXIta2V5cHJlc3MuaHRtbAogZWRpdGlu Zy9zZWxlY3Rpb24vZW1wdHktY2VsbC1yaWdodC1jbGljay5odG1sCiBlZGl0aW5nL3NlbGVjdGlv bi9tb3ZlLWJlZ2luLWVuZC5odG1sCiBlZGl0aW5nL3NlbGVjdGlvbi9zZWxlY3Qtb3V0LW9mLWVk aXRhYmxlLmh0bWwKQEAgLTc1Miw2ICs3NTMsMyBAQCB3ZWJraXQub3JnL2IvMTE2NjY4IGZhc3Qv ZXZlbnRzL3RvdWNoL2ZyYW1lLWhvdmVyLXVwZGF0ZS5odG1sIFsgU2tpcCBdCiAKICMgW1F0XSBO ZXdseSBhZGRkZWQgdGVzdCBpbiByMTUwMjM0IGZhc3QvZnJhbWVzL2ZsYXR0ZW5pbmcvc2Nyb2xs aW5nLWluLW9iamVjdC5odG1sIGZhaWxzCiB3ZWJraXQub3JnL2IvMTE2ODEwIGZhc3QvZnJhbWVz L2ZsYXR0ZW5pbmcvc2Nyb2xsaW5nLWluLW9iamVjdC5odG1sIFsgU2tpcCBdCi0KLSMgW1F0XVtX SzJdIE5ld2x5IGFkZGVkIGVkaXRpbmcvc2VsZWN0aW9uL2NhcmV0LWFmdGVyLWtleXByZXNzLmh0 bWwgY3Jhc2hpbmcKLXdlYmtpdC5vcmcvYi8xMTcwNzAgZWRpdGluZy9zZWxlY3Rpb24vY2FyZXQt YWZ0ZXIta2V5cHJlc3MuaHRtbCBbIFNraXAgXQo= 203650 2013-06-03 22:01:00 -0700 2013-06-06 03:27:40 -0700 Patch bug-117070-20130604085934.patch text/plain 2742 eustas U3VidmVyc2lvbiBSZXZpc2lvbjogMTUxMDkyCmRpZmYgLS1naXQgYS9MYXlvdXRUZXN0cy9DaGFu Z2VMb2cgYi9MYXlvdXRUZXN0cy9DaGFuZ2VMb2cKaW5kZXggOTFlYzE4OTY3M2JhZWY4M2U4MzEw NDQzNTFjNTEwZjI4OWYwZTg5MC4uOTUxMDViY2ZlMTQ0YTE4ZTNiNGI0NzI0ZDMyYWZlY2M1MDM5 YjhjYSAxMDA2NDQKLS0tIGEvTGF5b3V0VGVzdHMvQ2hhbmdlTG9nCisrKyBiL0xheW91dFRlc3Rz L0NoYW5nZUxvZwpAQCAtMSwzICsxLDE4IEBACisyMDEzLTA2LTAzICBFdWdlbmUgS2x5dWNobmlr b3YgIDxldXN0YXNAY2hyb21pdW0ub3JnPgorCisgICAgICAgIHNlbGVjdGlvblN0YXJ0L3NlbGVj dGlvbkVuZCByZXR1cm4gIm9ic29sZXRlIiB2YWx1ZXMgd2hlbiByZXF1ZXN0ZWQgZHVyaW5nICJp bnB1dCIgZXZlbnQKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dp P2lkPTExNzA3MAorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAg ICAgIFF0LVdLMiB0ZXN0IGNyYXNoZXMgd2hlbiBldmVudFNlbmRlci5rZXlEb3duIGludm9jYXRp b25zIGFyZSBuZXN0ZWQuCisKKyAgICAgICAgVGhpcyBwYXRjaCB1bnJvbGxzIHJlY3Vyc2lvbiB0 byBhdm9pZCBjcmFzaC4gCisKKyAgICAgICAgKiBlZGl0aW5nL3NlbGVjdGlvbi9jYXJldC1hZnRl ci1rZXlwcmVzcy5odG1sOgorICAgICAgICB1bnJvbGwgcmVjdXJzaXZlIGV2ZW50U2VuZGVyLmtl eURvd24gaW52b2NhdGlvbnMKKyAgICAgICAgKiBwbGF0Zm9ybS9xdC01LjAtd2syL1Rlc3RFeHBl Y3RhdGlvbnM6IFVuc2tpcCB0ZXN0LgorCiAyMDEzLTA2LTAyICBHeXV5b3VuZyBLaW0gIDxneXV5 b3VuZy5raW1Ac2Ftc3VuZy5jb20+CiAKICAgICAgICAgVW5yZXZpZXdlZCBFRkwgR2FyZGVuaW5n LgpkaWZmIC0tZ2l0IGEvTGF5b3V0VGVzdHMvZWRpdGluZy9zZWxlY3Rpb24vY2FyZXQtYWZ0ZXIt a2V5cHJlc3MuaHRtbCBiL0xheW91dFRlc3RzL2VkaXRpbmcvc2VsZWN0aW9uL2NhcmV0LWFmdGVy LWtleXByZXNzLmh0bWwKaW5kZXggYTZmMmM2ODE4NGE0NGZkYjA1ZDlhZjI0ODFlMGZlMDFmMDkx YTQwNC4uYTZlMWI5NWMyMmYyNTQ0Y2RlMzY2N2Q0Y2FhYjg0ZWI0NzZiMzc2NSAxMDA2NDQKLS0t IGEvTGF5b3V0VGVzdHMvZWRpdGluZy9zZWxlY3Rpb24vY2FyZXQtYWZ0ZXIta2V5cHJlc3MuaHRt bAorKysgYi9MYXlvdXRUZXN0cy9lZGl0aW5nL3NlbGVjdGlvbi9jYXJldC1hZnRlci1rZXlwcmVz cy5odG1sCkBAIC0xMywxNyArMTMsMTggQEAgY29uc3QgYmFja1NwYWNlID0gU3RyaW5nLmZyb21D aGFyQ29kZSg4KTsKIHZhciBpbnB1dCA9IFsiYSIsICJiIiwgImMiLCBiYWNrU3BhY2UsIGJhY2tT cGFjZSwgYmFja1NwYWNlXTsKIHZhciBvdXRwdXQgPSBbIiIsICJhIiwgImFiIiwgImFiYyIsICJh YiIsICJhIiwgIiJdOwogCi1mdW5jdGlvbiBzdGVwKCkgeworZnVuY3Rpb24gZHVtcFN0YXRlKCkg ewogICAgIHZhciBleHBlY3RlZFZhbHVlID0gb3V0cHV0LnNoaWZ0KCk7CiAgICAgc2hvdWxkQmVF cXVhbFRvU3RyaW5nKCJ0ZXN0LnZhbHVlIiwgZXhwZWN0ZWRWYWx1ZSk7CiAgICAgc2hvdWxkQmUo InRlc3Quc2VsZWN0aW9uU3RhcnQiLCAiIiArIGV4cGVjdGVkVmFsdWUubGVuZ3RoKTsKICAgICBz aG91bGRCZSgidGVzdC5zZWxlY3Rpb25FbmQiLCAiIiArIGV4cGVjdGVkVmFsdWUubGVuZ3RoKTsK ICAgICBkZWJ1ZygiIik7Ci0gICAgZXZlbnRTZW5kZXIua2V5RG93bihpbnB1dC5zaGlmdCgpKTsK IH0KIAotdGVzdC5hZGRFdmVudExpc3RlbmVyKCJpbnB1dCIsIHN0ZXApOwotc3RlcCgpOwordGVz dC5hZGRFdmVudExpc3RlbmVyKCJpbnB1dCIsIGR1bXBTdGF0ZSk7CitkdW1wU3RhdGUoKTsKK3do aWxlIChpbnB1dC5sZW5ndGgpCisgICAgZXZlbnRTZW5kZXIua2V5RG93bihpbnB1dC5zaGlmdCgp KTsKIDwvc2NyaXB0PgogPHNjcmlwdCBzcmM9Ii4uLy4uL2Zhc3QvanMvcmVzb3VyY2VzL2pzLXRl c3QtcG9zdC5qcyI+PC9zY3JpcHQ+CiA8L2JvZHk+CmRpZmYgLS1naXQgYS9MYXlvdXRUZXN0cy9w bGF0Zm9ybS9xdC01LjAtd2syL1Rlc3RFeHBlY3RhdGlvbnMgYi9MYXlvdXRUZXN0cy9wbGF0Zm9y bS9xdC01LjAtd2syL1Rlc3RFeHBlY3RhdGlvbnMKaW5kZXggY2YwOThmZmIyNjMxYzU4MTAwNTk5 MmM4N2EzY2M0ZThiYzAyMWI2MS4uMDdhZmM3N2ViMWFjNjYwMWRmMzY4MjVjMDM1MGI1ZGM0NDdk OWYzMSAxMDA2NDQKLS0tIGEvTGF5b3V0VGVzdHMvcGxhdGZvcm0vcXQtNS4wLXdrMi9UZXN0RXhw ZWN0YXRpb25zCisrKyBiL0xheW91dFRlc3RzL3BsYXRmb3JtL3F0LTUuMC13azIvVGVzdEV4cGVj dGF0aW9ucwpAQCAtMzksNyArMzksNiBAQCBlZGl0aW5nL3NlbGVjdGlvbi9jb250ZXh0LW1lbnUt b24tdGV4dC5odG1sCiBlZGl0aW5nL3NlbGVjdGlvbi9kb3VibGVjbGljay13aGl0ZXNwYWNlLWNy YXNoLmh0bWwKIGVkaXRpbmcvc2VsZWN0aW9uL2RvdWJsZWNsaWNrLXdoaXRlc3BhY2UtaW1nLWNy YXNoLmh0bWwKIGVkaXRpbmcvc2VsZWN0aW9uL2RvdWJsZWNsaWNrLXdoaXRlc3BhY2UuaHRtbAot ZWRpdGluZy9zZWxlY3Rpb24vY2FyZXQtYWZ0ZXIta2V5cHJlc3MuaHRtbAogZWRpdGluZy9zZWxl Y3Rpb24vZW1wdHktY2VsbC1yaWdodC1jbGljay5odG1sCiBlZGl0aW5nL3NlbGVjdGlvbi9tb3Zl LWJlZ2luLWVuZC5odG1sCiBlZGl0aW5nL3NlbGVjdGlvbi9zZWxlY3Qtb3V0LW9mLWVkaXRhYmxl Lmh0bWwK