101022 2012-11-02 00:02:09 -0700 Web Inspector: a small defect in "WorkersSidebarPanel.js" 2012-11-02 00:54:52 -0700 1 1 1 Unclassified WebKit Web Inspector (Deprecated) 528+ (Nightly build) All All RESOLVED FIXED P2 Normal --- 0 PeterHWang webkit-unassigned apavlov keishi loislo pfeldman pmuellr vsevik web-inspector-bugs webkit.review.bot yurys oldest_to_newest 756925 0 PeterHWang 2012-11-02 00:02:09 -0700 When click the label "Pause on start" of WorkerSidebarPane, in function _autoattachToWorkersClicked: function(event) { WorkerAgent.setAutoconnectToWorkers(event.target.checked); }, "event.target.checked" will cause a error report in "InspectorBackend.js:113": Protocol Error: Invalid type of argument 'value' for method 'Worker.setAutoconnectToWorkers' call. It must be 'boolean' but it is 'undefined'. Since in "Checkbox.js", the "label" element also registered "click" event, it has not a property named "checked". However, it's bind with CheckBox, so the "click" can be send to CheckBox finally. So this problem doesn't effect any function, just causes a error report. 756929 1 171992 PeterHWang 2012-11-02 00:09:05 -0700 Created attachment 171992 Patch 756956 2 171992 webkit.review.bot 2012-11-02 00:54:49 -0700 Comment on attachment 171992 Patch Clearing flags on attachment: 171992 Committed r133271: <http://trac.webkit.org/changeset/133271> 756957 3 webkit.review.bot 2012-11-02 00:54:52 -0700 All reviewed patches have been landed. Closing bug. 171992 2012-11-02 00:09:05 -0700 2012-11-02 00:54:48 -0700 Patch bug-101022-20121102150717.patch text/plain 1683 PeterHWang U3VidmVyc2lvbiBSZXZpc2lvbjogMTMzMjU1CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9D aGFuZ2VMb2cgYi9Tb3VyY2UvV2ViQ29yZS9DaGFuZ2VMb2cKaW5kZXggZGI3YjMwMThhZDhiYzc1 MjliNTI5NWU4MzVkZTQ3YjA5YWFiZjJlZC4uMmRjZWZlM2ZjYmM1ZjQ3YzhjYzk1YmI3OTZmZjkw MTU1MjU1MTg4YSAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCisrKyBiL1Nv dXJjZS9XZWJDb3JlL0NoYW5nZUxvZwpAQCAtMSwzICsxLDE4IEBACisyMDEyLTExLTAyICBQZXRl ciBXYW5nICA8cGV0ZXIud2FuZ0B0b3JjaG1vYmlsZS5jb20uY24+CisKKyAgICAgICAgV2ViIElu c3BlY3RvcjogYSBzbWFsbCBkZWZlY3QgaW4gIldvcmtlcnNTaWRlYmFyUGFuZWwuanMiCisgICAg ICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0xMDEwMjIKKworICAg ICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICBJbiAiYXV0b2F0dGFj aFRvV29ya2Vyc0NsaWNrZWQiLCB0aGUgImV2ZW50LnRhcmdldC5jaGVja2VkIiBkb2Vzbid0IGFs d3lhcyB2YWxpZCwgc2luY2UgdGhlCisgICAgICAgICJldmVudC50YXJnZXQiIG1pZ2h0IGJlIGEg bGFiZWwuCisKKyAgICAgICAgVGhlIHByb2JsZW0ganVzdCBjYXVzZXMgYW4gZXJyb3IgcmVwb3J0 LCBkb2Vzbid0IGVmZmVjdCBmdW5jdGlvbiwgc28gbm8gbmV3IHRlc3QgY2FzZS4KKworICAgICAg ICAqIGluc3BlY3Rvci9mcm9udC1lbmQvV29ya2Vyc1NpZGViYXJQYW5lLmpzOgorICAgICAgICAo V2ViSW5zcGVjdG9yLldvcmtlcnNTaWRlYmFyUGFuZS5wcm90b3R5cGUuX2F1dG9hdHRhY2hUb1dv cmtlcnNDbGlja2VkKToKKwogMjAxMi0xMS0wMSAgQW1pIEZpc2NobWFuICA8ZmlzY2htYW5AY2hy b21pdW0ub3JnPgogCiAgICAgICAgIEhUTUxNZWRpYVBsYXllciBzaG91bGQgZnJlZSBtX3BsYXll ciB3aGVuIHNyYyBpcyBzZXQvY2hhbmdlZApkaWZmIC0tZ2l0IGEvU291cmNlL1dlYkNvcmUvaW5z cGVjdG9yL2Zyb250LWVuZC9Xb3JrZXJzU2lkZWJhclBhbmUuanMgYi9Tb3VyY2UvV2ViQ29yZS9p bnNwZWN0b3IvZnJvbnQtZW5kL1dvcmtlcnNTaWRlYmFyUGFuZS5qcwppbmRleCAzZTM0NDY4MGM3 YzM4NzI0NjQ5YzExMTlkNWRkODRiZjgwY2NjNzAzLi4xZWFlNmJmODNjZGU2Y2UwM2E1NDgwMDg5 NDhkZTVmOWJhOTNhNzdjIDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViQ29yZS9pbnNwZWN0b3IvZnJv bnQtZW5kL1dvcmtlcnNTaWRlYmFyUGFuZS5qcworKysgYi9Tb3VyY2UvV2ViQ29yZS9pbnNwZWN0 b3IvZnJvbnQtZW5kL1dvcmtlcnNTaWRlYmFyUGFuZS5qcwpAQCAtMTE3LDcgKzExNyw3IEBAIFdl Ykluc3BlY3Rvci5Xb3JrZXJzU2lkZWJhclBhbmUucHJvdG90eXBlID0gewogCiAgICAgX2F1dG9h dHRhY2hUb1dvcmtlcnNDbGlja2VkOiBmdW5jdGlvbihldmVudCkKICAgICB7Ci0gICAgICAgIFdv cmtlckFnZW50LnNldEF1dG9jb25uZWN0VG9Xb3JrZXJzKGV2ZW50LnRhcmdldC5jaGVja2VkKTsK KyAgICAgICAgV29ya2VyQWdlbnQuc2V0QXV0b2Nvbm5lY3RUb1dvcmtlcnModGhpcy5fZW5hYmxl V29ya2Vyc0NoZWNrYm94LmNoZWNrZWQpOwogICAgIH0sCiAKICAgICBfX3Byb3RvX186IFdlYklu c3BlY3Rvci5TaWRlYmFyUGFuZS5wcm90b3R5cGUK