Bug 204508 – [css-grid] Serialization of grid-area, grid-row and grid-column should include "/" separator

• Component: CSS
• Status: RESOLVED FIXED

[Manuel Rego Casasnovas]

Currently this test fails http://w3c-test.org/css/css-grid/parsing/grid-area-valid.html because of serialization of grid-area, grid-row and grid-column doesn't include the "/" separator.

This works fine in Chromium and Firefox.

[Manuel Rego Casasnovas]

Created attachment 384345 [details]
Patch

[EWS Watchlist]

Comment on attachment 384345 [details]
Patch

Attachment 384345 [details] did not pass win-ews (win):
Output: https://webkit-queues.webkit.org/results/13279838

New failing tests:
css3/filters/blur-various-radii.html

[EWS Watchlist]

Created attachment 384365 [details]
Archive of layout-test-results from ews214 for win-future

The attached test failures were seen while running run-webkit-tests on the win-ews.
Bot: ews214  Port: win-future  Platform: CYGWIN_NT-10.0-17763-3.0.5-338.x86_64-x86_64-64bit

[Javier Fernandez]

Comment on attachment 384345 [details]
Patch

r=me as long as the bots are green before landing.

[Manuel Rego Casasnovas]

Comment on attachment 384345 [details]
Patch

Windows EWS failure is unrelated and the rest of EWS are green.

[WebKit Commit Bot]

Comment on attachment 384345 [details]
Patch

Clearing flags on attachment: 384345

Committed r252901: <https://trac.webkit.org/changeset/252901>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.

[Radar WebKit Bug Importer]

<rdar://problem/57504966>

[Oriol Brufau]

*** Bug 197067 has been marked as a duplicate of this bug. ***

[Darin Adler]

Comment on attachment 384345 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=384345&action=review

> Source/WebCore/css/StyleProperties.h:163
> +    String getShorthandValue(const StylePropertyShorthand&, const String& separator = " ") const;

Using a string object here is unnecessarily inefficient, creating and destroying a string every time this function is called. The argument should be typed const char* instead.Nothing else about the code would need to change.

[Manuel Rego Casasnovas]

(In reply to Darin Adler from comment #10)
> Comment on attachment 384345 [details]
> Patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=384345&action=review
> 
> > Source/WebCore/css/StyleProperties.h:163
> > +    String getShorthandValue(const StylePropertyShorthand&, const String& separator = " ") const;
> 
> Using a string object here is unnecessarily inefficient, creating and
> destroying a string every time this function is called. The argument should
> be typed const char* instead.Nothing else about the code would need to
> change.

Sorry about that, I've a patch to fix it at https://bugs.webkit.org/show_bug.cgi?id=204644