RESOLVED FIXED 133213
[CSS Grid Layout] Use gridRowCount() and gridColumnCount() when possible 
https://bugs.webkit.org/show_bug.cgi?id=133213
Summary [CSS Grid Layout] Use gridRowCount() and gridColumnCount() when possible
Manuel Rego Casasnovas
Reported 2014-05-23 05:09:52 PDT
We're not being consistent in RenderGrid code and sometimes we use m_grid.size() and other gridRowCount() (and the same for columns). This is a simple patch to use gridRowCount() and gridColumnCount() always when it's possible. Which will make the code more descriptive and easier to understand. BTW, a similar patch has already landed in Blink: https://codereview.chromium.org/235713003/
Attachments
Patch (2.75 KB, patch)
2014-05-23 05:19 PDT, Manuel Rego Casasnovas 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Manuel Rego Casasnovas
Comment 1 2014-05-23 05:19:32 PDT
Created attachment 231958 [details] Patch
Sergio Villar Senin
Comment 2 2014-05-23 08:13:01 PDT
Comment on attachment 231958 [details] Patch Nice cleanup
WebKit Commit Bot
Comment 3 2014-05-23 13:31:51 PDT
Comment on attachment 231958 [details] Patch Clearing flags on attachment: 231958 Committed r169277: <http://trac.webkit.org/changeset/169277>
WebKit Commit Bot
Comment 4 2014-05-23 13:31:54 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.