Centralise !important handling.

Document background-attachment parser and make it behave correctly.

svn path=/trunk/libcss/; revision=7856

1 files changed, 0 insertions, 45 deletions

diff --git a/src/parse/properties/generated_list.c b/src/parse/properties/generated_list.c
index 6c9c1c7..970da2a 100644
--- a/src/parse/properties/generated_list.c
+++ b/src/parse/properties/generated_list.c
@@ -69,10 +69,6 @@ css_error parse_content(css_language *c,
 		required_size += len;
 	}
 
-	error = parse_important(c, vector, &temp_ctx, &flags);
-	if (error != CSS_OK)
-		return error;
-
 	opv = buildOPV(CSS_PROP_CONTENT, flags, value);
 
 	/* Allocate result */
@@ -101,11 +97,6 @@ css_error parse_content(css_language *c,
 			return error;
 	}
 
-	/* Ensure we skip past !important */
-	error = parse_important(c, vector, ctx, &flags);
-	if (error != CSS_OK)
-		return error;
-
 	return CSS_OK;
 }
 
@@ -185,10 +176,6 @@ css_error parse_counter_increment(css_language *c,
 		required_size += sizeof(opv);
 	}
 
-	error = parse_important(c, vector, &temp_ctx, &flags);
-	if (error != CSS_OK)
-		return error;
-
 	opv = buildOPV(CSS_PROP_COUNTER_INCREMENT, flags, value);
 
 	/* Allocate result */
@@ -270,10 +257,6 @@ css_error parse_counter_increment(css_language *c,
 		ptr += sizeof(opv);
 	}
 
-	error = parse_important(c, vector, ctx, &flags);
-	if (error != CSS_OK)
-		return error;
-
 	return CSS_OK;
 }
 
@@ -353,10 +336,6 @@ css_error parse_counter_reset(css_language *c,
 		required_size += sizeof(opv);
 	}
 
-	error = parse_important(c, vector, &temp_ctx, &flags);
-	if (error != CSS_OK)
-		return error;
-
 	opv = buildOPV(CSS_PROP_COUNTER_RESET, flags, value);
 
 	/* Allocate result */
@@ -438,10 +417,6 @@ css_error parse_counter_reset(css_language *c,
 		ptr += sizeof(opv);
 	}
 
-	error = parse_important(c, vector, ctx, &flags);
-	if (error != CSS_OK)
-		return error;
-
 	return CSS_OK;
 }
 
@@ -462,10 +437,6 @@ css_error parse_list_style_image(css_language *c,
 			token->type != CSS_TOKEN_URI))
 		return CSS_INVALID;
 
-	error = parse_important(c, vector, ctx, &flags);
-	if (error != CSS_OK)
-		return error;
-
 	if (token->type == CSS_TOKEN_IDENT && 
 			token->ilower == c->strings[INHERIT]) {
 		flags |= FLAG_INHERIT;
@@ -515,10 +486,6 @@ css_error parse_list_style_position(css_language *c,
 	if (ident == NULL || ident->type != CSS_TOKEN_IDENT)
 		return CSS_INVALID;
 
-	error = parse_important(c, vector, ctx, &flags);
-	if (error != CSS_OK)
-		return error;
-
 	if (ident->ilower == c->strings[INHERIT]) {
 		flags |= FLAG_INHERIT;
 	} else if (ident->ilower == c->strings[INSIDE]) {
@@ -560,10 +527,6 @@ css_error parse_list_style_type(css_language *c,
 	if (ident == NULL || ident->type != CSS_TOKEN_IDENT)
 		return CSS_INVALID;
 
-	error = parse_important(c, vector, ctx, &flags);
-	if (error != CSS_OK)
-		return error;
-
 	if (ident->ilower == c->strings[INHERIT]) {
 		flags |= FLAG_INHERIT;
 	} else {
@@ -658,10 +621,6 @@ css_error parse_quotes(css_language *c,
 		required_size += sizeof(opv);
 	}
 
-	error = parse_important(c, vector, &temp_ctx, &flags);
-	if (error != CSS_OK)
-		return error;
-
 	opv = buildOPV(CSS_PROP_QUOTES, flags, value);
 
 	/* Allocate result */
@@ -736,10 +695,6 @@ css_error parse_quotes(css_language *c,
 		ptr += sizeof(opv);
 	}
 
-	error = parse_important(c, vector, ctx, &flags);
-	if (error != CSS_OK)
-		return error;
-
 	return CSS_OK;
 }