[webkit-reviews] review requested: [Bug 119061] Setting up OrderIterator shouldn't require an extra Vector : [Attachment 228585] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Fri Apr 4 05:03:39 PDT 2014


Manuel Rego Casasnovas <rego at igalia.com> has asked  for review:
Bug 119061: Setting up OrderIterator shouldn't require an extra Vector
https://bugs.webkit.org/show_bug.cgi?id=119061

Attachment 228585: Patch
https://bugs.webkit.org/attachment.cgi?id=228585&action=review

------- Additional Comments from Manuel Rego Casasnovas <rego at igalia.com>
This is a different approach partially based on Blink r153971.\n\nThe idea is
to avoid the extra loops in OrderIterator for each different order value. It
has some minor performance impact in the grid tests, however it drastically
improves the performance if you use different order values.\nOn top of that,
OrderIterator code is simplified.


More information about the webkit-reviews mailing list