62475 2011-06-10 14:05:54 -0700 Web Worker fails to fire error event when a resource fetch fail. 2011-06-10 17:21:59 -0700 1 1 1 Unclassified WebKit DOM 528+ (Nightly build) Unspecified Unspecified RESOLVED FIXED P2 Normal --- 1 levin levin ap dimich dslomov ian oldest_to_newest 418893 0 levin 2011-06-10 14:05:54 -0700 What steps will reproduce the problem? 1. Load the following HTML page <HTML> <BODY> <script type="text/javascript" language="javascript"> var worker = new Worker("no_exist.js"); worker.addEventListener('error', function(error) { alert("Failed to load resource 1"); }, false); worker.onerror = function(error) { alert("Failed to load resource 2"); }; </script> </BODY> </HTML> What is the expected result? Two alerts should appear (one for each handler)with the messages "Failed to load resource 1" and "Failed to load resource 2" as per the spec per the spec http://www.whatwg.org/specs/web-workers/current-work/#processing-model This works in Firefox. In Chromium db as http://code.google.com/p/chromium/issues/detail?id=85220 418921 1 96790 levin 2011-06-10 14:28:50 -0700 Created attachment 96790 Patch 418944 2 ian 2011-06-10 15:00:14 -0700 Actually per spec the order should be 2 then 1, because the spec says: "All event handlers on an object, whether an element or some other object, and whether set to null or to a Function object, must be registered as event listeners on the object when it is created, as if the addEventListener() method on the object's EventTarget interface had been invoked, with the event type (type argument) equal to the type corresponding to the event handler (the event handler event type), the listener set to be a target and bubbling phase listener (useCapture argument set to false), and the event listener itself (listener argument) set to do nothing while the event handler's value is not a Function object, and set to invoke the call() callback of the Function object associated with the event handler otherwise. Note: Event handlers therefore always run before event listeners attached using addEventListener()." 418948 3 96790 levin 2011-06-10 15:02:28 -0700 Comment on attachment 96790 Patch I'll change this test to verify the ordering. 418962 4 96790 levin 2011-06-10 15:18:19 -0700 Comment on attachment 96790 Patch It was a little while since I wrote the test. It does verify the ordering. Of course, it is backwards. I filed bug https://bugs.webkit.org/show_bug.cgi?id=62485 to address this. 419060 5 96790 dimich 2011-06-10 17:07:36 -0700 Comment on attachment 96790 Patch Could you add a comment into the test title with ref to the bug that will fix the order? 419064 6 levin 2011-06-10 17:21:59 -0700 (In reply to comment #5) > (From update of attachment 96790 [details]) > Could you add a comment into the test title with ref to the bug that will fix the order? Done and committed as http://trac.webkit.org/changeset/88579. 96790 2011-06-10 14:28:50 -0700 2011-06-10 17:07:36 -0700 Patch bug-62475-20110610142848.patch text/plain 4039 levin U3VidmVyc2lvbiBSZXZpc2lvbjogODg1NTMKZGlmZiAtLWdpdCBhL0xheW91dFRlc3RzL0NoYW5n ZUxvZyBiL0xheW91dFRlc3RzL0NoYW5nZUxvZwppbmRleCA0NjFmMjM5ZTM0Y2FiYTRlNzEwNjMx Y2Q5YWRjMjQyNGJmMWVlN2ZhLi5iODI3ZjUzMmM2ZmJiNmI2YWEzZTViZjg1YjUyNjkzNjc0NGY3 MTE5IDEwMDY0NAotLS0gYS9MYXlvdXRUZXN0cy9DaGFuZ2VMb2cKKysrIGIvTGF5b3V0VGVzdHMv Q2hhbmdlTG9nCkBAIC0xLDMgKzEsMTMgQEAKKzIwMTEtMDYtMTAgIERhdmlkIExldmluICA8bGV2 aW5AY2hyb21pdW0ub3JnPgorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgor CisgICAgICAgIFdlYiBXb3JrZXIgZmFpbHMgdG8gZmlyZSBlcnJvciBldmVudCB3aGVuIGEgcmVz b3VyY2UgZmV0Y2ggZmFpbHMuCisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3df YnVnLmNnaT9pZD02MjQ3NQorCisgICAgICAgICogaHR0cC90ZXN0cy93b3JrZXJzL3dvcmtlci13 b3JrZXJTY3JpcHROb3RUaGVyZS1leHBlY3RlZC50eHQ6IEFkZGVkLgorICAgICAgICAqIGh0dHAv dGVzdHMvd29ya2Vycy93b3JrZXItd29ya2VyU2NyaXB0Tm90VGhlcmUuaHRtbDogQWRkZWQuCisK IDIwMTEtMDYtMTAgIEFiaGlzaGVrIEFyeWEgIDxpbmZlcm5vQGNocm9taXVtLm9yZz4KIAogICAg ICAgICBSZXZpZXdlZCBieSBUb255IEdlbnRpbGNvcmUuCmRpZmYgLS1naXQgYS9MYXlvdXRUZXN0 cy9odHRwL3Rlc3RzL3dvcmtlcnMvd29ya2VyLXdvcmtlclNjcmlwdE5vdFRoZXJlLWV4cGVjdGVk LnR4dCBiL0xheW91dFRlc3RzL2h0dHAvdGVzdHMvd29ya2Vycy93b3JrZXItd29ya2VyU2NyaXB0 Tm90VGhlcmUtZXhwZWN0ZWQudHh0Cm5ldyBmaWxlIG1vZGUgMTAwNjQ0CmluZGV4IDAwMDAwMDAw MDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAuLjcyMmE5ODY4ZmJhZGViYmI0MjYwYWJl ZGMyZDIzODhmODk3YzMyNDcKLS0tIC9kZXYvbnVsbAorKysgYi9MYXlvdXRUZXN0cy9odHRwL3Rl c3RzL3dvcmtlcnMvd29ya2VyLXdvcmtlclNjcmlwdE5vdFRoZXJlLWV4cGVjdGVkLnR4dApAQCAt MCwwICsxLDYgQEAKK1Rlc3Qgd29ya2VyIGZpbGUgZG9lcyBub3QgZXhpc3QgZXJyb3IuIFNob3Vs ZCBwcmludCB0d28gIlBBU1MiIHN0YXRlbWVudHMgZm9sbG93ZWQgYnkgIkRPTkUiLgorCitQQVNT OiBlcnJvciBldmVudC4KK1BBU1M6IG9uZXJyb3IgY2FsbGVkLgorRE9ORQorCmRpZmYgLS1naXQg YS9MYXlvdXRUZXN0cy9odHRwL3Rlc3RzL3dvcmtlcnMvd29ya2VyLXdvcmtlclNjcmlwdE5vdFRo ZXJlLmh0bWwgYi9MYXlvdXRUZXN0cy9odHRwL3Rlc3RzL3dvcmtlcnMvd29ya2VyLXdvcmtlclNj cmlwdE5vdFRoZXJlLmh0bWwKbmV3IGZpbGUgbW9kZSAxMDA2NDQKaW5kZXggMDAwMDAwMDAwMDAw MDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMC4uOGJmMzI4Mzk2ODhmMjNiMjFiMWU4OTRmMTBj MjdjNWQ2MTFiMmIxNgotLS0gL2Rldi9udWxsCisrKyBiL0xheW91dFRlc3RzL2h0dHAvdGVzdHMv d29ya2Vycy93b3JrZXItd29ya2VyU2NyaXB0Tm90VGhlcmUuaHRtbApAQCAtMCwwICsxLDUwIEBA Cis8aHRtbD4KKzxib2R5PgorPHA+VGVzdCB3b3JrZXIgZmlsZSBkb2VzIG5vdCBleGlzdCBlcnJv ci4gU2hvdWxkIHByaW50IHR3byAiUEFTUyIgc3RhdGVtZW50cyBmb2xsb3dlZCBieSAiRE9ORSIu PC9wPgorPGRpdiBpZD1yZXN1bHQ+PC9kaXY+Cis8c2NyaXB0PgoraWYgKHdpbmRvdy5sYXlvdXRU ZXN0Q29udHJvbGxlcikgeworICAgIGxheW91dFRlc3RDb250cm9sbGVyLmR1bXBBc1RleHQoKTsK KyAgICBsYXlvdXRUZXN0Q29udHJvbGxlci53YWl0VW50aWxEb25lKCk7Cit9CitmdW5jdGlvbiBs b2cobWVzc2FnZSkKK3sKKyAgICBkb2N1bWVudC5nZXRFbGVtZW50QnlJZCgicmVzdWx0IikuaW5u ZXJIVE1MICs9IG1lc3NhZ2UgKyAiPGJyPiI7Cit9CisKK2Z1bmN0aW9uIGRvbmUoKQoreworICAg IGxvZygiRE9ORSIpOworICAgIGlmICh3aW5kb3cubGF5b3V0VGVzdENvbnRyb2xsZXIpCisgICAg ICAgIGxheW91dFRlc3RDb250cm9sbGVyLm5vdGlmeURvbmUoKTsKK30KKwordmFyIGV2ZW50Q291 bnQgPSAwOworCitmdW5jdGlvbiBldmVudERvbmUoKQoreworICAgIGV2ZW50Q291bnQrKzsKKyAg ICBpZiAoZXZlbnRDb3VudCA9PSAyKQorICAgICAgICBkb25lKCk7Cit9CisKK3ZhciB3b3JrZXI7 Cit0cnkgeworICAgIHdvcmtlciA9IG5ldyBXb3JrZXIoInRoaXMtZmlsZS1kb2VzLXNob3VsZC1u b3QtZXhpc3QuanMiKTsKK30gY2F0Y2ggKGVycm9yKSB7CisgICAgbG9nKCJGQUlMOiBFeGNlcHRp b24gdGhyb3duLiIpOworfQorCit3b3JrZXIuYWRkRXZlbnRMaXN0ZW5lcignZXJyb3InLCBmdW5j dGlvbihlcnJvcikgeworICAgIGxvZygiUEFTUzogZXJyb3IgZXZlbnQuIik7CisgICAgZXZlbnRE b25lKCk7Cit9LCBmYWxzZSk7CisKK3dvcmtlci5vbmVycm9yID0gIGZ1bmN0aW9uKGVycm9yKSB7 CisgICAgbG9nKCJQQVNTOiBvbmVycm9yIGNhbGxlZC4iKTsKKyAgICBldmVudERvbmUoKTsKK307 CisKKzwvc2NyaXB0PgorPC9ib2R5PgorPC9odG1sPgpkaWZmIC0tZ2l0IGEvU291cmNlL1dlYkNv cmUvQ2hhbmdlTG9nIGIvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCmluZGV4IGUzNjY3OWRiYmI0 OWVjODg0Mjg4MmU5ZjZkYWU3ZjJiOGQxZTllYjcuLmNmOGU4NGQxMDc5NTczYzU1Yzg3NWZkNzRj ZTE0YmRiMWFlYzMyMjcgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZworKysg Yi9Tb3VyY2UvV2ViQ29yZS9DaGFuZ2VMb2cKQEAgLTEsMyArMSwxNiBAQAorMjAxMS0wNi0xMCAg RGF2aWQgTGV2aW4gIDxsZXZpbkBjaHJvbWl1bS5vcmc+CisKKyAgICAgICAgUmV2aWV3ZWQgYnkg Tk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgV2ViIFdvcmtlciBmYWlscyB0byBmaXJlIGVycm9y IGV2ZW50IHdoZW4gYSByZXNvdXJjZSBmZXRjaCBmYWlscy4KKyAgICAgICAgaHR0cHM6Ly9idWdz LndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTYyNDc1CisKKyAgICAgICAgVGVzdDogaHR0cC90 ZXN0cy93b3JrZXJzL3dvcmtlci13b3JrZXJTY3JpcHROb3RUaGVyZS5odG1sCisKKyAgICAgICAg KiB3b3JrZXJzL1dvcmtlclNjcmlwdExvYWRlci5jcHA6CisgICAgICAgIChXZWJDb3JlOjpXb3Jr ZXJTY3JpcHRMb2FkZXI6OmRpZEZpbmlzaExvYWRpbmcpOiBFbnN1cmUgdGhhdAorICAgICAgICB0 aGUgY2xpZW50IGlzIG5vdGlmaWVkIG9mIHRoZSBlcnJvciB3aGVuIGl0IGhhcHBlbnMgZHVyaW5n IGxvYWRpbmcuCisKIDIwMTEtMDYtMTAgIEx1a2UgTWFjcGhlcnNvbiAgIDxtYWNwaGVyc29uQGNo cm9taXVtLm9yZz4KIAogICAgICAgICBSZXZpZXdlZCBieSBFcmljIFNlaWRlbC4KZGlmZiAtLWdp dCBhL1NvdXJjZS9XZWJDb3JlL3dvcmtlcnMvV29ya2VyU2NyaXB0TG9hZGVyLmNwcCBiL1NvdXJj ZS9XZWJDb3JlL3dvcmtlcnMvV29ya2VyU2NyaXB0TG9hZGVyLmNwcAppbmRleCBlZDExMTU5OWJj ZDZjZTYxZjMzN2MyZWU4NTE4NjE1NDI3M2M0YjI0Li4yMTQ5NWQ3ZDVjNzU5NmVlNjJiYWNjNWM0 MTVkNWY3ODg2MzVkNzE5IDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViQ29yZS93b3JrZXJzL1dvcmtl clNjcmlwdExvYWRlci5jcHAKKysrIGIvU291cmNlL1dlYkNvcmUvd29ya2Vycy9Xb3JrZXJTY3Jp cHRMb2FkZXIuY3BwCkBAIC0xMzYsOCArMTM2LDEwIEBAIHZvaWQgV29ya2VyU2NyaXB0TG9hZGVy OjpkaWRSZWNlaXZlRGF0YShjb25zdCBjaGFyKiBkYXRhLCBpbnQgbGVuKQogCiB2b2lkIFdvcmtl clNjcmlwdExvYWRlcjo6ZGlkRmluaXNoTG9hZGluZyh1bnNpZ25lZCBsb25nIGlkZW50aWZpZXIs IGRvdWJsZSkKIHsKLSAgICBpZiAobV9mYWlsZWQpCisgICAgaWYgKG1fZmFpbGVkKSB7CisgICAg ICAgIG5vdGlmeUVycm9yKCk7CiAgICAgICAgIHJldHVybjsKKyAgICB9CiAKICAgICBpZiAobV9k ZWNvZGVyKQogICAgICAgICBtX3NjcmlwdCArPSBtX2RlY29kZXItPmZsdXNoKCk7Cg==