87585 2012-05-26 20:58:14 -0700 [FileAPI] FileReader should fire progress event when blob has been completely read into memory 2012-05-27 00:34:23 -0700 1 1 1 Unclassified WebKit WebCore Misc. 528+ (Nightly build) All All RESOLVED FIXED P2 Normal --- 0 li.yin webkit-unassigned abarth ap haraken jianli webkit.review.bot oldest_to_newest 634709 0 li.yin 2012-05-26 20:58:14 -0700 From spec: http://www.w3.org/TR/FileAPI/#dfn-progress-event One progress event will fire when blob has been completely read into memory. Firefox, Opera and IE follows the spec. Webkit based browser doesn't do that, it only fires progress event at interval of 50ms. WebKit should add the behavior to make the conformance with the spec. 634717 1 144217 li.yin 2012-05-26 22:14:14 -0700 Created attachment 144217 Patch 634720 2 144217 haraken 2012-05-26 23:40:46 -0700 Comment on attachment 144217 Patch Looks OK. Thanks for improving the conformance with the spec. 634725 3 144217 webkit.review.bot 2012-05-27 00:34:18 -0700 Comment on attachment 144217 Patch Clearing flags on attachment: 144217 Committed r118622: <http://trac.webkit.org/changeset/118622> 634726 4 webkit.review.bot 2012-05-27 00:34:23 -0700 All reviewed patches have been landed. Closing bug. 144217 2012-05-26 22:14:14 -0700 2012-05-27 00:34:17 -0700 Patch bug-87585-20120528131745.patch text/plain 4524 li.yin U3VidmVyc2lvbiBSZXZpc2lvbjogMTE4NjIwCmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9D aGFuZ2VMb2cgYi9Tb3VyY2UvV2ViQ29yZS9DaGFuZ2VMb2cKaW5kZXggMjQ0ODZkZmE1N2ZlZjc4 NmJhNzQ5NTFjOTZiZDRjZmYyOTM0MTI4Zi4uMWNlMWQxYThlNmQ5ZGQ5ZDk1NGRhMWEzMGE5NDBj MjIwMjg5ZTFkZCAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCisrKyBiL1Nv dXJjZS9XZWJDb3JlL0NoYW5nZUxvZwpAQCAtMSwzICsxLDIxIEBACisyMDEyLTA1LTI3ICBMaSBZ aW4gIDxsaS55aW5AaW50ZWwuY29tPgorCisgICAgICAgIFtGaWxlQVBJXSBGaWxlUmVhZGVyIHNo b3VsZCBmaXJlIHByb2dyZXNzIGV2ZW50IHdoZW4gYmxvYiBoYXMgYmVlbiBjb21wbGV0ZWx5IHJl YWQgaW50byBtZW1vcnkKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcu Y2dpP2lkPTg3NTg1CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAg ICAgICAgRnJvbSBTcGVjOiBodHRwOi8vd3d3LnczLm9yZy9UUi9GaWxlQVBJLyNkZm4tcHJvZ3Jl c3MtZXZlbnQKKyAgICAgICAgT25lIHByb2dyZXNzIGV2ZW50IHdpbGwgZmlyZSB3aGVuIGJsb2Ig aGFzIGJlZW4gY29tcGxldGVseSByZWFkIGludG8gbWVtb3J5LgorICAgICAgICBGaXJlZm94LCBP cGVyYSBhbmQgSUUgZm9sbG93cyB0aGUgc3BlYy4KKyAgICAgICAgV2Via2l0IGJhc2VkIGJyb3dz ZXIgZG9lc24ndCBkbyB0aGF0LCBpdCBvbmx5IGZpcmVzIHByb2dyZXNzIGV2ZW50IGF0IGludGVy dmFsIG9mIDUwbXMuCisgICAgICAgIFdlYktpdCBzaG91bGQgYWRkIHRoZSBiZWhhdmlvciB0byBt YWtlIHRoZSBjb25mb3JtYW5jZSB3aXRoIHRoZSBzcGVjLgorCisgICAgICAgIFRlc3RzOiBmYXN0 L2ZpbGVzL2ZpbGUtcmVhZGVyLWV2ZW50LWxpc3RlbmVyLmh0bWwKKworICAgICAgICAqIGZpbGVh cGkvRmlsZVJlYWRlci5jcHA6CisgICAgICAgIChXZWJDb3JlOjpGaWxlUmVhZGVyOjpkaWRGaW5p c2hMb2FkaW5nKToKKwogMjAxMi0wNS0yNiAgTGkgWWluICA8bGkueWluQGludGVsLmNvbT4KIAog ICAgICAgICBbRmlsZUFQSV0gVGhlIHJlc3VsdCBhdHRyaWJ1dGUgb2YgRmlsZVJlYWRlciBzaHVv bGQgdXNlIG51bGwgdG8gcmVwbGFjZSBlbXB0eSBzdHJpbmcKZGlmZiAtLWdpdCBhL1NvdXJjZS9X ZWJDb3JlL2ZpbGVhcGkvRmlsZVJlYWRlci5jcHAgYi9Tb3VyY2UvV2ViQ29yZS9maWxlYXBpL0Zp bGVSZWFkZXIuY3BwCmluZGV4IDdkZDI5YWI3Y2I1OTJiOWM0ZGQ2MTI5YzA0MDVkY2EwMDAxNWVh OGQuLmFiN2MyYjNiMjBkODI2NDVkNmUzYjBiMjVmZjllYjI2MmUxMGMxNzIgMTAwNjQ0Ci0tLSBh L1NvdXJjZS9XZWJDb3JlL2ZpbGVhcGkvRmlsZVJlYWRlci5jcHAKKysrIGIvU291cmNlL1dlYkNv cmUvZmlsZWFwaS9GaWxlUmVhZGVyLmNwcApAQCAtMjE4LDYgKzIxOCw3IEBAIHZvaWQgRmlsZVJl YWRlcjo6ZGlkRmluaXNoTG9hZGluZygpCiAgICAgQVNTRVJUKG1fc3RhdGUgIT0gRE9ORSk7CiAg ICAgbV9zdGF0ZSA9IERPTkU7CiAKKyAgICBmaXJlRXZlbnQoZXZlbnROYW1lcygpLnByb2dyZXNz RXZlbnQpOwogICAgIGZpcmVFdmVudChldmVudE5hbWVzKCkubG9hZEV2ZW50KTsKICAgICBmaXJl RXZlbnQoZXZlbnROYW1lcygpLmxvYWRlbmRFdmVudCk7CiAgICAgCmRpZmYgLS1naXQgYS9MYXlv dXRUZXN0cy9DaGFuZ2VMb2cgYi9MYXlvdXRUZXN0cy9DaGFuZ2VMb2cKaW5kZXggNzM1N2JjODhk MTk1MGUxZjFkNDhkNjZmYmEzMWYxZDgxNTM4YjIxOS4uNTdjNTQxNjY2NWVlMjFkNzM2MjQyMzE3 NDI1NDc0MDFhNTRiMmUyMSAxMDA2NDQKLS0tIGEvTGF5b3V0VGVzdHMvQ2hhbmdlTG9nCisrKyBi L0xheW91dFRlc3RzL0NoYW5nZUxvZwpAQCAtMSwzICsxLDE5IEBACisyMDEyLTA1LTI3ICBMaSBZ aW4gIDxsaS55aW5AaW50ZWwuY29tPgorCisgICAgICAgIFtGaWxlQVBJXSBGaWxlUmVhZGVyIHNo b3VsZCBmaXJlIHByb2dyZXNzIGV2ZW50IHdoZW4gYmxvYiBoYXMgYmVlbiBjb21wbGV0ZWx5IHJl YWQgaW50byBtZW1vcnkKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcu Y2dpP2lkPTg3NTg1CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAg ICAgICAgRnJvbSBTcGVjOiBodHRwOi8vd3d3LnczLm9yZy9UUi9GaWxlQVBJLyNkZm4tcHJvZ3Jl c3MtZXZlbnQKKyAgICAgICAgT25lIHByb2dyZXNzIGV2ZW50IHdpbGwgZmlyZSB3aGVuIGJsb2Ig aGFzIGJlZW4gY29tcGxldGVseSByZWFkIGludG8gbWVtb3J5LgorICAgICAgICBGaXJlZm94LCBP cGVyYSBhbmQgSUUgZm9sbG93cyB0aGUgc3BlYy4KKyAgICAgICAgV2Via2l0IGJhc2VkIGJyb3dz ZXIgZG9lc24ndCBkbyB0aGF0LCBpdCBvbmx5IGZpcmVzIHByb2dyZXNzIGV2ZW50IGF0IGludGVy dmFsIG9mIDUwbXMuCisgICAgICAgIFdlYktpdCBzaG91bGQgYWRkIHRoZSBiZWhhdmlvciB0byBt YWtlIHRoZSBjb25mb3JtYW5jZSB3aXRoIHRoZSBzcGVjLgorCisgICAgICAgICogZmFzdC9maWxl cy9maWxlLXJlYWRlci1ldmVudC1saXN0ZW5lci1leHBlY3RlZC50eHQ6CisgICAgICAgICogZmFz dC9maWxlcy9maWxlLXJlYWRlci1ldmVudC1saXN0ZW5lci5odG1sOgorCiAyMDEyLTA1LTI2ICBM aSBZaW4gIDxsaS55aW5AaW50ZWwuY29tPgogCiAgICAgICAgIFtGaWxlQVBJXSBUaGUgcmVzdWx0 IGF0dHJpYnV0ZSBvZiBGaWxlUmVhZGVyIHNodW9sZCB1c2UgbnVsbCB0byByZXBsYWNlIGVtcHR5 IHN0cmluZwpkaWZmIC0tZ2l0IGEvTGF5b3V0VGVzdHMvZmFzdC9maWxlcy9maWxlLXJlYWRlci1l dmVudC1saXN0ZW5lci1leHBlY3RlZC50eHQgYi9MYXlvdXRUZXN0cy9mYXN0L2ZpbGVzL2ZpbGUt cmVhZGVyLWV2ZW50LWxpc3RlbmVyLWV4cGVjdGVkLnR4dAppbmRleCAxYTI5NTdmMTY2MDNmYmVj ZTc1MzYxZmZlYjdjYjAzMGFkNTcwMTllLi4xNjM5ZmVjMTJmNTE3YjM3ZDJlZTgyOGMwNTIwYjUz NDVhMjZmMDFiIDEwMDY0NAotLS0gYS9MYXlvdXRUZXN0cy9mYXN0L2ZpbGVzL2ZpbGUtcmVhZGVy LWV2ZW50LWxpc3RlbmVyLWV4cGVjdGVkLnR4dAorKysgYi9MYXlvdXRUZXN0cy9mYXN0L2ZpbGVz L2ZpbGUtcmVhZGVyLWV2ZW50LWxpc3RlbmVyLWV4cGVjdGVkLnR4dApAQCAtMiw2ICsyLDcgQEAK IFRlc3QgdGhhdCBGaWxlUmVhZGVyJ3MgZXZlbnQgbGlzdGVuZXIgd29ya3MuCiBSZWNlaXZlZCBs b2Fkc3RhcnQgZXZlbnQKIFJlY2VpdmVkIGZvbyBldmVudAorUmVjZWl2ZWQgcHJvZ3Jlc3MgZXZl bnQKIFJlY2VpdmVkIGxvYWQgZXZlbnQKIFJlY2VpdmVkIGxvYWRlbmQgZXZlbnQKIERPTkUKZGlm ZiAtLWdpdCBhL0xheW91dFRlc3RzL2Zhc3QvZmlsZXMvZmlsZS1yZWFkZXItZXZlbnQtbGlzdGVu ZXIuaHRtbCBiL0xheW91dFRlc3RzL2Zhc3QvZmlsZXMvZmlsZS1yZWFkZXItZXZlbnQtbGlzdGVu ZXIuaHRtbAppbmRleCAyNGY2MDQ2ZGM1NzUzM2I5ZDdjYzE4ZmYzMmFjMjQ4NTNlOTMyNWI5Li41 OGRlYjE1NWRiMGU4ZWZhZTkzNmVmMWI1ZTU5ZGQ0MDc1MWI0ZWQ5IDEwMDY0NAotLS0gYS9MYXlv dXRUZXN0cy9mYXN0L2ZpbGVzL2ZpbGUtcmVhZGVyLWV2ZW50LWxpc3RlbmVyLmh0bWwKKysrIGIv TGF5b3V0VGVzdHMvZmFzdC9maWxlcy9maWxlLXJlYWRlci1ldmVudC1saXN0ZW5lci5odG1sCkBA IC02LDYgKzYsNyBAQAogCiA8c2NyaXB0PgogdmFyIGZpbGVSZWFkZXI7Cit2YXIgcmVjZWl2ZWRQ cm9ncmVzc0V2ZW50ID0gZmFsc2U7CiAKIGZ1bmN0aW9uIGxvZyhtZXNzYWdlKQogewpAQCAtMjMs OCArMjQsMTUgQEAgZnVuY3Rpb24gb25Mb2FkU3RhcnQoKQogICAgIGZpbGVSZWFkZXIuZGlzcGF0 Y2hFdmVudChldmVudCk7CiB9CiAKK2Z1bmN0aW9uIG9uUHJvZ3Jlc3MoKQoreworICAgIHJlY2Vp dmVkUHJvZ3Jlc3NFdmVudCA9IHRydWU7Cit9CisKIGZ1bmN0aW9uIG9uTG9hZCgpCiB7CisgICAg aWYgKHJlY2VpdmVkUHJvZ3Jlc3NFdmVudCkKKyAgICAgICAgbG9nKCJSZWNlaXZlZCBwcm9ncmVz cyBldmVudCIpOwogICAgIGxvZygiUmVjZWl2ZWQgbG9hZCBldmVudCIpOwogfQogCkBAIC02NCw2 ICs3Miw3IEBAIGZ1bmN0aW9uIG9uSW5wdXRGaWxlQ2hhbmdlKCkKICAgICBmaWxlUmVhZGVyID0g bmV3IEZpbGVSZWFkZXIoKTsKICAgICBmaWxlUmVhZGVyLnJlYWRBc1RleHQoZmlsZSk7CiAgICAg ZmlsZVJlYWRlci5hZGRFdmVudExpc3RlbmVyKCJsb2Fkc3RhcnQiLCBvbkxvYWRTdGFydCwgZmFs c2UpOworICAgIGZpbGVSZWFkZXIuYWRkRXZlbnRMaXN0ZW5lcigicHJvZ3Jlc3MiLCBvblByb2dy ZXNzLCBmYWxzZSk7CiAgICAgZmlsZVJlYWRlci5hZGRFdmVudExpc3RlbmVyKCJsb2FkIiwgb25M b2FkLCBmYWxzZSk7CiAgICAgZmlsZVJlYWRlci5hZGRFdmVudExpc3RlbmVyKCJsb2FkZW5kIiwg b25Mb2FkRW5kLCBmYWxzZSk7CiAgICAgZmlsZVJlYWRlci5hZGRFdmVudExpc3RlbmVyKCJhYm9y dCIsIG9uQWJvcnQsIGZhbHNlKTsK