Once GridResolvedPosition patch has landed (bug #131732) it'd be nice to refactor current loops to make them simpler. This has been already implemented in Blink and should be ported into WebKit (see https://codereview.chromium.org/235663004/).
Created attachment 233386 [details] Patch Small refactoring of GridResolvedPosition loops.
Comment on attachment 233386 [details] Patch Why can't you use a range based for loop?
(In reply to comment #2) > (From update of attachment 233386 [details]) > Why can't you use a range based for loop? Good idea, I'm changing the patch to use them. Thanks for the suggestion.
Created attachment 233392 [details] Patch New version using range-based for loops.
Comment on attachment 233392 [details] Patch iterators are awesome. r=me.
Comment on attachment 233392 [details] Patch Clearing flags on attachment: 233392 Committed r170182: <http://trac.webkit.org/changeset/170182>
All reviewed patches have been landed. Closing bug.