Bug 1295271 - Remove all _moz_ prefixes from all instances of _moz_column
Command used:
find -E ./* -iregex '.*(\.cpp|\.h)' | xargs egrep -l '.*_moz_column.*' | xargs sed -i '' 's/_moz_column/column/g'
MozReview-Commit-ID: CJbSfDfhmle
--- a/layout/generic/nsColumnSetFrame.cpp
+++ b/layout/generic/nsColumnSetFrame.cpp
@@ -78,17 +78,17 @@ nsColumnSetFrame::PaintColumnRule(nsRend
ruleStyle = colStyle->mColumnRuleStyle;
nsPresContext* presContext = PresContext();
nscoord ruleWidth = colStyle->GetComputedColumnRuleWidth();
if (!ruleWidth)
return;
nscolor ruleColor =
- GetVisitedDependentColor(eCSSProperty__moz_column_rule_color);
+ GetVisitedDependentColor(eCSSProperty_column_rule_color);
// In order to re-use a large amount of code, we treat the column rule as a border.
// We create a new border style object and fill in all the details of the column rule as
// the left border. PaintBorder() does all the rendering for us, so we not
// only save an enormous amount of code but we'll support all the line styles that
// we support on borders!
nsStyleBorder border(presContext);
Sides skipSides;
--- a/layout/style/Declaration.cpp
+++ b/layout/style/Declaration.cpp
@@ -737,17 +737,17 @@ Declaration::GetValue(nsCSSPropertyID aP
case eCSSProperty_border_top:
case eCSSProperty_border_right:
case eCSSProperty_border_bottom:
case eCSSProperty_border_left:
case eCSSProperty_border_inline_start:
case eCSSProperty_border_inline_end:
case eCSSProperty_border_block_start:
case eCSSProperty_border_block_end:
- case eCSSProperty__moz_column_rule:
+ case eCSSProperty_column_rule:
case eCSSProperty_outline: {
const nsCSSPropertyID* subprops =
nsCSSProps::SubpropertyEntryFor(aProperty);
MOZ_ASSERT(StringEndsWith(nsCSSProps::GetStringValue(subprops[2]),
NS_LITERAL_CSTRING("-color")),
"third subprop must be the color property");
const nsCSSValue *colorValue = data->ValueFor(subprops[2]);
bool isMozUseTextColor =
@@ -1143,17 +1143,17 @@ Declaration::GetValue(nsCSSPropertyID aP
const nsCSSValue &midValue =
*data->ValueFor(eCSSProperty_marker_mid);
const nsCSSValue &startValue =
*data->ValueFor(eCSSProperty_marker_start);
if (endValue == midValue && midValue == startValue)
AppendValueToString(eCSSProperty_marker_end, aValue, aSerialization);
break;
}
- case eCSSProperty__moz_columns: {
+ case eCSSProperty_columns: {
// Two values, column-count and column-width, separated by a space.
const nsCSSPropertyID* subprops =
nsCSSProps::SubpropertyEntryFor(aProperty);
AppendValueToString(subprops[0], aValue, aSerialization);
aValue.Append(char16_t(' '));
AppendValueToString(subprops[1], aValue, aSerialization);
break;
}
--- a/layout/style/StyleAnimationValue.cpp
+++ b/layout/style/StyleAnimationValue.cpp
@@ -3664,17 +3664,17 @@ StyleAnimationValue::ExtractComputedValu
GetComputedBorder().side_); \
break;
BORDER_WIDTH_CASE(eCSSProperty_border_bottom_width, bottom)
BORDER_WIDTH_CASE(eCSSProperty_border_left_width, left)
BORDER_WIDTH_CASE(eCSSProperty_border_right_width, right)
BORDER_WIDTH_CASE(eCSSProperty_border_top_width, top)
#undef BORDER_WIDTH_CASE
- case eCSSProperty__moz_column_rule_width:
+ case eCSSProperty_column_rule_width:
aComputedValue.SetCoordValue(
static_cast<const nsStyleColumn*>(styleStruct)->
GetComputedColumnRuleWidth());
break;
case eCSSProperty_border_bottom_color:
ExtractBorderColor(aStyleContext, styleStruct, NS_SIDE_BOTTOM,
aComputedValue);
@@ -3697,30 +3697,30 @@ StyleAnimationValue::ExtractComputedValu
static_cast<const nsStyleOutline*>(styleStruct);
nscolor color;
if (!styleOutline->GetOutlineColor(color))
color = aStyleContext->StyleColor()->mColor;
aComputedValue.SetColorValue(color);
break;
}
- case eCSSProperty__moz_column_rule_color: {
+ case eCSSProperty_column_rule_color: {
const nsStyleColumn *styleColumn =
static_cast<const nsStyleColumn*>(styleStruct);
nscolor color;
if (styleColumn->mColumnRuleColorIsForeground) {
color = aStyleContext->StyleColor()->mColor;
} else {
color = styleColumn->mColumnRuleColor;
}
aComputedValue.SetColorValue(color);
break;
}
- case eCSSProperty__moz_column_count: {
+ case eCSSProperty_column_count: {
const nsStyleColumn *styleColumn =
static_cast<const nsStyleColumn*>(styleStruct);
if (styleColumn->mColumnCount == NS_STYLE_COLUMN_COUNT_AUTO) {
aComputedValue.SetAutoValue();
} else {
aComputedValue.SetIntValue(styleColumn->mColumnCount,
eUnit_Integer);
}
--- a/layout/style/nsCSSParser.cpp
+++ b/layout/style/nsCSSParser.cpp
@@ -7453,19 +7453,19 @@ static const nsCSSPropertyID kBorderBloc
eCSSProperty_border_block_start_color
};
static const nsCSSPropertyID kBorderBlockEndIDs[] = {
eCSSProperty_border_block_end_width,
eCSSProperty_border_block_end_style,
eCSSProperty_border_block_end_color
};
static const nsCSSPropertyID kColumnRuleIDs[] = {
- eCSSProperty__moz_column_rule_width,
- eCSSProperty__moz_column_rule_style,
- eCSSProperty__moz_column_rule_color
+ eCSSProperty_column_rule_width,
+ eCSSProperty_column_rule_style,
+ eCSSProperty_column_rule_color
};
bool
CSSParserImpl::ParseEnum(nsCSSValue& aValue,
const KTableEntry aKeywordTable[])
{
nsSubstring* ident = NextIdent();
if (nullptr == ident) {
@@ -11635,19 +11635,19 @@ CSSParserImpl::ParsePropertyByFunction(n
return ParseBoxCornerRadius(aPropID);
case eCSSProperty_box_shadow:
case eCSSProperty_text_shadow:
return ParseShadowList(aPropID);
case eCSSProperty_clip:
return ParseRect(eCSSProperty_clip);
- case eCSSProperty__moz_columns:
+ case eCSSProperty_columns:
return ParseColumns();
- case eCSSProperty__moz_column_rule:
+ case eCSSProperty_column_rule:
return ParseBorderSide(kColumnRuleIDs, false);
case eCSSProperty_content:
return ParseContent();
case eCSSProperty_counter_increment:
case eCSSProperty_counter_reset:
return ParseCounterData(aPropID);
case eCSSProperty_cursor:
return ParseCursor();
@@ -13797,18 +13797,18 @@ CSSParserImpl::ParseColumns()
{
// We use a similar "fake value" hack to ParseListStyle, because
// "auto" is acceptable for both column-count and column-width.
// If the fake "auto" value is found, and one of the real values isn't,
// that means the fake auto value is meant for the real value we didn't
// find.
static const nsCSSPropertyID columnIDs[] = {
eCSSPropertyExtra_x_auto_value,
- eCSSProperty__moz_column_count,
- eCSSProperty__moz_column_width
+ eCSSProperty_column_count,
+ eCSSProperty_column_width
};
const int32_t numProps = MOZ_ARRAY_LENGTH(columnIDs);
nsCSSValue values[numProps];
int32_t found = ParseChoice(values, columnIDs, numProps);
if (found < 1) {
return false;
}
--- a/layout/style/nsCSSPropList.h
+++ b/layout/style/nsCSSPropList.h
@@ -1478,100 +1478,100 @@ CSS_PROP_SVG(
CSS_PROPERTY_PARSE_VALUE,
"",
VARIANT_HK,
kColorInterpolationKTable,
CSS_PROP_NO_OFFSET,
eStyleAnimType_Discrete)
CSS_PROP_COLUMN(
-moz-column-count,
- _moz_column_count,
+ column_count,
CSS_PROP_DOMPROP_PREFIXED(ColumnCount),
CSS_PROPERTY_PARSE_VALUE |
// Need to reject 0 in addition to negatives. If we accept 0, we
// need to change NS_STYLE_COLUMN_COUNT_AUTO to something else.
CSS_PROPERTY_VALUE_AT_LEAST_ONE,
"",
VARIANT_AHI,
nullptr,
offsetof(nsStyleColumn, mColumnCount),
eStyleAnimType_Custom)
CSS_PROP_COLUMN(
-moz-column-fill,
- _moz_column_fill,
+ column_fill,
CSS_PROP_DOMPROP_PREFIXED(ColumnFill),
CSS_PROPERTY_PARSE_VALUE,
"",
VARIANT_HK,
kColumnFillKTable,
CSS_PROP_NO_OFFSET,
eStyleAnimType_None)
CSS_PROP_COLUMN(
-moz-column-gap,
- _moz_column_gap,
+ column_gap,
CSS_PROP_DOMPROP_PREFIXED(ColumnGap),
CSS_PROPERTY_PARSE_VALUE |
CSS_PROPERTY_VALUE_NONNEGATIVE,
"",
VARIANT_HL | VARIANT_NORMAL | VARIANT_CALC,
nullptr,
offsetof(nsStyleColumn, mColumnGap),
eStyleAnimType_Coord)
CSS_PROP_SHORTHAND(
-moz-column-rule,
- _moz_column_rule,
+ column_rule,
CSS_PROP_DOMPROP_PREFIXED(ColumnRule),
CSS_PROPERTY_PARSE_FUNCTION,
"")
CSS_PROP_COLUMN(
-moz-column-rule-color,
- _moz_column_rule_color,
+ column_rule_color,
CSS_PROP_DOMPROP_PREFIXED(ColumnRuleColor),
CSS_PROPERTY_PARSE_VALUE |
CSS_PROPERTY_IGNORED_WHEN_COLORS_DISABLED,
"",
VARIANT_HCK,
kBorderColorKTable,
CSS_PROP_NO_OFFSET,
eStyleAnimType_Custom)
CSS_PROP_COLUMN(
-moz-column-rule-style,
- _moz_column_rule_style,
+ column_rule_style,
CSS_PROP_DOMPROP_PREFIXED(ColumnRuleStyle),
CSS_PROPERTY_PARSE_VALUE,
"",
VARIANT_HK,
kBorderStyleKTable,
CSS_PROP_NO_OFFSET,
eStyleAnimType_None)
CSS_PROP_COLUMN(
-moz-column-rule-width,
- _moz_column_rule_width,
+ column_rule_width,
CSS_PROP_DOMPROP_PREFIXED(ColumnRuleWidth),
CSS_PROPERTY_PARSE_VALUE |
CSS_PROPERTY_VALUE_NONNEGATIVE,
"",
VARIANT_HKL | VARIANT_CALC,
kBorderWidthKTable,
CSS_PROP_NO_OFFSET,
eStyleAnimType_Custom)
CSS_PROP_COLUMN(
-moz-column-width,
- _moz_column_width,
+ column_width,
CSS_PROP_DOMPROP_PREFIXED(ColumnWidth),
CSS_PROPERTY_PARSE_VALUE |
CSS_PROPERTY_VALUE_NONNEGATIVE,
"",
VARIANT_AHL | VARIANT_CALC,
nullptr,
offsetof(nsStyleColumn, mColumnWidth),
eStyleAnimType_Coord)
CSS_PROP_SHORTHAND(
-moz-columns,
- _moz_columns,
+ columns,
CSS_PROP_DOMPROP_PREFIXED(Columns),
CSS_PROPERTY_PARSE_FUNCTION,
"")
CSS_PROP_DISPLAY(
contain,
contain,
Contain,
CSS_PROPERTY_PARSE_VALUE |
--- a/layout/style/nsCSSProps.cpp
+++ b/layout/style/nsCSSProps.cpp
@@ -2837,27 +2837,27 @@ static const nsCSSPropertyID gOutlineSub
// It also depends on the color being third.
eCSSProperty_outline_width,
eCSSProperty_outline_style,
eCSSProperty_outline_color,
eCSSProperty_UNKNOWN
};
static const nsCSSPropertyID gColumnsSubpropTable[] = {
- eCSSProperty__moz_column_count,
- eCSSProperty__moz_column_width,
+ eCSSProperty_column_count,
+ eCSSProperty_column_width,
eCSSProperty_UNKNOWN
};
static const nsCSSPropertyID gColumnRuleSubpropTable[] = {
// nsCSSDeclaration.cpp outputs the subproperties in this order.
// It also depends on the color being third.
- eCSSProperty__moz_column_rule_width,
- eCSSProperty__moz_column_rule_style,
- eCSSProperty__moz_column_rule_color,
+ eCSSProperty_column_rule_width,
+ eCSSProperty_column_rule_style,
+ eCSSProperty_column_rule_color,
eCSSProperty_UNKNOWN
};
static const nsCSSPropertyID gFlexSubpropTable[] = {
eCSSProperty_flex_grow,
eCSSProperty_flex_shrink,
eCSSProperty_flex_basis,
eCSSProperty_UNKNOWN
--- a/layout/style/nsComputedDOMStylePropertyList.h
+++ b/layout/style/nsComputedDOMStylePropertyList.h
@@ -270,24 +270,24 @@ COMPUTED_STYLE_PROP(border_left_colors,
COMPUTED_STYLE_PROP(border_right_colors, BorderRightColors)
COMPUTED_STYLE_PROP(border_top_colors, BorderTopColors)
COMPUTED_STYLE_PROP(box_align, BoxAlign)
COMPUTED_STYLE_PROP(box_direction, BoxDirection)
COMPUTED_STYLE_PROP(box_flex, BoxFlex)
COMPUTED_STYLE_PROP(box_ordinal_group, BoxOrdinalGroup)
COMPUTED_STYLE_PROP(box_orient, BoxOrient)
COMPUTED_STYLE_PROP(box_pack, BoxPack)
-COMPUTED_STYLE_PROP(_moz_column_count, ColumnCount)
-COMPUTED_STYLE_PROP(_moz_column_fill, ColumnFill)
-COMPUTED_STYLE_PROP(_moz_column_gap, ColumnGap)
-//// COMPUTED_STYLE_PROP(_moz_column_rule, ColumnRule)
-COMPUTED_STYLE_PROP(_moz_column_rule_color, ColumnRuleColor)
-COMPUTED_STYLE_PROP(_moz_column_rule_style, ColumnRuleStyle)
-COMPUTED_STYLE_PROP(_moz_column_rule_width, ColumnRuleWidth)
-COMPUTED_STYLE_PROP(_moz_column_width, ColumnWidth)
+COMPUTED_STYLE_PROP(column_count, ColumnCount)
+COMPUTED_STYLE_PROP(column_fill, ColumnFill)
+COMPUTED_STYLE_PROP(column_gap, ColumnGap)
+//// COMPUTED_STYLE_PROP(column_rule, ColumnRule)
+COMPUTED_STYLE_PROP(column_rule_color, ColumnRuleColor)
+COMPUTED_STYLE_PROP(column_rule_style, ColumnRuleStyle)
+COMPUTED_STYLE_PROP(column_rule_width, ColumnRuleWidth)
+COMPUTED_STYLE_PROP(column_width, ColumnWidth)
COMPUTED_STYLE_PROP(float_edge, FloatEdge)
COMPUTED_STYLE_PROP(force_broken_image_icon, ForceBrokenImageIcon)
COMPUTED_STYLE_PROP(image_region, ImageRegion)
COMPUTED_STYLE_PROP(orient, Orient)
COMPUTED_STYLE_PROP(osx_font_smoothing, OsxFontSmoothing)
COMPUTED_STYLE_PROP(_moz_outline_radius_bottomLeft, OutlineRadiusBottomLeft)
COMPUTED_STYLE_PROP(_moz_outline_radius_bottomRight,OutlineRadiusBottomRight)
COMPUTED_STYLE_PROP(_moz_outline_radius_topLeft, OutlineRadiusTopLeft)
--- a/layout/style/nsStyleContext.cpp
+++ b/layout/style/nsStyleContext.cpp
@@ -1512,17 +1512,17 @@ nsStyleContext::GetVisitedDependentColor
{
NS_ASSERTION(aProperty == eCSSProperty_color ||
aProperty == eCSSProperty_background_color ||
aProperty == eCSSProperty_border_top_color ||
aProperty == eCSSProperty_border_right_color ||
aProperty == eCSSProperty_border_bottom_color ||
aProperty == eCSSProperty_border_left_color ||
aProperty == eCSSProperty_outline_color ||
- aProperty == eCSSProperty__moz_column_rule_color ||
+ aProperty == eCSSProperty_column_rule_color ||
aProperty == eCSSProperty_text_decoration_color ||
aProperty == eCSSProperty_text_emphasis_color ||
aProperty == eCSSProperty__webkit_text_fill_color ||
aProperty == eCSSProperty__webkit_text_stroke_color ||
aProperty == eCSSProperty_fill ||
aProperty == eCSSProperty_stroke,
"we need to add to nsStyleContext::CalcStyleDifference");