Stylesheet rule.cssText of content property should be quoted when the value is string.

PropertyContent's value should be quoted when the value is string.

Since PropertyContent's value is CSSValueList, added forceQuoteString parameter to customCssText of CSSValueList and CSSPrimitiveValue.

BUG=247231
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=153033

[tasak, 2013-06-10 12:35:53]

To avoid regression, I didn't use quoteCSSString in CSSPrimitiveValue.cpp
instead of quoteCSSStringIfNeeded.

I found that animationName property also uses string and  its cssText should not
be quoted.

[dominicc (has gone to gerrit), 2013-06-11 01:32:46]

Nit inline.

https://codereview.chromium.org/16736004/diff/1/Source/core/css/StyleProperty...
File Source/core/css/StylePropertySerializer.cpp (right):

https://codereview.chromium.org/16736004/diff/1/Source/core/css/StyleProperty...
Source/core/css/StylePropertySerializer.cpp:91: value =
toCSSValueList(property.value())->customCssText(true);
Would you consider using an enum instead of a bool? This callsite is mysterious.
But customCssText(QuoteString) is less mysterious. (You can probably think of a
better name than QuoteString.)

[tasak, 2013-06-11 11:29:14]

Thank you for reviewing.

https://codereview.chromium.org/16736004/diff/1/Source/core/css/StyleProperty...
File Source/core/css/StylePropertySerializer.cpp (right):

https://codereview.chromium.org/16736004/diff/1/Source/core/css/StyleProperty...
Source/core/css/StylePropertySerializer.cpp:91: value =
toCSSValueList(property.value())->customCssText(true);
On 2013/06/11 01:32:46, dominicc wrote:
> Would you consider using an enum instead of a bool? This callsite is
mysterious.
> But customCssText(QuoteString) is less mysterious. (You can probably think of
a
> better name than QuoteString.)

Sure. Done.

[dominicc (has gone to gerrit), 2013-06-11 21:06:27]

On 2013/06/11 11:29:14, tasak wrote:
> Sure. Done.

Looks splendid.

eseidel--this is ready for you to review.

[eseidel, 2013-06-25 21:13:59]

lgtm

Sorry, I've been on vacation.

[commit-bot: I haz the power, 2013-06-25 21:15:00]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/tasak@google.com/16736004/6001

[commit-bot: I haz the power, 2013-06-25 22:43:34]

Change committed as 153033