46044 2010-09-18 17:05:45 -0700 Gradient: Pre-reserve space for 2 color stops 2010-09-18 17:23:50 -0700 1 1 1 Unclassified WebKit Layout and Rendering 528+ (Nightly build) All All RESOLVED FIXED Performance P2 Normal --- 1 kling webkit-unassigned oliver oldest_to_newest 281218 0 kling 2010-09-18 17:05:45 -0700 Most gradients have only 2 color stops. We can avoid growing the Vector<ColorStop> dynamically in the common case by using an inlineCapacity of 2. This yields a 0.4% speedup on Parcycle <http://www.mrspeaker.net/dev/parcycle/> 281219 1 68019 kling 2010-09-18 17:06:54 -0700 Created attachment 68019 Proposed patch 281220 2 68019 oliver 2010-09-18 17:18:05 -0700 Comment on attachment 68019 Proposed patch r=me, i suspect this will actually save memory in the common case 281221 3 68019 kling 2010-09-18 17:23:40 -0700 Comment on attachment 68019 Proposed patch Clearing flags on attachment: 68019 Committed r67801: <http://trac.webkit.org/changeset/67801> 281222 4 kling 2010-09-18 17:23:50 -0700 All reviewed patches have been landed. Closing bug. 68019 2010-09-18 17:06:54 -0700 2010-09-18 17:23:40 -0700 Proposed patch bug-46044.diff text/plain 1164 kling ZGlmZiAtLWdpdCBhL1dlYkNvcmUvQ2hhbmdlTG9nIGIvV2ViQ29yZS9DaGFuZ2VMb2cKaW5kZXgg ZGE3ZDIyNy4uMzk4ZjM0OSAxMDA2NDQKLS0tIGEvV2ViQ29yZS9DaGFuZ2VMb2cKKysrIGIvV2Vi Q29yZS9DaGFuZ2VMb2cKQEAgLTEsMyArMSwxNSBAQAorMjAxMC0wOS0xOCAgQW5kcmVhcyBLbGlu ZyAgPGFuZHJlYXMua2xpbmdAbm9raWEuY29tPgorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9E WSAoT09QUyEpLgorCisgICAgICAgIEdyYWRpZW50OiBQcmUtcmVzZXJ2ZSBzcGFjZSBmb3IgMiBj b2xvciBzdG9wcworICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/ aWQ9NDYwNDQKKworICAgICAgICBNb3N0IGdyYWRpZW50cyBoYXZlIG9ubHkgMiBjb2xvciBzdG9w cywgcHJlLXJlc2VydmUgc3BhY2UgZm9yIHRoZXNlIGluIG1fc3RvcHMKKyAgICAgICAgdG8gYXZv aWQgZ3Jvd2luZyB0aGUgdmVjdG9yIGR5bmFtaWNhbGx5IGluIHRoZSBjb21tb24gY2FzZS4KKwor ICAgICAgICAqIHBsYXRmb3JtL2dyYXBoaWNzL0dyYWRpZW50Lmg6CisKIDIwMTAtMDktMTggIE1p aGFpIFBhcnBhcml0YSAgPG1paGFpcEBjaHJvbWl1bS5vcmc+CiAKICAgICAgICAgUmV2aWV3ZWQg YnkgU2ltb24gRnJhc2VyLgpkaWZmIC0tZ2l0IGEvV2ViQ29yZS9wbGF0Zm9ybS9ncmFwaGljcy9H cmFkaWVudC5oIGIvV2ViQ29yZS9wbGF0Zm9ybS9ncmFwaGljcy9HcmFkaWVudC5oCmluZGV4IDYw NTJhZWYuLjVmOTQxNmEgMTAwNjQ0Ci0tLSBhL1dlYkNvcmUvcGxhdGZvcm0vZ3JhcGhpY3MvR3Jh ZGllbnQuaAorKysgYi9XZWJDb3JlL3BsYXRmb3JtL2dyYXBoaWNzL0dyYWRpZW50LmgKQEAgLTE0 Miw3ICsxNDIsNyBAQCBuYW1lc3BhY2UgV2ViQ29yZSB7CiAgICAgICAgIEZsb2F0UG9pbnQgbV9w MTsKICAgICAgICAgZmxvYXQgbV9yMDsKICAgICAgICAgZmxvYXQgbV9yMTsKLSAgICAgICAgbXV0 YWJsZSBWZWN0b3I8Q29sb3JTdG9wPiBtX3N0b3BzOworICAgICAgICBtdXRhYmxlIFZlY3RvcjxD b2xvclN0b3AsIDI+IG1fc3RvcHM7CiAgICAgICAgIG11dGFibGUgYm9vbCBtX3N0b3BzU29ydGVk OwogICAgICAgICBtdXRhYmxlIGludCBtX2xhc3RTdG9wOwogICAgICAgICBHcmFkaWVudFNwcmVh ZE1ldGhvZCBtX3NwcmVhZE1ldGhvZDsK