Summary: | [CSS Grid Layout] Add GridSpan::iterator | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Manuel Rego Casasnovas <rego> | ||||||
Component: | Layout and Rendering | Assignee: | Manuel Rego Casasnovas <rego> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | andersca, bjonesbe, commit-queue, darin, esprehn+autocc, glenn, jfernandez, kondapallykalyan, simon.fraser, svillar | ||||||
Priority: | P2 | ||||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Unspecified | ||||||||
Attachments: |
|
Description
Manuel Rego Casasnovas
2014-06-19 03:28:42 PDT
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. |