Attachment #199234 for bug #279187

Lines 1212-1218 Link Here

(-)branches/KDE/4.3/kdelibs/khtml/css/css_valueimpl.cpp (-1 / +3 lines)
1212	text = getValueName(m_value.ident);	1212	text = getValueName(m_value.ident);
1213	break;	1213	break;
1214	case CSSPrimitiveValue::CSS_ATTR:	1214	case CSSPrimitiveValue::CSS_ATTR:
1215	// ###	1215	text = "attr(";
		1216	text += DOMString( m_value.string );
		1217	text += ")";
1216	break;	1218	break;
1217	case CSSPrimitiveValue::CSS_COUNTER:	1219	case CSSPrimitiveValue::CSS_COUNTER:
1218	text = "counter(";	1220	text = "counter(";




                if ( args->size() != 1)
                    return false;
                Value *a = args->current();
                if (a->unit != CSSPrimitiveValue::CSS_IDENT) {
                    isValid=false;
                    break;
                }
                if (qString(a->string)[0] == '-') {
                    isValid=false;
                    break;
                }
                parsedValue = new CSSPrimitiveValueImpl(domString(a->string), CSSPrimitiveValue::CSS_ATTR);
            }
            else


    CounterImpl *counter = new CounterImpl;
    Value *i = args->current();
    if (i->unit != CSSPrimitiveValue::CSS_IDENT) goto invalid;
    if (qString(i->string)[0] == '-') goto invalid;
    counter->m_identifier = domString(i->string);
    if (counters) {
        i = args->next();

Return to bug 279187

Lines 1513-1518 Link Here

(-)branches/KDE/4.3/kdelibs/khtml/css/cssparser.cpp (-1 / +10 lines)
1513	if ( args->size() != 1)	1513	if ( args->size() != 1)
1514	return false;	1514	return false;
1515	Value *a = args->current();	1515	Value *a = args->current();
		1516	if (a->unit != CSSPrimitiveValue::CSS_IDENT) {
		1517	isValid=false;
		1518	break;
		1519	}
		1520	if (qString(a->string)[0] == '-') {
		1521	isValid=false;
		1522	break;
		1523	}
1516	parsedValue = new CSSPrimitiveValueImpl(domString(a->string), CSSPrimitiveValue::CSS_ATTR);	1524	parsedValue = new CSSPrimitiveValueImpl(domString(a->string), CSSPrimitiveValue::CSS_ATTR);
1517	}	1525	}
1518	else	1526	else
Lines 1565-1571 Link Here
1565		1573
1566	CounterImpl *counter = new CounterImpl;	1574	CounterImpl *counter = new CounterImpl;
1567	Value *i = args->current();	1575	Value *i = args->current();
1568	// if (i->unit != CSSPrimitiveValue::CSS_IDENT) goto invalid;	1576	if (i->unit != CSSPrimitiveValue::CSS_IDENT) goto invalid;
		1577	if (qString(i->string)[0] == '-') goto invalid;
1569	counter->m_identifier = domString(i->string);	1578	counter->m_identifier = domString(i->string);
1570	if (counters) {	1579	if (counters) {
1571	i = args->next();	1580	i = args->next();