142050 2015-02-26 11:48:58 -0800 Web Inspector: Set/Map appear as lossless when they have lossy entries 2015-02-26 17:19:45 -0800 1 1 1 Unclassified WebKit Web Inspector 528+ (Nightly build) All All RESOLVED FIXED DoNotImportToRadar P2 Normal --- 1 joepeck joepeck graouts joepeck jonowells mattbaker nvasilyev timothy webkit-bug-importer oldest_to_newest 1072448 0 joepeck 2015-02-26 11:48:58 -0800 * SUMMARY Set/Map appear as lossless when they have lossy entries * STEPS TO REPRODUCE 1. js> s = new Set; s.add(1); s.add([1,2,3, 4, 5, 6, {a:1}]); s.add("hello"); s.add({b:1}); s => Set should be expandable since the inner Object is lossy. 1072450 1 247436 joepeck 2015-02-26 11:51:20 -0800 Created attachment 247436 [PATCH] Proposed Fix 1072572 2 joepeck 2015-02-26 17:19:45 -0800 http://trac.webkit.org/changeset/180714 247436 2015-02-26 11:51:20 -0800 2015-02-26 11:53:03 -0800 [PATCH] Proposed Fix collection-lossy.patch text/plain 1903 joepeck ZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJJbnNwZWN0b3JVSS9DaGFuZ2VMb2cgYi9Tb3VyY2UvV2Vi SW5zcGVjdG9yVUkvQ2hhbmdlTG9nCmluZGV4IGMzNTE5NzguLmFmM2I0ZGQgMTAwNjQ0Ci0tLSBh L1NvdXJjZS9XZWJJbnNwZWN0b3JVSS9DaGFuZ2VMb2cKKysrIGIvU291cmNlL1dlYkluc3BlY3Rv clVJL0NoYW5nZUxvZwpAQCAtMSwzICsxLDE0IEBACisyMDE1LTAyLTI2ICBKb3NlcGggUGVjb3Jh cm8gIDxwZWNvcmFyb0BhcHBsZS5jb20+CisKKyAgICAgICAgV2ViIEluc3BlY3RvcjogU2V0L01h cCBhcHBlYXIgYXMgbG9zc2xlc3Mgd2hlbiB0aGV5IGhhdmUgbG9zc3kgZW50cmllcworICAgICAg ICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MTQyMDUwCisKKyAgICAg ICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgKiBVc2VySW50ZXJmYWNl L1ZpZXdzL09iamVjdFByZXZpZXdWaWV3LmpzOgorICAgICAgICAoV2ViSW5zcGVjdG9yLk9iamVj dFByZXZpZXdWaWV3LnByb3RvdHlwZS5fYXBwZW5kRW50cnlQcmV2aWV3cyk6CisgICAgICAgIFRh a2UgaW50byBhY2NvdW50IHRoZSBsb3NzbGVzcyBzdGF0ZXMgb2YgZW50cnkga2V5cyBhbmQgdmFs dWVzLgorCiAyMDE1LTAyLTI1ICBKb3NlcGggUGVjb3Jhcm8gIDxwZWNvcmFyb0BhcHBsZS5jb20+ CiAKICAgICAgICAgV2ViIEluc3BlY3RvcjogTmV3IE9iamVjdFRyZWUgVUkgZm9yIEFycmF5cyAv IE1hcHMgLyBTZXRzCmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViSW5zcGVjdG9yVUkvVXNlckludGVy ZmFjZS9WaWV3cy9PYmplY3RQcmV2aWV3Vmlldy5qcyBiL1NvdXJjZS9XZWJJbnNwZWN0b3JVSS9V c2VySW50ZXJmYWNlL1ZpZXdzL09iamVjdFByZXZpZXdWaWV3LmpzCmluZGV4IDczNGMyMzYuLjBh ZjMyZjYgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9XZWJJbnNwZWN0b3JVSS9Vc2VySW50ZXJmYWNlL1Zp ZXdzL09iamVjdFByZXZpZXdWaWV3LmpzCisrKyBiL1NvdXJjZS9XZWJJbnNwZWN0b3JVSS9Vc2Vy SW50ZXJmYWNlL1ZpZXdzL09iamVjdFByZXZpZXdWaWV3LmpzCkBAIC0xNDUsMTMgKzE0NSwxNyBA QCBXZWJJbnNwZWN0b3IuT2JqZWN0UHJldmlld1ZpZXcucHJvdG90eXBlID0gewogICAgICAgICAg ICAgaWYgKGkgPiAwKQogICAgICAgICAgICAgICAgIGVsZW1lbnQuYXBwZW5kQ2hpbGQoZG9jdW1l bnQuY3JlYXRlVGV4dE5vZGUoIiwgIikpOwogCisgICAgICAgICAgICB2YXIga2V5UHJldmlld0xv c3NsZXNzID0gdHJ1ZTsKICAgICAgICAgICAgIHZhciBlbnRyeSA9IHByZXZpZXcuY29sbGVjdGlv bkVudHJ5UHJldmlld3NbaV07CiAgICAgICAgICAgICBpZiAoZW50cnkua2V5UHJldmlldykgewot ICAgICAgICAgICAgICAgIHRoaXMuX2FwcGVuZFByZXZpZXcoZWxlbWVudCwgZW50cnkua2V5UHJl dmlldyk7CisgICAgICAgICAgICAgICAga2V5UHJldmlld0xvc3NsZXNzID0gdGhpcy5fYXBwZW5k UHJldmlldyhlbGVtZW50LCBlbnRyeS5rZXlQcmV2aWV3KTsKICAgICAgICAgICAgICAgICBlbGVt ZW50LmFwcGVuZENoaWxkKGRvY3VtZW50LmNyZWF0ZVRleHROb2RlKCIgPT4gIikpOwogICAgICAg ICAgICAgfQogCi0gICAgICAgICAgICB0aGlzLl9hcHBlbmRQcmV2aWV3KGVsZW1lbnQsIGVudHJ5 LnZhbHVlUHJldmlldyk7CisgICAgICAgICAgICB2YXIgdmFsdWVQcmV2aWV3TG9zc2xlc3MgPSB0 aGlzLl9hcHBlbmRQcmV2aWV3KGVsZW1lbnQsIGVudHJ5LnZhbHVlUHJldmlldyk7CisKKyAgICAg ICAgICAgIGlmICgha2V5UHJldmlld0xvc3NsZXNzIHx8ICF2YWx1ZVByZXZpZXdMb3NzbGVzcykK KyAgICAgICAgICAgICAgICBsb3NzbGVzcyA9IGZhbHNlOwogICAgICAgICB9CiAKICAgICAgICAg aWYgKHByZXZpZXcub3ZlcmZsb3cpCg==