402706 - change names of new width keywords

Status: RESOLVED FIXED

(Core :: CSS Parsing and Computation, defect, P2)

Description

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

In bug 311415 I added support for some new keywords on the width property.  The CSS WG just agreed to add them to css3-box, but changed the names (which I was never happy with anyway).  So we should change our -moz-prefixed names to match the new names, since we haven't shipped the -moz-prefixed names in any release yet.

-moz-intrinsic => -moz-max-content
-moz-min-intrinsic => -moz-min-content
-moz-fill => -moz-available
-moz-shrink-wrap => -moz-fit-content

Comment 1

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Attachment: patch

I generated this patch by running

find . -type f -exec perl -pi /builds/trunk/working/subst.pl {} \;

in mozilla/layout, where subst.pl contained:

s/-moz-fill/-moz-available/g;
s/_moz_fill/_moz_available/g;
s/NS_STYLE_WIDTH_FILL/NS_STYLE_WIDTH_AVAILABLE/g;
s/-moz-intrinsic/-moz-max-content/g;
s/_moz_intrinsic/_moz_max_content/g;
s/NS_STYLE_WIDTH_INTRINSIC/NS_STYLE_WIDTH_MAX_CONTENT/g;
s/-moz-min-intrinsic/-moz-min-content/g;
s/_moz_min_intrinsic/_moz_min_content/g;
s/NS_STYLE_WIDTH_MIN_INTRINSIC/NS_STYLE_WIDTH_MIN_CONTENT/g;
s/-moz-shrink-wrap/-moz-fit-content/g;
s/_moz_shrink_wrap/_moz_fit_content/g;
s/NS_STYLE_WIDTH_SHRINK_WRAP/NS_STYLE_WIDTH_FIT_CONTENT/g;

and then I hand-edited:
layout/base/nsStyleConsts.h, to line up the whitespace
layout/style/nsCSSKeywordList.h, to put the lines back in alphabetical order

Comment 2

[Boris Zbarsky [:bzbarsky]]

Comment on attachment 288753 [details] [diff] [review]
patch

r+sr=bzbarsky

Comment 3

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Fix checked in to trunk.

Adding dev-doc-needed since we need to search-replace the docs on bug 311415 with this change.

Comment 4

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

I just fixed:
http://developer.mozilla.org/en/docs/Firefox_3_for_developers#CSS
http://developer.mozilla.org/en/docs/CSS:width#Values
http://developer.mozilla.org/en/docs/CSS:max-width#Values
http://developer.mozilla.org/en/docs/CSS:min-width#Values
but I'm not sure if anything else needs updating.

Comment 5

[Eric Shepherd [:sheppy]]

I fixed one last place; marking this as complete.