On 2012/03/16 05:06:10, Jasvir wrote:
LGTM
http://codereview.appspot.com/5836053/diff/5001/tests/com/google/caja/plugin/css-stylesheet-tests.js
File tests/com/google/caja/plugin/css-stylesheet-tests.js (right):
http://codereview.appspot.com/5836053/diff/5001/tests/com/google/caja/plugin/css-stylesheet-tests.js#newcode144
tests/com/google/caja/plugin/css-stylesheet-tests.js:144: "altGolden":
'a{font:12pt "times new roman" , "times" ,'
Aren't font names case sensitive?
Hrmm. http://www.w3.org/TR/css3-syntax/ says
The following rules always hold:
All CSS style sheets are case-insensitive, except for parts that are not
under the control of CSS. For example, the case-sensitivity of values of
the HTML attributes "id" and "class", of font names, and of URIs lies
outside the scope of this specification. Note in particular that element
names are case-insensitive in HTML, but case-sensitive in XML.
http://www.w3.org/TR/css3-fonts/ contradicts that
For other family names, the user agent attempts to find the family
name among fonts defined via @font-face rules and then among available
system fonts, matching names with a case-insensitive comparison.
and that is specifically for font-families which seems controlling.
http://codereview.appspot.com/5836053/
