85346 2012-05-01 21:58:40 -0700 EventSource: Events should not be dispatched after close() 2012-06-25 17:00:26 -0700 1 1 1 Unclassified WebKit WebCore Misc. 528+ (Nightly build) All All RESOLVED FIXED P2 Normal --- 1 yutak webkit-unassigned abarth adam.bergkvist ap pf sam tasak webkit.review.bot oldest_to_newest 613961 0 yutak 2012-05-01 21:58:40 -0700 Original report: http://code.google.com/p/chromium/issues/detail?id=125190 Reported by VMuhachev, Apr 26 (5 days ago) After EventSource#close() method is called, no events should be fired and readyState attribute should stay equal to EventSource.CLOSED See: http://html5.org/tools/web-apps-tracker?from=6771&to=6772 , https://www.w3.org/Bugs/Public/show_bug.cgi?id=14331#c5 event-stream: :text/event-stream data: data0; data: data1; var es = new EventSource('...'); var counter = 0; es.onmessage = function () { counter += 1; if (counter > 1) { alert('bug!'); } es.close(); }; 621996 1 141501 pf 2012-05-11 14:58:44 -0700 Created attachment 141501 Patch 646298 2 pf 2012-06-11 15:00:42 -0700 Ping? 657023 3 141501 webkit.review.bot 2012-06-25 17:00:13 -0700 Comment on attachment 141501 Patch Clearing flags on attachment: 141501 Committed r121195: <http://trac.webkit.org/changeset/121195> 657024 4 webkit.review.bot 2012-06-25 17:00:26 -0700 All reviewed patches have been landed. Closing bug. 141501 2012-05-11 14:58:44 -0700 2012-06-25 17:00:12 -0700 Patch bug-85346-20120511145843.patch text/plain 4710 pf U3VidmVyc2lvbiBSZXZpc2lvbjogMTE2NjkxCmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9D aGFuZ2VMb2cgYi9Tb3VyY2UvV2ViQ29yZS9DaGFuZ2VMb2cKaW5kZXggNzkxZmEwOTAwYzhjNDU2 ZTIxMzdiOTRmZGY3OGZiMzc0OGI2MDE0My4uYjAwZDljMWIwNGIyOGE1NTc1YmE1MjdkN2FhMmY4 NjI0ZDI4ODU3MyAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCisrKyBiL1Nv dXJjZS9XZWJDb3JlL0NoYW5nZUxvZwpAQCAtMSwzICsxLDI0IEBACisyMDEyLTA1LTExICBQYWJs byBGbG91cmV0ICA8cGFibG9mQG1vdG9yb2xhLmNvbT4KKworICAgICAgICBFdmVudFNvdXJjZTog RXZlbnRzIHNob3VsZCBub3QgYmUgZGlzcGF0Y2hlZCBhZnRlciBjbG9zZSgpCisgICAgICAgIGh0 dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD04NTM0NgorCisgICAgICAgIFJl dmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgIFNwZWMgY2hhbmdlZCB0byBtYWtl IHN1cmUgdGhhdCBubyBldmVudHMgYXJlIGRpc3BhdGNoZWQgYWZ0ZXIgY2xvc2UoKSBpcworICAg ICAgICBjYWxsZWQsIGV2ZW4gaWYgbW9yZSBkYXRhIHdhcyByZWNlaXZlZCBiZWZvcmUgdGhlIGNh bGwgdG8gY2xvc2UoKS4KKworICAgICAgICBTZWUsCisgICAgICAgIGh0dHBzOi8vd3d3LnczLm9y Zy9CdWdzL1B1YmxpYy9zaG93X2J1Zy5jZ2k/aWQ9MTQzMzEjYzUKKyAgICAgICAgaHR0cDovL2h0 bWw1Lm9yZy90b29scy93ZWItYXBwcy10cmFja2VyP2Zyb209Njc3MSZ0bz02NzcyCisKKyAgICAg ICAgRmlyZWZveCBiZWhhdmVzIGxpa2UgdGhpcyBhbHJlYWR5LgorCisgICAgICAgIFRlc3Q6IGh0 dHAvdGVzdHMvZXZlbnRzb3VyY2UvZXZlbnRzb3VyY2UtZXZlbnRzLWFmdGVyLWNsb3NlLmh0bWwK KworICAgICAgICAqIHBhZ2UvRXZlbnRTb3VyY2UuY3BwOgorICAgICAgICAoV2ViQ29yZTo6RXZl bnRTb3VyY2U6OnBhcnNlRXZlbnRTdHJlYW0pOgorCiAyMDEyLTA1LTEwICBQYWJsbyBGbG91cmV0 ICA8cGFibG9mQG1vdG9yb2xhLmNvbT4KIAogICAgICAgICBJbXBsZW1lbnQgY3NzMy1jb25kaXRp b25hbCdzIEBzdXBwb3J0cyBydWxlCmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9wYWdlL0V2 ZW50U291cmNlLmNwcCBiL1NvdXJjZS9XZWJDb3JlL3BhZ2UvRXZlbnRTb3VyY2UuY3BwCmluZGV4 IDU0M2YxZTFhYzc0MjM4YTMzOWUyMTgxMjk4MDI4MzBmNGFhMjk0MTMuLjFjZjhhYmFiYjIwN2Iw NzllNTkzNTZlYjVmN2VlYjkwOGMzMGY0NGIgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9XZWJDb3JlL3Bh Z2UvRXZlbnRTb3VyY2UuY3BwCisrKyBiL1NvdXJjZS9XZWJDb3JlL3BhZ2UvRXZlbnRTb3VyY2Uu Y3BwCkBAIC0zMTUsNiArMzE1LDExIEBAIHZvaWQgRXZlbnRTb3VyY2U6OnBhcnNlRXZlbnRTdHJl YW0oKQogCiAgICAgICAgIHBhcnNlRXZlbnRTdHJlYW1MaW5lKGJ1ZlBvcywgZmllbGRMZW5ndGgs IGxpbmVMZW5ndGgpOwogICAgICAgICBidWZQb3MgKz0gbGluZUxlbmd0aCArIDE7CisKKyAgICAg ICAgLy8gRXZlbnRTb3VyY2UuY2xvc2UoKSBtaWdodCd2ZSBiZWVuIGNhbGxlZCBieSBvbmUgb2Yg dGhlIG1lc3NhZ2UgZXZlbnQgaGFuZGxlcnMuCisgICAgICAgIC8vIFBlciBzcGVjLCBubyBmdXJ0 aGVyIG1lc3NhZ2VzIHNob3VsZCBiZSBmaXJlZCBhZnRlciB0aGF0LgorICAgICAgICBpZiAobV9z dGF0ZSA9PSBDTE9TRUQpCisgICAgICAgICAgICBicmVhazsKICAgICB9CiAKICAgICBpZiAoYnVm UG9zID09IGJ1ZlNpemUpCmRpZmYgLS1naXQgYS9MYXlvdXRUZXN0cy9DaGFuZ2VMb2cgYi9MYXlv dXRUZXN0cy9DaGFuZ2VMb2cKaW5kZXggYTZmNTBlNjRhMjM2ZWIyN2MyZTU4M2U3ODEzOGE3MTFl ZmVhZGIyMi4uMGE1ZGQ2NzA0MjhjMDI0ZDI3NGZkNmE0YThlNjJjZTcxNWM2NDAxOCAxMDA2NDQK LS0tIGEvTGF5b3V0VGVzdHMvQ2hhbmdlTG9nCisrKyBiL0xheW91dFRlc3RzL0NoYW5nZUxvZwpA QCAtMSwzICsxLDEzIEBACisyMDEyLTA1LTExICBQYWJsbyBGbG91cmV0ICA8cGFibG9mQG1vdG9y b2xhLmNvbT4KKworICAgICAgICBFdmVudFNvdXJjZTogRXZlbnRzIHNob3VsZCBub3QgYmUgZGlz cGF0Y2hlZCBhZnRlciBjbG9zZSgpCisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3No b3dfYnVnLmNnaT9pZD04NTM0NgorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEp LgorCisgICAgICAgICogaHR0cC90ZXN0cy9ldmVudHNvdXJjZS9ldmVudHNvdXJjZS1ldmVudHMt YWZ0ZXItY2xvc2UtZXhwZWN0ZWQudHh0OiBBZGRlZC4KKyAgICAgICAgKiBodHRwL3Rlc3RzL2V2 ZW50c291cmNlL2V2ZW50c291cmNlLWV2ZW50cy1hZnRlci1jbG9zZS5odG1sOiBBZGRlZC4KKwog MjAxMi0wNS0xMCAgUGFibG8gRmxvdXJldCAgPHBhYmxvZkBtb3Rvcm9sYS5jb20+CiAKICAgICAg ICAgSW1wbGVtZW50IGNzczMtY29uZGl0aW9uYWwncyBAc3VwcG9ydHMgcnVsZQpkaWZmIC0tZ2l0 IGEvTGF5b3V0VGVzdHMvaHR0cC90ZXN0cy9ldmVudHNvdXJjZS9ldmVudHNvdXJjZS1ldmVudHMt YWZ0ZXItY2xvc2UtZXhwZWN0ZWQudHh0IGIvTGF5b3V0VGVzdHMvaHR0cC90ZXN0cy9ldmVudHNv dXJjZS9ldmVudHNvdXJjZS1ldmVudHMtYWZ0ZXItY2xvc2UtZXhwZWN0ZWQudHh0Cm5ldyBmaWxl IG1vZGUgMTAwNjQ0CmluZGV4IDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAw MDAuLjBlOWJmNWI4OWNiYjQzZTI1NjE1MzQ1ZTVmODc1ZjcyMmZlMzM5ZDYKLS0tIC9kZXYvbnVs bAorKysgYi9MYXlvdXRUZXN0cy9odHRwL3Rlc3RzL2V2ZW50c291cmNlL2V2ZW50c291cmNlLWV2 ZW50cy1hZnRlci1jbG9zZS1leHBlY3RlZC50eHQKQEAgLTAsMCArMSw0IEBACitUZXN0IHRoYXQg bm8gbW9yZSBtZXNzYWdlIGV2ZW50cyBhcmUgZmlyZWQgYWZ0ZXIgRXZlbnRTb3VyY2UuY2xvc2Uo KSBpcyBjYWxsZWQsIGV2ZW4gaWYgaXQgbWVhbnMgZGlzY2FyZGluZyBldmVudHMgdGhhdCB3ZXJl IGFscmVhZHkgcHJvY2Vzc2VkLiBQYXNzZXMgaWYgb25seSBvbmUgbWVzc2FnZSBpcyByZWNlaXZl ZC4KKworR290IG1lc3NhZ2UgIzEKKwpkaWZmIC0tZ2l0IGEvTGF5b3V0VGVzdHMvaHR0cC90ZXN0 cy9ldmVudHNvdXJjZS9ldmVudHNvdXJjZS1ldmVudHMtYWZ0ZXItY2xvc2UuaHRtbCBiL0xheW91 dFRlc3RzL2h0dHAvdGVzdHMvZXZlbnRzb3VyY2UvZXZlbnRzb3VyY2UtZXZlbnRzLWFmdGVyLWNs b3NlLmh0bWwKbmV3IGZpbGUgbW9kZSAxMDA2NDQKaW5kZXggMDAwMDAwMDAwMDAwMDAwMDAwMDAw MDAwMDAwMDAwMDAwMDAwMDAwMC4uNzA1MjRjMjUxZjhhMTkxNDFkZWU3YzdmZmJlYmQ1NDYzNDhm NmU2ZQotLS0gL2Rldi9udWxsCisrKyBiL0xheW91dFRlc3RzL2h0dHAvdGVzdHMvZXZlbnRzb3Vy Y2UvZXZlbnRzb3VyY2UtZXZlbnRzLWFmdGVyLWNsb3NlLmh0bWwKQEAgLTAsMCArMSwzNCBAQAor PCFET0NUWVBFIGh0bWw+Cis8cD4KKyAgICBUZXN0IHRoYXQgbm8gbW9yZSBtZXNzYWdlIGV2ZW50 cyBhcmUgZmlyZWQgYWZ0ZXIgRXZlbnRTb3VyY2UuY2xvc2UoKSBpcyBjYWxsZWQsIGV2ZW4gaWYg aXQKKyAgICBtZWFucyBkaXNjYXJkaW5nIGV2ZW50cyB0aGF0IHdlcmUgYWxyZWFkeSBwcm9jZXNz ZWQuCisgICAgUGFzc2VzIGlmIG9ubHkgb25lIG1lc3NhZ2UgaXMgcmVjZWl2ZWQuCis8L3A+Cis8 cHJlIGlkPWxvZz48L3ByZT4KKzxzY3JpcHQ+CisgICAgaWYgKHdpbmRvdy5sYXlvdXRUZXN0Q29u dHJvbGxlcikgeworICAgICAgICBsYXlvdXRUZXN0Q29udHJvbGxlci5kdW1wQXNUZXh0KCk7Cisg ICAgICAgIGxheW91dFRlc3RDb250cm9sbGVyLndhaXRVbnRpbERvbmUoKTsKKyAgICB9CisKKyAg ICB2YXIgbG9nID0gZG9jdW1lbnQuZ2V0RWxlbWVudEJ5SWQoImxvZyIpOworICAgIHZhciBlcyA9 IG5ldyBFdmVudFNvdXJjZSgicmVzb3VyY2VzL2V2ZW50LXN0cmVhbS5waHAiKTsKKyAgICB2YXIg Y291bnRlciA9IDA7CisgICAgZXMuYWRkRXZlbnRMaXN0ZW5lcignbWVzc2FnZScsIGZ1bmN0aW9u IChlKSB7CisgICAgICAgIGxvZy5pbm5lckhUTUwgKz0gIkdvdCBtZXNzYWdlICMiICsgKytjb3Vu dGVyICsgIlxuIjsKKworICAgICAgICBpZiAoY291bnRlciA+IDEpCisgICAgICAgICAgICBsb2cu aW5uZXJIVE1MICs9ICJGQUlMLiBIYW5kbGVyIGNhbGxlZCBhZnRlciB0aGUgc291cmNlIHdhcyBj bG9zZWQgZXhwbGljaXRlbHkuXG4iOworCisgICAgICAgIGVzLmNsb3NlKCk7CisKKyAgICAgICAg c2V0VGltZW91dChmdW5jdGlvbiAoKSB7CisgICAgICAgICAgICAvLyBOZWVkIHRvIHdhaXQgdG8g c2VlIGlmIHdlJ3JlIGNhbGxlZCBhZ2Fpbi4KKyAgICAgICAgICAgIC8vIGV2ZW50LXN0cmVhbS5w aHAgc2VuZHMgYSBidW5jaCBvZiBldmVudHMgYmVmb3JlIGZsdXNoaW5nLCBzbyBpZiBjbG9zZSgp IGRpZG4ndCB0YWtlCisgICAgICAgICAgICAvLyBlZmZlY3Qgd2UnZCBnZXQgYSBzZWNvbmQgbWVz c2FnZSBwcmFjdGljYWxseSBpbnN0YW50YW5lb3VzbHksIHdhaXRpbmcgMTAwbXMgc2hvdWxkIGJl IG9rLgorICAgICAgICAgICAgaWYgKHdpbmRvdy5sYXlvdXRUZXN0Q29udHJvbGxlcikKKyAgICAg ICAgICAgICAgICBsYXlvdXRUZXN0Q29udHJvbGxlci5ub3RpZnlEb25lKCk7CisgICAgICAgIH0s IDEwMCk7CisgICAgfSwgZmFsc2UpOworCis8L3NjcmlwdD4K