29801 2009-09-28 02:33:53 -0700 addEventListener of MessagePort should also call start(), like onmessage 2009-09-29 10:41:42 -0700 1 1 1 Unclassified WebKit WebCore JavaScript 528+ (Nightly build) All All RESOLVED INVALID P2 Normal --- 0 zherczeg webkit-unassigned ap sam oldest_to_newest 150477 0 zherczeg 2009-09-28 02:33:53 -0700 My qestion is: should we do it all the time, or only when eventType is "message". In case of the latter, where should I create a global string "message"? 150481 1 40223 zherczeg 2009-09-28 03:37:22 -0700 Created attachment 40223 fix for calling the call start() 150544 2 sam 2009-09-28 09:57:16 -0700 This needs a regression test. 150546 3 ap 2009-09-28 10:18:32 -0700 Also, please explain why this should be so. I don't think the spec says that. 150554 4 zherczeg 2009-09-28 10:44:37 -0700 (In reply to comment #3) > Also, please explain why this should be so. I don't think the spec says that. See the SharedWorker example here: http://www.whatwg.org/specs/web-workers/current-work/ See the code example after "The viewer is more involved:" worker.port.addEventListener('message', configure, false); is used instead of worker.port.onmessage = configure Moreover, more addEventListener (4, exactly) is applied for one worker object (actually I didn't tested that, yet) You think the example is wrong? How can I make regression test for Shared Workers? 150565 5 ap 2009-09-28 11:05:16 -0700 > See the SharedWorker example here: > http://www.whatwg.org/specs/web-workers/current-work/ The examples section is non-normative. We need to have this specified in normative text - I'd suggest e-mailing WhatWG mailing list about this issue. > How can I make regression test for Shared Workers? We have a number of tests for workers in LayoutTests/fast/workers, so you could use one of those as an example. 150951 6 zherczeg 2009-09-29 10:41:42 -0700 According to the discussion on whatwg (http://lists.whatwg.org/htdig.cgi/whatwg-whatwg.org/2009-September/023246.html), this bug turned out to be a feature. And they will probably change the example to call worker.start() after the listeners added. I close this bug now. Days of debugging goes waste again. I am not a lucky person nowadays. 40223 2009-09-28 03:37:22 -0700 2009-09-28 09:57:02 -0700 fix for calling the call start() 0001-Call-start-for-addEventListener.patch text/plain 1590 zherczeg RnJvbSA2YTAyYjI4NTk2MjliYzhiNDE3ZDVlOTI2MWU5NDBmNjY3MjY3NzMzIE1vbiBTZXAgMTcg MDA6MDA6MDAgMjAwMQpGcm9tOiBab2x0YW4gSGVyY3plZyA8emhlcmN6ZWdAaW5mLnUtc3plZ2Vk Lmh1PgpEYXRlOiBNb24sIDI4IFNlcCAyMDA5IDExOjM4OjEyICswMjAwClN1YmplY3Q6IFtQQVRD SF0gQ2FsbCBzdGFydCBmb3IgYWRkRXZlbnRMaXN0ZW5lcgoKU2lnbmVkLW9mZi1ieTogWm9sdGFu IEhlcmN6ZWcgPHpoZXJjemVnQGluZi51LXN6ZWdlZC5odT4KLS0tCiBXZWJDb3JlL0NoYW5nZUxv ZyAgICAgICAgICAgfCAgIDEzICsrKysrKysrKysrKysKIFdlYkNvcmUvZG9tL01lc3NhZ2VQb3J0 LmNwcCB8ICAgIDEgKwogMiBmaWxlcyBjaGFuZ2VkLCAxNCBpbnNlcnRpb25zKCspLCAwIGRlbGV0 aW9ucygtKQoKZGlmZiAtLWdpdCBhL1dlYkNvcmUvQ2hhbmdlTG9nIGIvV2ViQ29yZS9DaGFuZ2VM b2cKaW5kZXggYzJmZjJhMi4uYzMxYjFlNiAxMDA2NDQKLS0tIGEvV2ViQ29yZS9DaGFuZ2VMb2cK KysrIGIvV2ViQ29yZS9DaGFuZ2VMb2cKQEAgLTEsMyArMSwxNiBAQAorMjAwOS0wOS0yOCAgWm9s dGFuIEhlcmN6ZWcgIDx6aGVyY3plZ0BpbmYudS1zemVnZWQuaHU+CisKKyAgICAgICAgUmV2aWV3 ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgYWRkRXZlbnRMaXN0ZW5lciBvZiBNZXNz YWdlUG9ydCBzaG91bGQgYWxzbyBjYWxsIHN0YXJ0KCkKKyAgICAgICAgaHR0cHM6Ly9idWdzLndl YmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTI5ODAxCisKKyAgICAgICAgYWRkRXZlbnRMaXN0ZW5l ciwganVzdCBsaWtlIG9ubWVzc2FnZSwgc2hvdWxkIGFsc28gY2FsbAorICAgICAgICBzdGFydCgp IHRvIGVuYWJsZSByZWNlaXZpbmcgbWVzc2FnZXMKKworICAgICAgICAqIGRvbS9NZXNzYWdlUG9y dC5jcHA6CisgICAgICAgIChXZWJDb3JlOjpNZXNzYWdlUG9ydDo6YWRkRXZlbnRMaXN0ZW5lcik6 CisKIDIwMDktMDktMTYgIEd1c3Rhdm8gTm9yb25oYSBTaWx2YSAgPGd1c3Rhdm8ubm9yb25oYUBj b2xsYWJvcmEuY28udWs+CiAKICAgICAgICAgUmV2aWV3ZWQgYnkgWGFuIExvcGV6LgpkaWZmIC0t Z2l0IGEvV2ViQ29yZS9kb20vTWVzc2FnZVBvcnQuY3BwIGIvV2ViQ29yZS9kb20vTWVzc2FnZVBv cnQuY3BwCmluZGV4IGJmZDc5MzIuLjQ3Nzc4MTcgMTAwNjQ0Ci0tLSBhL1dlYkNvcmUvZG9tL01l c3NhZ2VQb3J0LmNwcAorKysgYi9XZWJDb3JlL2RvbS9NZXNzYWdlUG9ydC5jcHAKQEAgLTIwMCw2 ICsyMDAsNyBAQCB2b2lkIE1lc3NhZ2VQb3J0OjphZGRFdmVudExpc3RlbmVyKGNvbnN0IEF0b21p Y1N0cmluZyYgZXZlbnRUeXBlLCBQYXNzUmVmUHRyPEV2ZQogICAgICAgICBsaXN0ZW5lcnMuYXBw ZW5kKGV2ZW50TGlzdGVuZXIpOwogICAgICAgICBtX2V2ZW50TGlzdGVuZXJzLmFkZChldmVudFR5 cGUsIGxpc3RlbmVycyk7CiAgICAgfSAgICAKKyAgICBzdGFydCgpOwogfQogCiB2b2lkIE1lc3Nh Z2VQb3J0OjpyZW1vdmVFdmVudExpc3RlbmVyKGNvbnN0IEF0b21pY1N0cmluZyYgZXZlbnRUeXBl LCBFdmVudExpc3RlbmVyKiBldmVudExpc3RlbmVyLCBib29sKQotLSAKMS42LjAuNAoK