RESOLVED FIXED 154472
[css-grid] Swap columns and rows in grid-template shorthand 
https://bugs.webkit.org/show_bug.cgi?id=154472
Summary [css-grid] Swap columns and rows in grid-template shorthand
Manuel Rego Casasnovas
Reported Friday, February 19, 2016 10:29:06 PM UTC
[css-grid] Swap columns and rows in grid-template shorthand
Attachments
Patch (34.31 KB, patch)
2016-02-19 14:29 PST, Manuel Rego Casasnovas 		no flags
DetailsFormatted DiffDiff
Patch (34.41 KB, patch)
2016-02-22 05:22 PST, Manuel Rego Casasnovas 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Manuel Rego Casasnovas
Comment 1 Friday, February 19, 2016 10:29:42 PM UTC
Created attachment 271801 [details] Patch
Manuel Rego Casasnovas
Comment 2 Friday, February 19, 2016 10:31:11 PM UTC
This is a patch ported from Blink: https://crrev.com/3d6b5d3df0a988475328dc5a4aabce5c4f4aa211
Darin Adler
Comment 3 Monday, February 22, 2016 1:32:56 AM UTC
Comment on attachment 271801 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=271801&action=review > Source/WebCore/css/CSSParser.cpp:5567 > + if (RefPtr<CSSValue> columnsValue = parseGridTrackList()) { I think auto is better here than naming the type. > Source/WebCore/css/CSSParser.cpp:5659 > + RefPtr<CSSValue> rowsValue = nullptr; No need for the "= nullptr" here. 1) All new RefPtr objects are null. 2) All code paths below initialize rowsValue anyway. > Source/WebCore/css/CSSParser.cpp:5666 > + RefPtr<CSSValue> columnsValue = parseGridTemplateColumns(); I would prefer auto here to naming the type.
Manuel Rego Casasnovas
Comment 4 Monday, February 22, 2016 1:22:12 PM UTC
Created attachment 271917 [details] Patch
WebKit Commit Bot
Comment 5 Monday, February 22, 2016 2:08:38 PM UTC
Comment on attachment 271917 [details] Patch Clearing flags on attachment: 271917 Committed r196934: <http://trac.webkit.org/changeset/196934>
WebKit Commit Bot
Comment 6 Monday, February 22, 2016 2:08:42 PM UTC
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.